-
Notifications
You must be signed in to change notification settings - Fork 236
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
Fix couchbase vectordb support #987
base: main
Are you sure you want to change the base?
Conversation
…pport Signed-off-by: Mark Sze <[email protected]>
Signed-off-by: Mark Sze <[email protected]>
Signed-off-by: Mark Sze <[email protected]>
Signed-off-by: Mark Sze <[email protected]>
Signed-off-by: Mark Sze <[email protected]>
Signed-off-by: Mark Sze <[email protected]>
Signed-off-by: Mark Sze <[email protected]>
Signed-off-by: Mark Sze <[email protected]>
@@ -112,6 +112,11 @@ retrievechat-qdrant = [ | |||
"fastembed>=0.3.1", | |||
] | |||
|
|||
retrievechat-couchbase = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found 1 issue is " import error for SentenceTransformer" which is package incompatibility. It could be resolved by install "urllib3<2". Don't you have the same error before?
Hey @davorrunje @marklysze , can I help you with a free forever couchbase cloud cluster for testing? The only drawback of the free cluster is that it will turn down every 3 days if there's no activity and will require manual process to turn it back on. |
@@ -116,6 +122,9 @@ jobs: | |||
- name: Install pgvector when on linux | |||
run: | | |||
uv pip install --system -e .[test,retrievechat-pgvector,retrievechat-mongodb] | |||
- name: Install couchbase when on linux |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this pip install can be combined with the above pip install statement at line 124
Why are these changes needed?
Related issue number
Checks