-
Notifications
You must be signed in to change notification settings - Fork 720
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
Unrecognized request argument supplied: azure on GitHub Actions #356
Comments
i was able to reproduce that with a new version of litellm |
Hi @mrT23, Re: improving package stability
Open to any additional ideas / suggestions you might have for how we can improve things. Re: azure I think that could've been better communicated, and some of the changes we're making (e.g. semvars, guides, etc.) are related to that. We don't anticipate or want to make changes to completion() params, and i think the Have you seen any issues beyond this? |
I'm also happy to reintroduce the azure flag check, so we're backwards compatible. |
@krrishdholakia i reverted the line to:
is it correct now ? |
maybe it should be:
? |
Yes - it should be 'azure/' + model |
Please tag me in any issues that come up due to issues on our end @mrT23 Want to make sure we're a stable, reliable solution for y'all. |
Here's our azure docs for context. |
@krrishdholakia thanks for the quick response. i updated the code |
Sounds good - let me know how we can better communicate changes (if any occur). |
@higebu note that you can also work against a stable docker, to ensure no surprises
|
@mrT23 @krrishdholakia Thank you for the quick fix! |
On GitHub Actions, the error
openai.error.InvalidRequestError: Unrecognized request argument supplied: azure
is raised.GitHub Actions yaml:
Full error log:
The text was updated successfully, but these errors were encountered: