microsoft / DiskANN

Graph-structured Indices for Scalable, Fast, Fresh and Filtered Approximate Nearest Neighbor Search
Other
1.11k stars 218 forks source link

Fix calculation of current_point_offset in test_insert_consolidate_deletes #501

Closed theantony closed 9 months ago

theantony commented 10 months ago

The program builds the streaming index after two optional steps: 1) skipping S points from the input file and 2) batch building of initial index using B points from the input file.

After these two steps, the offset to the input file should be S + B, but the current code first sets it to S in line 163 then overwrites it to B in line 249, instead of adding B to the offset. The tool which test_insert_deletes_consolidate was based on was using += in the modified line.