-
Notifications
You must be signed in to change notification settings - Fork 117
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
Added azureai client #36
base: main
Are you sure you want to change the base?
Conversation
README.md
Outdated
### AzureAI Compatible APIs | ||
```bash | ||
export AZUREAI_API_KEY=secret_abcdefg | ||
export AZUREAI_API_BASE="https://api.endpoints.anyscale.com/v1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be an example Azure AI endpoint instead of an anyscale endpoint?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated the API_BASE to .ai.azure.com
Signed-off-by: VindyaKonjarla <[email protected]>
Is it normal there is no difference compared to Note : |
Signed-off-by: Sameep Kumar Panigrahi <[email protected]>
@NicolasGrosjeanProbayes , Updated the file for the ValueError. Please review.And on your question, that is expected. |
I am a simple user, I have no write access, so my review will bring nothing. I don't see the added value compared to OpenAI Compatible API if the code is the same. Maybe I don't see the need to have the parameters in |
No description provided.