Skip to content

MB-68044: Limiting Training Size of Faiss Indexes - #274

Merged
Likith101 merged 3 commits into
masterfrom
VecEx1
Aug 5, 2026
Merged

MB-68044: Limiting Training Size of Faiss Indexes#274
Likith101 merged 3 commits into
masterfrom
VecEx1

Conversation

@Likith101

@Likith101 Likith101 commented Oct 17, 2024

Copy link
Copy Markdown
Member
  • Added nlist to params. This is only populated during index time
  • Set the number of training vectors to number of centroids * 40

@Likith101

Copy link
Copy Markdown
Member Author

Tested out recall of latency vs latency with limited training using the recall option as the ground truth

Latency => Recall@3: 93.40% Recall@10: 90.32% Recall@100: 85.22%
Latency LT => Recall@3: 92.52% Recall@10: 89.00% Recall@100: 84.03%

@Likith101 Likith101 changed the title Toy: Limited Training Size MB-68044: Limiting Training Size of Faiss Indexes Jul 28, 2026

@Thejas-bhat Thejas-bhat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please resolve the merge conflicts. can you just double check the recall numbers once? also it would be better to make this tunable via bleve/cbft maybe?

@Likith101

Copy link
Copy Markdown
Member Author

please resolve the merge conflicts. can you just double check the recall numbers once? also it would be better to make this tunable via bleve/cbft maybe?

I am not sure if making it tunable helps much. Since a user needs to be very familiar with how this setting can affect things.

When I initially tested, recall numbers seemed to drop by 0.5 with accuracy being the same. Do you want a rerun of the test once again?

@Likith101

Copy link
Copy Markdown
Member Author
Metric With change Without change
Recall@3 88.1% 87.7%
Accuracy@3 89.3% 89.3%
Index Build Time 1170.6 1581.7

@Likith101
Likith101 merged commit 6b4cace into master Aug 5, 2026
9 checks passed
@Likith101
Likith101 deleted the VecEx1 branch August 5, 2026 06:21
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.

3 participants