-
-
Notifications
You must be signed in to change notification settings - Fork 497
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]: Azure Asynchronous Filter #407
Comments
Thank you for the issue and PR. The Azure API and the default OpenAI API are becoming increasingly divergent. Not sure if it is the right way to modify this package more and more, because this will make the response objects harder to use in context of the original OpenAI and still not fully support OpenAI on Azure. We will have to discuss how to proceed here. |
Hello @gehrisandro, i fully agree with you. Meanwhile, can we merge the pull request so we can continue working? I would like to avoid forking two repositories (we use Laravel) just for one line of code :/ |
I have merged your PR, as it affects the direct OpenAI users, not really. I will make a new release the next days. But still, I think having a dedicated package would be the best choice. Having two different packages is of course a maintenance burden, but I can't think of a good way to have both in a single package. Atm, I don't have time to create (and test) a full Azure implementation, but I will keep this in mind, for later. |
Thank you for this PR and the merge! @gehrisandro I'm looking forward to the release :-) Is there a possibility to help? |
Description
Enabling Asynchronous Filter will cause a
Undefined array key "delta"
errorSteps To Reproduce
Enable Asynchronous Filter on Azure OpenAI Studio
Create a OpenAI client that works with Azure
OpenAI PHP Client Version
v0.9.1
PHP Version
8.3.6
Notes
No response
The text was updated successfully, but these errors were encountered: