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

Improve indexing performance #699

Merged
merged 3 commits into from
Mar 18, 2023
Merged

Improve indexing performance #699

merged 3 commits into from
Mar 18, 2023

Conversation

eddyxu
Copy link
Contributor

@eddyxu eddyxu commented Mar 18, 2023

No description provided.

@eddyxu eddyxu requested a review from changhiskhan March 18, 2023 04:23
@eddyxu eddyxu self-assigned this Mar 18, 2023
@eddyxu
Copy link
Contributor Author

eddyxu commented Mar 18, 2023

Run sift benchmark

Running with ivf512
nprobes: 1, refine=None, recall@10=0.554, mean(s)=0.002198166847229004
nprobes: 1, refine=1, recall@10=0.565, mean(s)=0.0022933125495910645
nprobes: 1, refine=5, recall@10=0.790, mean(s)=0.002348010540008545
nprobes: 1, refine=10, recall@10=0.788, mean(s)=0.002333817481994629
nprobes: 10, refine=None, recall@10=0.667, mean(s)=0.0036499619483947752
nprobes: 10, refine=1, recall@10=0.657, mean(s)=0.003946800231933594
nprobes: 10, refine=5, recall@10=0.954, mean(s)=0.004477522373199463
nprobes: 10, refine=10, recall@10=0.975, mean(s)=0.0038238167762756348
nprobes: 50, refine=None, recall@10=0.656, mean(s)=0.0102131986618042
nprobes: 50, refine=1, recall@10=0.649, mean(s)=0.010562391281127929
nprobes: 50, refine=5, recall@10=0.959, mean(s)=0.011186907291412354
nprobes: 50, refine=10, recall@10=0.991, mean(s)=0.011933960914611817

Copy link
Contributor

@changhiskhan changhiskhan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a blocker but wondering if this shareable with computing distortion.

@eddyxu eddyxu merged commit 32d343e into main Mar 18, 2023
@eddyxu eddyxu deleted the lei/idx_perf branch March 18, 2023 04:58
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