faiss icon indicating copy to clipboard operation
faiss copied to clipboard

Allow k and M suffixes in IVF indexes

Open mdouze opened this issue 1 year ago • 1 comments

Summary: Allows factory strings like IVF3k,Flat as a shorthand for 3072 centroids.

The main question is whether k or M should be metric (k=1000) or power of 2 (k=1024):

  • pro-metric: standard,

  • pro-power of 2: in practice we use powers of 2 most often

The suffixes ki and Mi should be used for powers of 2 but this makes the notation more heavy (which is what we wanted to avoid in the first place).

So I picked power of 2.

Differential Revision: D62019941

mdouze avatar Aug 30 '24 09:08 mdouze

This pull request was exported from Phabricator. Differential Revision: D62019941

facebook-github-bot avatar Aug 30 '24 09:08 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D62019941

facebook-github-bot avatar Sep 10 '24 06:09 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D62019941

facebook-github-bot avatar Sep 10 '24 06:09 facebook-github-bot

This pull request has been merged in facebookresearch/faiss@d85fda7fca18128cb8cca47205663a47a7e4649e.

facebook-github-bot avatar Sep 10 '24 10:09 facebook-github-bot