Skip to content

Conversation

@meiravgri
Copy link
Collaborator

@meiravgri meiravgri commented Sep 8, 2025

backport of #768 to 0.6

Add virtual indexCapacity() API to VecSimIndexInterface

API Changes

  • BruteForce: Capacity determined based on idToLabelMapping size (aligned with its capacity)
  • HNSW: Rename existing getIndexCapacity and change it to override the new virtual API

meiravgri and others added 2 commits September 8, 2025 16:44
* imp bemchmar

fix counters

* fix

(cherry picked from commit
fd08b55)

* add indexCapacity API
@codecov
Copy link

codecov bot commented Sep 9, 2025

Codecov Report

❌ Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 94.99%. Comparing base (520531a) to head (a10bc71).
⚠️ Report is 1 commits behind head on 0.6.

Files with missing lines Patch % Lines
src/VecSim/algorithms/brute_force/brute_force.h 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              0.6     #773      +/-   ##
==========================================
- Coverage   95.04%   94.99%   -0.06%     
==========================================
  Files          60       60              
  Lines        3432     3434       +2     
==========================================
  Hits         3262     3262              
- Misses        170      172       +2     

☔ 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 changed the title Backport-768-to-0.6 [0.6] [MOD-11237] Block Size Boundary Oscillation Benchmark (#768 Sep 9, 2025
@meiravgri meiravgri added this pull request to the merge queue Sep 11, 2025
Merged via the queue into 0.6 with commit 71bd103 Sep 11, 2025
25 of 27 checks passed
@meiravgri meiravgri deleted the backport-768-to-0.6 branch September 11, 2025 09:04
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.

3 participants