Skip to content

Adding support of multi-vector collection for Qdrant driver #312

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

Merged
merged 6 commits into from
Mar 31, 2025

Conversation

Dammerzone
Copy link
Contributor

@Dammerzone Dammerzone commented Mar 19, 2025

Description

Adding the support of multi-vector for Qdrant driver

This changes enables the selection of a specific vector to search in when querying point in multi-vector collectionusing Qdrant driver

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Documentation update
  • Project configuration change

Complexity

Low complexity due to the addition of an already existing parameter of the qdrand native client

Complexity: Low

How Has This Been Tested?

  • Unit tests
  • E2E tests
  • Manual tests

Checklist

The following requirements should have been met (depending on the changes in the branch):

  • Documentation has been updated
  • Unit tests have been updated
  • E2E tests have been updated
  • Examples have been updated
  • New files have copyright header
  • CLA (https://neo4j.com/developer/cla/) has been signed
  • CHANGELOG.md updated if appropriate

@Dammerzone Dammerzone requested a review from a team as a code owner March 19, 2025 17:10
@Dammerzone
Copy link
Contributor Author

Just signed the CLA

@stellasia
Copy link
Contributor

@CodiumAI-Agent /update_changelog

@CodiumAI-Agent
Copy link

Changelog updates: 🔄

@neo4j/neo4j-graphrag-python

Next

Added

  • Enabled multi-vector collection support for Qdrant driver.

to commit the new content to the CHANGELOG.md file, please type:
'/update_changelog --pr_update_changelog.push_changelog_changes=true'

@stellasia
Copy link
Contributor

Hi @Dammerzone ,

Thanks for your contribution and for signing the CLA. The change looks good to me. Once the CI is fixed, may I ask you to add a line in the CHANGELOG?

@CodiumAI-Agent
Copy link

Changelog updates: 🔄

Added

  • Added support for multi-vector collection in Qdrant driver.

to commit the new content to the CHANGELOG.md file, please type:
'/update_changelog --pr_update_changelog.push_changelog_changes=true'

@Dammerzone
Copy link
Contributor Author

Hello @stellasia, thanks for the review, I'll do that !

@stellasia
Copy link
Contributor

@CodiumAI-Agent /update_changelog --pr_update_changelog.push_changelog_changes=true

@stellasia
Copy link
Contributor

Hi @Dammerzone , Thank you for fixing the CI and keeping your branch up-to-date. The only missing piece is the CHANGELOG update, you can use the automatically generated proposal above or anything else, up to you :)

@Dammerzone
Copy link
Contributor Author

Dammerzone commented Mar 31, 2025

Oh my bad, I thought you already done that using the command ! Its done 👍

Copy link
Contributor

@stellasia stellasia left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

Oh my bad, I thought you already done that using the command !

Yeah sorry, I also thought it'd do it but apparently we haven't given the bot write permission to the repo. Thank you!

@stellasia stellasia merged commit bb8fe0a into neo4j:main Mar 31, 2025
7 checks passed
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.

3 participants