zilliztech / GPTCache

Semantic cache for LLMs. Fully integrated with LangChain and llama_index.
https://gptcache.readthedocs.io
MIT License
7.06k stars 500 forks source link

Support for weaviate vector store #474

Closed sunilkumardash9 closed 1 year ago

sunilkumardash9 commented 1 year ago

Added support for the Weaviate vector store #340

sre-ci-robot commented 1 year ago

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

SimFG commented 1 year ago

@sunilkumardash9 thank your contribution. please use the dev as the target branch

sre-ci-robot commented 1 year ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: sunilkumardash9 To complete the pull request process, please assign cxie after the PR has been reviewed. You can assign the PR to them by writing /assign @cxie 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: - **[OWNERS](https://github.com/zilliztech/GPTCache/blob/dev/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
sunilkumardash9 commented 1 year ago

@SimFG I think we need to update the SQLite versions. Python 3.8 probably ships with an older version that isn't supported anymore in chroma.

SimFG commented 1 year ago

@sunilkumardash9 it will be fixed in #488 pr

SimFG commented 1 year ago

@sunilkumardash9 I have handled the error, pr: #492

codecov[bot] commented 1 year ago

Codecov Report

Merging #474 (3dc0c73) into dev (bd152a7) will decrease coverage by 0.14%. The diff coverage is 85.29%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/zilliztech/GPTCache/pull/474/graphs/tree.svg?width=650&height=150&src=pr&token=E30WxqBeJJ&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech)](https://app.codecov.io/gh/zilliztech/GPTCache/pull/474?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech) ```diff @@ Coverage Diff @@ ## dev #474 +/- ## ========================================== - Coverage 94.07% 93.94% -0.14% ========================================== Files 93 94 +1 Lines 3817 3883 +66 ========================================== + Hits 3591 3648 +57 - Misses 226 235 +9 ``` | [Impacted Files](https://app.codecov.io/gh/zilliztech/GPTCache/pull/474?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech) | Coverage Δ | | |---|---|---| | [gptcache/manager/scalar\_data/mongo.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/474?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvbWFuYWdlci9zY2FsYXJfZGF0YS9tb25nby5weQ==) | `92.95% <0.00%> (ø)` | | | [gptcache/manager/vector\_data/weaviate.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/474?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvbWFuYWdlci92ZWN0b3JfZGF0YS93ZWF2aWF0ZS5weQ==) | `82.00% <82.00%> (ø)` | | | [gptcache/manager/vector\_data/manager.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/474?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvbWFuYWdlci92ZWN0b3JfZGF0YS9tYW5hZ2VyLnB5) | `99.19% <100.00%> (+0.10%)` | :arrow_up: | | [gptcache/utils/\_\_init\_\_.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/474?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvdXRpbHMvX19pbml0X18ucHk=) | `89.34% <100.00%> (+0.17%)` | :arrow_up: |
SimFG commented 1 year ago

@sunilkumardash9 i will merge the pr: https://github.com/zilliztech/GPTCache/pull/493, because the pr fully meet the entry requirements. Do you have some ideas? If have, please tell me as soon as possible, it is expected that I will merge today.

sunilkumardash9 commented 1 year ago

@SimFG No sir, you are free to merge.

SimFG commented 1 year ago

@sunilkumardash9 thank for your contribution and attention

SimFG commented 1 year ago

@sunilkumardash9 i will close the pull request. And if you have more ideas, please feel free to open a new pull request