- Issue created by @a.dmitriiev
- 🇩🇪Germany a.dmitriiev
Be aware, there is a change in this MR that is done to OpenAIProvider base class. In the MR from this issue https://www.drupal.org/project/ai_provider_openai/issues/3531260 ✨ Remove OpenAI SDK dependency and extend it from Drupal AI core module Active the plugin from ai_provider_openai now extends the base class, but method
chat
is still the old one and there OpenAI iterator is used, not the one from the AI Core module and also the stream output usage settings is not passed. So when testing, using the code from MR https://git.drupalcode.org/project/ai_provider_openai/-/merge_requests/4... but also comment outchat
method, so the one from base class is used. - Merge request !731Issue #3535010: Streamed response is not fully logged → (Open) created by a.dmitriiev
- 🇩🇪Germany a.dmitriiev
Probably the same changes would be needed in other places where Streamed output is used, but let's start with CKEditor submodule, as an example.
- 🇩🇪Germany marcus_johansson
Check: https://www.drupal.org/project/ai/issues/3526936#comment-16185232 🐛 PostStreamingResponseEvent never gets triggered Active
I think we can use that issue as the minor things needed to fix in 1.1.0 for the bug and the structural changes needed for 1.2.0 here if you agree?
- 🇩🇪Germany a.dmitriiev
I converted this MR to enable usage report for OpenAI based provider with streamed responses and also cherry pick the change from here https://www.drupal.org/project/ai_provider_openai/issues/3535032 🐛 Add trigger of the poststreamevent for logging Active to the base class of Open AI based message iterator.