Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[FEA] Codepacking for IVF-flat #1632
[FEA] Codepacking for IVF-flat #1632
Changes from 6 commits
cc9cbd3
28484ef
e39ee56
68bf927
78d6380
49a8834
897338e
c1d80f5
2a2ee51
834dd2c
6013429
ab6345a
ed80d1a
cdff9e1
7412272
700ea82
27451c6
9d742ef
d1ef8a1
e187147
4f233a6
4ee99e3
22f4f80
9f4e22c
c95d1e0
da78c66
5cc6dc9
15db0c6
154dc6d
47d6421
0f1d106
e2e1308
3f470c8
41a49b2
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have a reason to add the
_interleaved
suffix? Faiss has a function called(un)pack_1
.Alternativele, IVF-PQ has functions
(un)pack
, that can take one or many vectors at once. We shall aim for consistency with one of these.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright, (un)pack_1 sounds like the better naming convention to avoid confusion. I was also told by @cjnolet that the current IVF-PQ CodePacker needs some refactoring and the names of the IVF-PQ functions can be changed to (un)pack_1 then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IVF-PQ functions can unpack multiple vectors at a time, I would not name them (un)pack_1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The way to think of
ivf_flat_codepacker.hpp
is that the codepacker a public API and is the smallest atomic unit (packing and unpacking one single row). This can now be used in host and device functions alike -- for packing and unpacking multiple records either through a kernel, or through a CPU for loop.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The helper
pack_full_list
is just an example of using the CodePacker for packing a full list.