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

Added vector index interface and fixed import errors #3429

Merged
merged 3 commits into from
Jun 3, 2023

Conversation

tgaddair
Copy link
Collaborator

@tgaddair tgaddair commented Jun 2, 2023

Short term goal is to improve error messages like those seen in #3427 when deps aren't installed. Longer term, this opens up an easy path to onboard additional vector index solutions.

@github-actions
Copy link

github-actions bot commented Jun 2, 2023

Unit Test Results

  6 files  ±0    6 suites  ±0   1h 20m 57s ⏱️ - 5m 57s
33 tests ±0  29 ✔️ ±0    4 💤 ±0  0 ±0 
99 runs  ±0  87 ✔️ ±0  12 💤 ±0  0 ±0 

Results for commit 5f3a120. ± Comparison against base commit 09351a3.

♻️ This comment has been updated with latest results.

@tgaddair tgaddair merged commit 29f720d into master Jun 3, 2023
16 checks passed
@tgaddair tgaddair deleted the llm-deps-errors branch June 3, 2023 06:09
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