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: Adding sqlite-vec to default installation #5138

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jyejare
Copy link

@jyejare jyejare commented Mar 11, 2025

What this PR does / why we need it:

The changes in the PR:

  • adds the sqlite-vec dependency to the required list of installation from the optional dependencies.
  • The changes are made in both pyproject and setup files

Which issue(s) this PR fixes:

#5103

Misc

@jyejare jyejare force-pushed the required-sqlite-vec branch from 4699843 to 22508dc Compare March 11, 2025 14:10
@franciscojavierarceo franciscojavierarceo self-requested a review March 11, 2025 14:24
@jyejare jyejare requested a review from a team as a code owner March 11, 2025 14:38
@franciscojavierarceo
Copy link
Member

unit tests are failing for torch, thing we need to pin that exact version.

@jyejare jyejare force-pushed the required-sqlite-vec branch from 6ed39c3 to e8a5e48 Compare March 11, 2025 15:04
Signed-off-by: jyejare <jyejare@redhat.com>
@jyejare jyejare force-pushed the required-sqlite-vec branch from e8a5e48 to 3202eeb Compare March 12, 2025 18:26
Signed-off-by: jyejare <jyejare@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants