-
Notifications
You must be signed in to change notification settings - Fork 424
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
Support for Azure OpenAi embedding model #692
Comments
Hi @shubhiroy! Thanks for offering to contribute to this. Should be very straightforward to add, very similar to https://github.com/NVIDIA/NeMo-Guardrails/blob/develop/nemoguardrails/embeddings/providers/openai.py. Let me know if you need more guidance. Looking forward to your PR! |
@shubhiroy and @drazvan - Could you please confirm if this feature is ready? |
Hi @Narayanan12345 , PR is still in pending state. Waiting for some reviewer to merge it. |
@Narayanan12345 , @ShubhiAbb , @shubhiroy There is a conflict with dev branch that needs to be resolved by @shubhiroy once the conflict is resolved I'll merge. |
@Pouyanpi - I will resolve it in this week. Thanks. |
I'd like to propose adding support for the Azure OpenAI embedding model. My primary reason for this is to enable the use of the text-embedding-ada-002 model as the embedding model.
I believe this could be beneficial for those who are already using Azure's services or prefer to work within that ecosystem. I'm also happy to contribute to implementing this feature. Would there be any preference on how to integrate Azure OpenAI support? We could potentially add it in the configuration file, similar to other models.
The text was updated successfully, but these errors were encountered: