-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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: Qdrant storage connector #1023
Conversation
Hi @cpacker. Could you please take a look when possible please? |
Thanks for submitting this PR! We are happy to support more storage backends, however since we are a fairly small team, we ask that external contributors continue to maintain their storage integrations over time. MemGPT is under active development, so it's possible that the storage connectors will need to be refactored or modified in the future, and our team can only commit to maintaining the default storage connectors. |
Hi, @sarahwooders. I've rechecked the implementation as per recent updates. Please take a look when possible. |
An update.
|
Failing https://github.com/cpacker/MemGPT/actions/runs/9253848636/job/25755429975?pr=1023 appears to be due to missing credentials. |
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.
lgtm - thanks for the contribution! we can update the documentation sometime tomorrow
|
Description
This PR adds support for Qdrant - https://qdrant.tech/ to be used as a storage connector.
Tests
tests/test_storage.py
andtests/load_archival.py
.If a Qdrant instance is not running or the client library is not installed the Qdrant tests are skipped.
To run Qdrant server