-
Notifications
You must be signed in to change notification settings - Fork 16.2k
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
LangChain Community: AzureCosmosDBNoSqlVectorStore: Improvements to the Hybrid Search #29621
base: master
Are you sure you want to change the base?
LangChain Community: AzureCosmosDBNoSqlVectorStore: Improvements to the Hybrid Search #29621
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…3011/langchain into users/akataria/cosmosdb_retriever
@baskaryan , can you please review the PR. Can you also please look into the one lint pipeline that is failing, I am not sure what is the issue there. |
text_key: str = "text", | ||
embedding_key: str = "embedding", |
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.
There are lots of breaking changes in this PR. Ideally, any user code that was working before this change will continue to work after. Is that possible?
Improvements to Hybrid Search
Adding support for multiple text fields to be used for Full Text Search
Adding support for VectorStoreRetriever
Updated test cases for the above
Updated the integration notebooks
Add tests and docs: If you're adding a new integration, please include
docs/docs/integrations
directory.Lint and test: Run
make format
,make lint
andmake test
from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/Additional guidelines:
If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, ccurme, vbarda, hwchase17.