Skip to content

Conversation

@github-actions
Copy link

Description

Backport of #740 to 8.2.

* add fp16 tests

* more tests

* add printing utility to VecSimQueryReply and VecSimQueryResult

more tests

* more tests

add validateSVSIndexAttributesInfo to compare info to params

add populate_float16_vec

* more tests

* imp getStoredVectorDataByLabel for multi and no-compressed

more tests

* fix quant_modes

* multi index tests

* tiered tests

* fix test to fail

* temproraly push test_override_all to float32 tests

* update to crash

* revert svs tests chnages

fix cosine batch iterators tests and all fp16 tests

* test

* add range test

* skip test_get_distance for scalaar quant

* align svs_get_distance name test with test_svs.cpp

* clean ups

* add coverage to print VecSimQueryResult and VecSimQueryReply

* guard validateSVSIndexAttributesInfo with #if HAVE_SVS for the include

* fix

* align names with main branhc

(cherry picked from commit 00bf35d)
@GuyAv46 GuyAv46 marked this pull request as draft October 15, 2025 09:30
@GuyAv46 GuyAv46 marked this pull request as ready for review October 15, 2025 09:30
@GuyAv46 GuyAv46 requested a review from meiravgri October 15, 2025 09:30
@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

❌ Patch coverage is 96.15385% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 96.86%. Comparing base (ee075bc) to head (2bb0f86).
⚠️ Report is 1 commits behind head on 8.2.

Files with missing lines Patch % Lines
src/VecSim/algorithms/svs/svs.h 94.73% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              8.2     #802      +/-   ##
==========================================
+ Coverage   96.68%   96.86%   +0.17%     
==========================================
  Files         126      126              
  Lines        7670     7695      +25     
==========================================
+ Hits         7416     7454      +38     
+ Misses        254      241      -13     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@meiravgri meiravgri enabled auto-merge October 15, 2025 12:50
@meiravgri meiravgri added this pull request to the merge queue Oct 15, 2025
Merged via the queue into 8.2 with commit d3edd09 Oct 15, 2025
18 checks passed
@meiravgri meiravgri deleted the backport-740-to-8.2 branch October 15, 2025 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant