-
Notifications
You must be signed in to change notification settings - Fork 20
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
Compilation issue #8
Comments
Hey @donlk , You can change I'm not too sure as to why THNets_CPU wouldn't be included in the compilation process. To ensure that this version builds, I just did a clean Git pull and recompile. It built flawlessly for me. However, I'll add in an explicit install to the openMVG_targets for you. Give me 5 minutes. |
Hey @donlk ,
and test if it works? |
Just tried it. Previous issues are gone, but there is a new one:
|
@donlk this seems like a compiler error with no forced inlining, as referenced here: BVLC/caffe#4046 and here: tensorflow/tensorflow#1346 . I just added a CMake forced inlining function. Can you please retry? Thanks. |
Yes you're right, i forgot i already encountered this issue in the past during OpenCV compilation which required the same solution. |
Thanks @donlk ! I'll check out that issue ASAP. |
@mdaiter @csp256
Did you guys used CUDA 8 as a base for the implementation? I'm getting compilation errors about definitions allegedly introduced in CUDA 8.0.
Here:
nvcc fatal : Value 'sm_60' is not defined for option 'gpu-architecture' CMake Error at cunn_generated_ReLU.cu.o.cmake:203 (message): Error generating /mnt/linuxdata/Development/work/projects/sfmrecon/built/linux-x86_64/openMVG-prefix/src/openMVG-build/openMVG/features/deep/cunnproduction/CMakeFiles/cunn.dir//./cunn_generated_ReLU.cu.o
and here:
nvcc fatal : Unsupported gpu architecture 'compute_60' CMake Error at latch_cuda_generated_latch.cu.o.cmake:203 (message): Error generating /mnt/linuxdata/Development/work/projects/sfmrecon/built/linux-x86_64/openMVG-prefix/src/openMVG-build/openMVG/features/latch/CMakeFiles/latch_cuda.dir//./latch_cuda_generated_latch.cu.o
Also, i compiled Torch myself, but i did not get a THNets_CPU library as an output that is apparently required:
CMake Error: install(EXPORT "openMVG-targets" ...) includes target "deepDescriptorTHNets_static" which requires target "THNets_CPU" that is not in the export set.
And lastly, i see thnets depends on cudnn for which the usage is bound to registration here https://developer.nvidia.com/cudnn
This might complicate the build process for openMVG and violate its openness to some degree.
I'm using my own openMVG branch merged with the latest develop upstream and mdaiter's custom one:
https://github.com/donlk/openMVG/tree/gpu_matcher
The text was updated successfully, but these errors were encountered: