support concurrent add in ivf #780
Conversation
Codecov ReportAttention: Patch coverage is @@ Coverage Diff @@
## main #780 +/- ##
==========================================
- Coverage 90.63% 90.13% -0.50%
==========================================
Files 218 222 +4
Lines 14444 14991 +547
==========================================
+ Hits 13091 13512 +421
- Misses 1353 1479 +126
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
|
bugbot run |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
| @@ -800,6 +800,9 @@ TestIVFConcurrentAdd(const fixtures::IVFTestIndexPtr& test_index, | |||
| for (auto dim : resource->dims) { | |||
| for (auto train_type : resource->train_types) { | |||
| for (auto& [base_quantization_str, recall] : resource->test_cases) { | |||
| if (base_quantization_str == "pqfs,fp16") { | |||
There was a problem hiding this comment.
pqfs doesn't support concurrent due the package
There was a problem hiding this comment.
package only used after build
|
bugbot run |
|
@inabao cannot cherry-pick this pull request (conflict), please create a new request to branch 0.15 |
Signed-off-by: jinjiabao.jjb <jinjiabao.jjb@antgroup.com>
No description provided.