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

Adding Qdrant #2269

Closed
wants to merge 2 commits into from
Closed

Adding Qdrant #2269

wants to merge 2 commits into from

Conversation

mkhludnev
Copy link

@mkhludnev mkhludnev commented May 14, 2024

Pull Request Checklist

  • Target branch: Pull requests should target the dev branch.
  • Description: Introducing Qdrant vectorstore
  • Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
  • Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
  • Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
  • Testing: Have you written and run sufficient tests for the changes?
  • Code Review: Have you self-reviewed your code and addressed any coding standard issues?

Description

It introduces Qdrant as alternative to Chroma. Beforehand it abstract Chroma under VectorStore.


Changelog Entry

Added

  • Qdrant vector store

Fixed

  • [List any fixes, corrections, or bug fixes]

Changed

  • [List any changes, updates, refactorings, or optimizations]

Removed

  • [List any removed features, files, or deprecated functionalities]

Security

  • [List any new or updated security-related changes, including vulnerability fixes]

Breaking Changes

  • [List any breaking changes affecting compatibility or functionality]

Additional Information

  • [Insert any additional context, notes, or explanations for the changes]

  • [Reference any related issues, commits, or other relevant information]

@mkhludnev
Copy link
Author

Just want to show what I'm up to.

  1. Is introducing Qdrant makes any sense at all?
  2. wdyt re approach taken?
  3. is there a tests I can use for covering this?

Once again, it's incomplete. What to get some feedback in the beginning.

@mkhludnev mkhludnev changed the base branch from main to dev May 15, 2024 15:45
@mkhludnev mkhludnev changed the title DRAFT: adding Qdrant Adding Qdrant May 15, 2024
@mkhludnev mkhludnev marked this pull request as ready for review May 15, 2024 16:08
@tjbck
Copy link
Contributor

tjbck commented May 25, 2024

We might want this to be added as a pipeline instead of adding Qdrant directly to the main webui project. https://github.com/open-webui/pipelines

open-webui/pipelines#9

@tjbck
Copy link
Contributor

tjbck commented Jun 3, 2024

Closing in favour of open-webui/pipelines#9

@tjbck tjbck closed this Jun 3, 2024
@mkhludnev
Copy link
Author

@tjbck thanks for reply. I really like pipelines approach. But it's not clear how to pass collection_names via pipelines API to RAG pipeline?

@tjbck
Copy link
Contributor

tjbck commented Jun 3, 2024

@mkhludnev not yet supported but I'll be adding all the custom hooked need to enable such interactions!

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.

None yet

2 participants