Update Azure OpenAI to check ModelAuthProvider during Embedding and Image model creation #1162
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1155.
@geoand I've updated it the same way we did checks for
ChatModel
andStreamingChatModel
.I've confirmed that with this PR, after deploying
text-embedding-ada-002
in my Azure OpenAI account, with thesecure-poem-multiple-providers
demo, that if I addand have
EmbeddingModel
injected and the Azure OpenAI related method updated toI get the token count and logs confirming the remote call was successful, with the user authentication token:
I haven't actually updated the demo, because Google authentication plus Gemini is also shown there and it is not clear to me yet what Vertex-AI offers for embedding models, while having Azure OpenAI only embedding model would impact users who only want to try the demo with Gemini... May be we can do an advanced, everything Azure OpenAI kind of demo later, but for now, I can confirm the updates in this PR are sufficient.