-
Notifications
You must be signed in to change notification settings - Fork 33
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
segfault when doing index.search using faiss-cpu #15
Comments
Not sure, likely the ABI compatibility issue in the runtime. Do you have any stack trace? |
Not sure if that really helps |
Ah interesting... |
Do you think a similar fix as #1 (comment) could be possible ? |
Ok no I identified the actual issue. These commands reproduce my issue :
With test.py content being the thing in the first post I'm using python 3.7.5 if that can make a difference Can you reproduce ? any idea why that would happen ? |
There could be many possibilities, but I could not reproduce this on docker ubuntu 20.04 image. It could be bad interaction with multiple package managers (pip and conda), it could be lack of cpu feature (avx2), etc. |
Also, if you |
Indeed looks like my CPU does not support avx2 What I'm quite confused about is if I do not update pip (so keeping pip 18.1 with python 3.7) then everything works. |
Ah I think I found something. Are there some differences in this second file that might explain my result ? If that's the case, would it be possible to provide a non avx2 build or do you advise I build it myself ? (and do you know a way to identify if the issue is missing avx2 ?) |
I tried using python 3.8 instead of 3.7 as per #8 (comment) but this makes no difference in my case |
Tried to build myself but getting some other errors when trying to follow https://github.com/kyamagu/faiss-wheels#linux , I opened a separate issue for this (#17) |
manylinux1 and manylinux2010 have different glibc version dependency, and they are usually forward compatible: manylinux1 works on centos 5 environment and above, manylinux2010 works on centos 6 and above. ubuntu 20.04 should be fine with both. Hmm, it makes sense to support option to disable / enable avx2. |
If you build from source, you can also try the official approach:
|
So as discussed in the github action PR, the new version will not use avx2 instructions. That's great. I understand it's not possible to change an existing release and you follow faiss versioning which makes sense. I will use it until faiss release a new version, at this point I'll be able to go back to faiss-cpu Again thanks for the support and for building this simple and useful repo, I think at some point you could consider simply doing a PR to faiss main repo, they might just accept it and maintain it in the future. |
@rom1504 Great. As a side note, faiss maintainers seem not to be interested in supporting PyPI: facebookresearch/faiss#170 (comment) |
Running the faiss example
I'm getting a segfault when doing the search.
OS: ubuntu 20.04
version : 1.6.3 of faiss-cpu
any idea what could be wrong ?
The text was updated successfully, but these errors were encountered: