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

[Feature] Add min_nprobe to get_optimal_hyperparameters function #174

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

victor-paltz
Copy link
Contributor

@victor-paltz victor-paltz commented Sep 29, 2023

Some IVF KNN indices built by autofaiss have bad recall scores due to the unreachable speed constraints given in the requirements.

A way to guarantee decent recall scores is to force a minimal value to nprobe for IVF-like indices (it was already the case for IVF_HNSW indices with a min_ef_search parameter).

This commit makes it possible to set a custom value for min_nprobe

@rom1504
Copy link
Contributor

rom1504 commented Jan 12, 2024

There is a merge conflict 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

Successfully merging this pull request may close these issues.

2 participants