Fix recall computation for fewer than k groundtruth results#1069

Merged

magdalendobson merged 20 commits into

mainfrom

users/magdalen/patch_recall_calculation

May 20, 2026

Commits on May 14, 2026

finish up recall computation patch
Magdalen Manohar
committed
fix bug in recall computation
Magdalen Manohar
committed
remove stray extra registered benchmarks
Magdalen Manohar
committed
Potential fix for pull request finding

magdalendobson
and
Copilot
authored
Fix groundtruth ncols check: use < instead of != to allow files with more neighbors than needed

Copilot
and
magdalendobson
authored
Add meaningful recall value assertions for variable-length dynamic groundtruth tests

Copilot
and
magdalendobson
authored
Update doc comment for average field and add tests for zero/non-zero groundtruth mix

Copilot
and
magdalendobson
authored
Remove trailing whitespace from test code

Copilot
and
magdalendobson
authored

Commits on May 15, 2026

Merge branch 'main' of github.com:microsoft/DiskANN into users/magdalen/patch_recall_calculation
Magdalen Manohar
committed
small changes to tests
Magdalen Manohar
committed
fmt
Magdalen Manohar
committed
fix clippy, add max_k() function to topk search
Magdalen Manohar
committed
add a test for loading groundtruth with expected k
Magdalen Manohar
committed

Commits on May 19, 2026

apply suggestions from review
Magdalen Manohar
committed
rename error, add test for new error
Magdalen Manohar
committed
fmt + clippy
Magdalen Manohar
committed
Merge main into users/magdalen/patch_recall_calculation

Copilot
and
magdalendobson
authored

Commits on May 20, 2026

improve handling when not enough groundtruth
Magdalen Manohar
committed
added Debug, update docstring
Magdalen Manohar
committed
fmt
Magdalen Manohar
committed