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
feat: add momento vector index as a vector store provider #11567
feat: add momento vector index as a vector store provider #11567
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Adds Momento Vector Index as a vector store provider. Provides implementation, integration tests, examples, and documentation.
The standard in the Momento ecosystem is now to refer to `MOMENTO_API_KEY` as opposed to `MOMENTO_AUTH_TOKEN`. This changes refactors to prefer the former, while keeping backwards compatibility.
For ease of use allow users to instantiate the client either by passing in a client object (dependency injection) or with the client parameters.
b5b0847
to
ded2e78
Compare
I rebased and solved merge conflicts in ded2e78 |
) | ||
""" | ||
|
||
_client: "PreviewVectorIndexClient" |
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.
VectorStore isn't a pydantic BaseModel, so these will all be class attributes. is that intentional?
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.
No it wasn't. Good catch!
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.
Fixed in d9010a6
lgtm, thanks @malandis! |
Thanks @baskaryan and @hwchase17 🙇 . Any chance @momentohq can get a Twitter shoutout for the contribution so we can reshare also 🙏 ? |
…ai#11567) **Description**: - Added Momento Vector Index (MVI) as a vector store provider. This includes an implementation with docstrings, integration tests, a notebook, and documentation on the docs pages. - Updated the Momento dependency in pyproject.toml and the lock file to enable access to MVI. - Refactored the Momento cache and chat history session store to prefer using "MOMENTO_API_KEY" over "MOMENTO_AUTH_TOKEN" for consistency with MVI. This change is backwards compatible with the previous "auth_token" variable usage. Updated the code and tests accordingly. **Dependencies**: - Updated Momento dependency in pyproject.toml. **Testing**: - Run the integration tests with a Momento API key. Get one at the [Momento Console](https://console.gomomento.com) for free. MVI is available in AWS us-west-2 with a superuser key. - `MOMENTO_API_KEY=<your key> poetry run pytest tests/integration_tests/vectorstores/test_momento_vector_index.py` **Tag maintainer:** @eyurtsev **Twitter handle**: Please mention @momentohq for this addition to langchain. With the integration of Momento Vector Index, Momento caching, and session store, Momento provides serverless support for the core langchain data needs. Also mention @mlonml for the integration.
Description:
Dependencies:
Testing:
MOMENTO_API_KEY=<your key> poetry run pytest tests/integration_tests/vectorstores/test_momento_vector_index.py
Tag maintainer:
@eyurtsev
Twitter handle:
Please mention @momentohq for this addition to langchain. With the integration of Momento Vector Index, Momento caching, and session store, Momento provides serverless support for the core langchain data needs.
Also mention @mlonml for the integration.