erikbern / ann-benchmarks

Benchmarks of approximate nearest neighbor libraries in Python
http://ann-benchmarks.com
MIT License
4.88k stars 735 forks source link

Fix hamming distance test #513

Closed C0rWin closed 5 months ago

ankane commented 5 months ago

Hi @C0rWin, check out #508.

C0rWin commented 5 months ago

Hi @C0rWin, check out #508.

I might be missing something, but master still has this issue:

https://github.com/erikbern/ann-benchmarks/blob/75043ab482d91afae82ac4033cbdd98997121d58/ann_benchmarks/distance.py#L30

ankane commented 5 months ago

That code is correct for the benchmarks to show the correct recall with the hosted datasets (see the linked issue or test it yourself).