-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Error while installing PCDet in Windows #681
Comments
PyTorch version 1.1 fell behind, it's too old for spconv. |
Hi! Just tried with the version you suggested @xiazhongyv. Most specifically :
|
What GPU do you have? "undefined in device code" perhaps indicates that it might be an error related to your GPU. |
Hi @MartinHahner ! |
Ok, by doing some more information I found this (https://docs.nvidia.com/cuda/cuda-c-programming-guide/index.html#const-variables) :
Checking inside this "iou3d_nms_kernel.cu" file, I noticed the problematic EPS variable is indeed a const floating variable. By trying to compile on windows, we don't follow the conditions above. I tried to modify the file by replacing the variable references directly by the value. The error does not appear anymore when compiling, but I now have other errors, here is the whole log when trying to execute "python setup.py develop" :
I noticed 2 error messages in this log :
|
You are using a GPU with the Maxwell architecture: You need to have at least GPU with the Pascal architecture: |
@MartinHahner Hmmm I'm surprised by that, I've not seen anywhere in their repo before talking about some hardware constraints, especially knowing that Maxwell architecture is supposed to be compatible with Cuda 10.2 ... |
@MartinHahner Tried to install it in a machine with a Pascal GPU (Quadro P2000) with Windows 10, and the same issue remains. |
|
@FindDefinition Thanks for the heads-up! The code not being C++ standard is not ideal indeed...! I applied your suggestion to modify the code, and I also had to modify some more bit of code in Lines 118 and 169 of iou3d_nms.cpp from : Now I still have errors when trying to compile ... :
Seems like a conflict of the same function being implemented on Cuda and on CPU at the same time... |
@Andjii |
@FindDefinition Ok so I renamed the check_rect_cross function in the .cu file to check_rect_cross_cuda, and replaced all instances of "unsigned long long" to "uint64_t", and "long" to "int32_t" both in iou3d_nms.cpp and iou32_nms_kernel.cu files. Now it compiled without error! Seems like OpenPCDet has some modifications to do to be portable to Windows...! |
This issue is stale because it has been open for 30 days with no activity. |
This issue was closed because it has been inactive for 14 days since being marked as stale. |
this is helpful for compiling PCDet lib in windows, thanks a lot |
@Andjii , hello, first of all, thanks for all your discussion of compiling PCDet in windows, i ' ve encountered nearly the same issues with you when installing PCDet in windows, finally, i got the errors of no "sys/mman.h" file found in the system, i realised that this is only exited in Linux/Mac system, this error was caused by installing the lib of "SharedArray" when it automatically executed the commander of "pip install SharedArray", this library is only supported in Linux or Mac, So, have you encounter this error when you install PCDet Lib in Windows? if so, how did you solve it? |
@Andjii I've had a similar proble and here high-level solutions have been discussed. But digging just a little bit into what the error really is, I realized that
A dirty hack with replacing |
I meet the same problem and solve it, I submit my code, and hope it can help others. see this |
I also have the problem (no file "sys/mman.h" ),did you solve it ? |
The problem still exists on NVIDIA Quadro P3200 (Compute capability version = 6.1, i.e., code=sm_61, Pascal architecture), so this issue is still not resolved. It could keep open. |
Fix open-mmlab#681. 1. const float -> constexpr float, referring to opencv/opencv#13491 and opencv/opencv#13960 2. rename check_rect_cross function to resolve name conflicts 3. change long into int32_t (unsigned long long seems no problem yet)
Hi everyone,
I've been trying to port some neural network code from a colleague that uses OpenPCDet from Linux to Windows 10, and everytime we try to execute the setup.py file, we have the same error :
(Please note that this error even happends when installing PCDet in standalone)
Is OpenPCDet compatible with Windows? If yes, any idea on this issue?
We've tried with :
The text was updated successfully, but these errors were encountered: