-
Notifications
You must be signed in to change notification settings - Fork 0
Introduce Valkey Vector Store for LangChain AWS #1
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
base: develop
Are you sure you want to change the base?
Conversation
MatthiasHowellYopp
left a comment
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.
Minor questions - basically looks good.
| print(f" Score: {score:.4f}") | ||
| print(f" Metadata: {doc.metadata}\n") | ||
|
|
||
| # Example with filtering (if filters are implemented) |
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.
do we not know if filters are implemented? or this depends on some other variable implementation that may or may not be present?
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.
don't know enough about python to know if an empty init file makes sense or not.
94672a2 to
456c553
Compare
456c553 to
07e6ebf
Compare
| name: str = Field(...) | ||
|
|
||
|
|
||
| class TextFieldSchema(ValkeyField): |
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.
Valkey doesn't support TEXT field yet. How does the schema creation work?
This is a draft PR, set for merging into another temporary branch (for now) for the primary purpose of collecting initial feedback from internal BitQuill developers before promoting it to the LangChain team.
See also the Valkey Vector Store GH Issue for tracking purposes.