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.

Sign up for GitHub

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

Jump to bottom

fixed langchain embedding call for azure: #87

Open

someguy234 wants to merge 2 commits into danny-avila:main from someguy234:main

someguy234 commented Oct 14, 2024

See this issue/fix for reasoning. Otherwise RAG w/ Azure endpoints fails.
langchain-ai/langchain#4575

someguy234 and others added 2 commits

December 18, 2024 13:54


          fixed langchain embedding call for azure: langchain-ai/langchain#4575

c6d6d1a


          Fixing issue w/ Azure embeddings

b77ccaa

someguy234 force-pushed the main branch from 5614ded to b77ccaa Compare

December 18, 2024 19:28

rgknight commented Jan 30, 2025

A couple of notes:

You added the environment variable twice
The issue you linked to is closed. A fix was implemented in Sept 2023. Seems like shouldn't be need this if the langchain version used by this repo is at all up to date?
Does RAG on Azure endpoints fail for you? What is happening?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet