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

Fix the wrong return value of onnx similarity evaluation #460

Closed SimFG closed 1 year ago

SimFG commented 1 year ago

issue: #459 #386

sre-ci-robot commented 1 year ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: SimFG

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/zilliztech/GPTCache/blob/dev/OWNERS)~~ [SimFG] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
codecov[bot] commented 1 year ago

Codecov Report

Merging #460 (234662d) into dev (de35ae6) will decrease coverage by 0.03%. The diff coverage is 83.33%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/zilliztech/GPTCache/pull/460/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/460?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech) ```diff @@ Coverage Diff @@ ## dev #460 +/- ## ========================================== - Coverage 94.02% 93.99% -0.03% ========================================== Files 92 92 Lines 3583 3583 ========================================== - Hits 3369 3368 -1 - Misses 214 215 +1 ``` | [Impacted Files](https://app.codecov.io/gh/zilliztech/GPTCache/pull/460?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/manager.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/460?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvbWFuYWdlci9zY2FsYXJfZGF0YS9tYW5hZ2VyLnB5) | `90.47% <0.00%> (-4.77%)` | :arrow_down: | | [gptcache/processor/post.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/460?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvcHJvY2Vzc29yL3Bvc3QucHk=) | `100.00% <ø> (ø)` | | | [gptcache/similarity\_evaluation/distance.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/460?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvc2ltaWxhcml0eV9ldmFsdWF0aW9uL2Rpc3RhbmNlLnB5) | `100.00% <ø> (ø)` | | | [gptcache/adapter/adapter.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/460?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvYWRhcHRlci9hZGFwdGVyLnB5) | `87.32% <100.00%> (ø)` | | | [gptcache/similarity\_evaluation/onnx.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/460?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvc2ltaWxhcml0eV9ldmFsdWF0aW9uL29ubngucHk=) | `95.83% <100.00%> (ø)` | | | [gptcache/utils/softmax.py](https://app.codecov.io/gh/zilliztech/GPTCache/pull/460?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zilliztech#diff-Z3B0Y2FjaGUvdXRpbHMvc29mdG1heC5weQ==) | `100.00% <100.00%> (ø)` | |