-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
memory-mapped faiss indices #4101
Comments
Yes that seems very good. |
@alexanderguzhva this is very promising if we can replace the @mdouze what are your thoughts on how we should move forward in the contribution. |
@navneet1v I will publish a PR next week, I'm waiting for some extra things to be verified, such as using data from some 3-rd party preallocated memory area as well |
thank you for the update. Excited to see this feature coming out. |
Summary
A prototype of memory-mapped files that contain Faiss Indices (IndexFlatCodes and HNSW at this moment).
zilliztech/knowhere#996
@mdouze Could you please give your comments on the approach, just in case I've missed some important corner cases? Despite the link is from the Faiss fork, it should be pretty straightforward to understand the changes. Basically, I replace
std::vector<>
with a custom container that supports mmap-ping.The PR for Faiss will be implemented as soon as we agree on the details.
Thanks.
The text was updated successfully, but these errors were encountered: