-
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
ld Error during link faiss_test because of python functions #3944
Comments
but i tried add find_package(PythonLibs REQUIRED)
target_link_libraries(faiss_test PUBLIC ${PYTHON_LIBRARIES}) into tests/CMakeLists.txt for I'm not sure if this is a problem with my system configuration or if faiss is missing the relevant cmake configuration for |
Sad!!! After #3939, this issue still exists. :( |
Hi, thanks for the clear repro steps, do you still see this after this change? #3946 |
thanks for your attention, still got error, and I currently have to fix it by manually introducing |
Thanks @Coien-rr , I can reproduce, I will raise a PR to fix it. Edit: If you need to be unblocked, these steps were what I was following earlier, and I don't see the error from those:
I will see how this differs from |
i rebuild |
We have reverted it on main branch in #3954 as you saw, so I will go ahead and close this for now. If you see the issue again after we re-add it, please let us know. |
Summary: prior issues: 1. Nightlies were breaking due to yaml changes. Those are reverted. 2. AIX build broke (external). Fix is to add a conditional in tests/CMakeLists.txt. 3. Build issue facebookresearch#3944 Differential Revision: D64440629
…e"" (facebookresearch#3981) Summary: prior issues: 1. Nightlies were breaking due to yaml changes. Those are reverted. 2. AIX build broke (external). Fix is to add a conditional in tests/CMakeLists.txt. 3. Build issue facebookresearch#3944 Differential Revision: D64440629
Hi @Coien-rr , we plan to re-add it in #3981. I tried your cmake -> make steps, and could not reproduce. My conda env:
Then I ran your cmake -> make steps. |
…e"" (facebookresearch#3981) Summary: prior issues: 1. Nightlies were breaking due to yaml changes. Those are reverted. 2. AIX build broke (external). Fix is to add a conditional in tests/CMakeLists.txt. 3. Build issue facebookresearch#3944 Differential Revision: D64440629
…e"" (facebookresearch#3981) Summary: prior issues: 1. Nightlies were breaking due to yaml changes. Those are reverted. 2. AIX build broke (external). Fix is to add a conditional in tests/CMakeLists.txt. 3. Build issue facebookresearch#3944 Differential Revision: D64440629
…e"" (#3981) Summary: Pull Request resolved: #3981 prior issues: 1. Nightlies were breaking due to yaml changes. Those are reverted. 2. AIX build broke (external). Fix is to add a conditional in tests/CMakeLists.txt. 3. Build issue #3944. Could not repro now. Reviewed By: mdouze Differential Revision: D64440629 fbshipit-source-id: a86b27c25ada0d07e9d3b4c6e4f00b2e6b637fbe
Summary
when i using
cmake report ld error during
Linking CXX executable faiss_test
Here is the complete cmake output:
Platform
OS: Linux
Faiss version: git commit 3daf48c
Installed from: compiled by myself
Faiss compilation options:
cmake -B build . -DFAISS_ENABLE_GPU=OFF -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DBUILD_TESTING=ON -DFAISS_ENABLE_PYTHON=ON -DFAISS_OPT_LEVEL=avx2 -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=ON
Running on:
Interface:
Reproduction instructions
cmake -B build . -DFAISS_ENABLE_GPU=OFF -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DBUILD_TESTING=ON -DFAISS_ENABLE_PYTHON=ON -DFAISS_OPT_LEVEL=avx2 -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=ON
The text was updated successfully, but these errors were encountered: