-
Notifications
You must be signed in to change notification settings - Fork 14.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
qdrant: Documentation for the new QdrantVectorStore class #24166
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
hey! I think my comment wasn't clear. Let's delete qdrant-new.ipynb
and merge any relevant contents into qdrant.ipynb! If the old one is deprecated/not recommended for new users, let's either delete those sections of qdrant.ipynb, or put them at the bottom with a note that you recommend using the new implementation instead.
As we've discussed before, having these two separate implementations is going to be confusing for users, so some notes about why someone might use one vs the other would be appreciated as well!
Hey @efriis. I've updated the changes into The page documents only the new usage pattern. |
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.
looks great thank you!
Description
Follow up on #24165. Adds a page to document the latest usage of the new
QdrantVectorStore
class.