Skip to content
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 RAG Storage #524

Merged
merged 10 commits into from
Jan 8, 2025
Merged

feat: Qdrant RAG Storage #524

merged 10 commits into from
Jan 8, 2025

Conversation

Anush008
Copy link
Contributor

@Anush008 Anush008 commented Dec 24, 2024

Description

This PR adds support for Qdrant - https://qdrant.tech/ to be used as RAG storage.

@Anush008 Anush008 marked this pull request as ready for review December 24, 2024 16:46
Signed-off-by: Anush008 <[email protected]>
Copy link
Member

@pmeier pmeier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Anush008 👋 and thanks a ton for the contribution. The patch looks really good already! I have a few small comments, but otherwise this is almost good to go!

Don't worry about the failing doc CI. I'm currently catching up after some hiatus and haven't gotten around to open an issue for it.

ragna/source_storages/_qdrant.py Show resolved Hide resolved
ragna/source_storages/_qdrant.py Show resolved Hide resolved
ragna/source_storages/_qdrant.py Show resolved Hide resolved
ragna/source_storages/_qdrant.py Outdated Show resolved Hide resolved
@Anush008
Copy link
Contributor Author

Anush008 commented Jan 2, 2025

Hi @pmeier. Thanks for reviewing.

ragna-docker.toml Outdated Show resolved Hide resolved
ragna/source_storages/_qdrant.py Outdated Show resolved Hide resolved
ragna/source_storages/_qdrant.py Show resolved Hide resolved
Anush008 and others added 2 commits January 3, 2025 13:29
Co-authored-by: Philip Meier <[email protected]>
Signed-off-by: Anush008 <[email protected]>
@pmeier
Copy link
Member

pmeier commented Jan 3, 2025

Thanks Anush, LGTM! Any remaining nits that I might have, I'll fix myself. However, we need to wait until #524 (comment) is resolved. @smokestacklightnin is actively working on that. I'll merge afterwards if CI stays green.

Copy link
Member

@pmeier pmeier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Anush008 Turns out Chroma is the outlier and Qdrant is actually aligned with the filtering behavior of LanceDB and what Chroma used to do. Thus, there is no need to do anything else. Let's merge!

@pmeier pmeier merged commit 0f6693b into Quansight:main Jan 8, 2025
3 of 11 checks passed
@Anush008
Copy link
Contributor Author

Anush008 commented Jan 8, 2025

🥳

@Anush008 Anush008 deleted the qdrant-feat branch January 9, 2025 07:03
@pmeier pmeier mentioned this pull request Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants