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

Add integration for Marqo vector store #609

Open
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

sky-2002
Copy link

This PR adds marqo as a vector store to GPTCache. It was requested here in marqo's issues.

Important:

  • Currently, this integration would work with marqo version 1.5.0
  • To run smoothly, install client marqo==2.1.0 with pip install marqo==2.1.0. I could not use this version based format in import_marqo function as specified. Would appreciate suggestions on this.
  • Marqo 2.0 still has to implement some features which are present in 1.5.0, I will update this integration when they release those features in version 2.0 onwards.

Discussions and suggestions are welcome!

Note: I have tested the functionality individually. But due to open AI credits issue, could not test it end-2-end, would be great if someone could do that.

@sre-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: sky-2002
To complete the pull request process, please assign simfg after the PR has been reviewed.
You can assign the PR to them by writing /assign @simfg in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sre-ci-robot
Copy link
Collaborator

Welcome @sky-2002! It looks like this is your first PR to zilliztech/GPTCache 🎉

Signed-off-by: Aakash Thatte <[email protected]>
Signed-off-by: Aakash Thatte <[email protected]>
Signed-off-by: Aakash Thatte <[email protected]>
Signed-off-by: Aakash Thatte <[email protected]>
Signed-off-by: Aakash Thatte <[email protected]>
Signed-off-by: Aakash Thatte <[email protected]>
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.

2 participants