Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Range results order flipped, need to update HTTPANN #48

Open
harsha-simhadri opened this issue Oct 7, 2021 · 2 comments
Open

Range results order flipped, need to update HTTPANN #48

harsha-simhadri opened this issue Oct 7, 2021 · 2 comments

Comments

@harsha-simhadri
Copy link
Owner

@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.

@alexklibisz
Copy link
Contributor

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

@harsha-simhadri
Copy link
Owner Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants