Skip to content
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

Set CUDA_ACHITECUTRE to native when undefined for NeoML/Python #684

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zimka
Copy link
Contributor

@zimka zimka commented Jul 13, 2022

Whenever I try to build NeoML/Python with python setup.py install, I get the following error:

...
-- Configuring done
CMake Error in CMakeLists.txt:
  CUDA_ARCHITECTURES is empty for target "PythonWrapper".
...
CMake Generate step failed.  Build files cannot be regenerated correctly.

As far as I understand, this is because I do not specify CMAKE_CUDA_ARCHITECTURES variable.
This PR sets it to native when variable is empty.
Somehow works for my CMake 3.18.

Signed-off-by: Boris Zimka <boris.zimka@abbyy.com>
@FedyuninV
Copy link
Contributor

Wow. That's the first time I see this problem not as a CMake Warning but as an error. I will take a look at that, because I think that should be fixed somewhere in MathEngine-related cmake files.

Give me please info about you OS/CUDA ver/exact cmake ver/build command.

@zimka
Copy link
Contributor Author

zimka commented Jul 19, 2022

Sure, please check the following info:

  1. OS
    OS Name: Microsoft Windows 10 Pro
    OS Version: 10.0.19044 N/A Build 19044

  2. CUDA > nvcc.exe --version
    nvcc: NVIDIA (R) Cuda compiler driver
    Copyright (c) 2005-2021 NVIDIA Corporation
    Built on Thu_Jan_28_19:41:49_Pacific_Standard_Time_2021
    Cuda compilation tools, release 11.2, V11.2.142
    Build cuda_11.2.r11.2/compiler.29558016_0
    (GPU: NVIDIA GeForce RTX 2080 SUPER)

  3. > cmake.exe --version
    cmake version 3.20.3

  4. build (in conda 4.12.0):
    >cd neoml\NeoML\Python
    >python setup.py install
    (.cfg files are not used)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants