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

Despite WITH_CERES option, ceres is actually required to compile #45

Open
severin-lemaignan opened this issue Jan 20, 2016 · 3 comments

Comments

@severin-lemaignan
Copy link
Contributor

While the CMakeLists exposes a WITH_CERES option, ceres is indeed required to compile v4r, due to modules/reconstruction/include/v4r/reconstruction/impl/ReprojectionError.hpp including ceres/rotation.h.

@ToMadoRe
Copy link
Contributor

Yes, you are right. At the moment, we do not conditionally compile files / modules depending on whether these 3rd party libraries are installed. This would require to rewrite some cmake files of V4R but is planned to be implemented in the future.

@CWallbridge
Copy link

I seem to have encountered a similar issue. Also the libceres-dev package that can be installed with Ubuntu does not appear to provide the CeresConfig.cmake or ceres-config.cmake file required for being built WITH_CERES enabled. Is there somewhere I can get these files?

@taketwo
Copy link
Contributor

taketwo commented Nov 21, 2016

I would recommend to compile Ceres from source and tell CMake where you installed it with -DCeres_DIR=<install_dir>/share/Ceres.

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

4 participants