big-ann-benchmarks icon indicating copy to clipboard operation
big-ann-benchmarks copied to clipboard

Range results order flipped, need to update HTTPANN

Open harsha-simhadri opened this issue 2 years ago • 2 comments

@alexklibisz Since FAISS used D,I order to return range search results, in a recent commit, I changed the range search result return order in the base ANN class to D,I.

Please consider updating HttpANN class as well.

harsha-simhadri avatar Oct 07 '21 09:10 harsha-simhadri

Interesting. How did the tests still pass? Do they not check this?

alexklibisz avatar Oct 07 '21 12:10 alexklibisz

Hmm, it did pass. Let me see what happened here.

harsha-simhadri avatar Oct 07 '21 17:10 harsha-simhadri