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

Problems running PyTorch bidings #49

Closed
danperazzo opened this issue Feb 22, 2022 · 5 comments
Closed

Problems running PyTorch bidings #49

danperazzo opened this issue Feb 22, 2022 · 5 comments

Comments

@danperazzo
Copy link

Hello everyone, thank you for sharing this great technique! I have been having problems building the Python bidings from source.
I tried running the command with python setup.py install, however, I got the following error message: error_log.txt.

My system setup is as follows:
Windows 10
GTX 1060
CUDA 11.3
Visual Studio 2019
Cmake 3.22.1
PyTorch 1.10.2

Thanks for the attention :)

@Tom94
Copy link
Collaborator

Tom94 commented Feb 22, 2022

These errors look like a mismatch between the installed visual studio and CUDA version -- unfortunately, there are frequent breaking changes between the two.

I recommend updating Visual Studio 2019 through the VS installer, afterwards installing the latest CUDA 11.6, rebooting, and then trying again. Fingers crossed!

@danperazzo
Copy link
Author

Did not work :/

@danperazzo
Copy link
Author

To provide an update, updating the Visual Studio 2019 broke my installation of tiny-cuda-nn :/ I don't know if there is a problem with the incompatibility.

@danperazzo
Copy link
Author

danperazzo commented Feb 23, 2022

I managed to install the Python bindings! Here are the configurations I used:

Windows 10
GTX 1060
CUDA 11.6.1
Visual Studio 2019 (using the CXX compiler MSVC 19.29.30140.0)
Cmake 3.22.1
PyTorch 1.10.2 (installed with conda with cudatoolkit 11.3)

Initially, when I installed the python bindings (running python setup.py install , I got the following error:

fully_fused_mlp.cu
C:\Users\Temp\Desktop\3d-object-stationary-camera-implicit\code_c\tiny-cuda-nn\src\fully_fused_mlp.cu(415): error: name followed by "::" must be a class or namespace name

C:\Users\Temp\Desktop\3d-object-stationary-camera-implicit\code_c\tiny-cuda-nn\src\fully_fused_mlp.cu(493): error: name followed by "::" must be a class or namespace name

C:\Users\Temp\Desktop\3d-object-stationary-camera-implicit\code_c\tiny-cuda-nn\src\fully_fused_mlp.cu(493): error: name followed by "::" must be a class or namespace name

3 errors detected in the compilation of "../../src/fully_fused_mlp.cu".
error: command 'C:\\Program Files\\NVIDIA GPU Computing Toolkit\\CUDA\\v11.6\\bin\\nvcc.exe' failed with exit status 1

However, since my GPU does not have so much memory, I removed the FullyFused from my installation (it would not work anyway). Doing this ensured the instalation weny smoothly and I managed to run the sample python example

@Tom94
Copy link
Collaborator

Tom94 commented Feb 24, 2022

This is a really good catch, thank you! I pushed a fix to setup.py that won't include fully fused MLPs on older architectures where compilation would fail.

(CMake already does this, but I had forgotten to replicate the behavior in Python.)

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

No branches or pull requests

2 participants