cython wrapper build broke !
Issue #426
closed
Merging PR #379 might have broken the cython wrapper. @jlblancoc Did anything change in how Eigen includes are done, that might not have made it into the cython sub-directory?
In file included from /Users/dellaert/git/gtsam/build/cython/gtsam_eigency/core.cpp:633:
/Users/dellaert/git/gtsam/build/cython/gtsam_eigency/eigency_cpp.h:1:10: fatal error: 'Eigen/Dense' file not found
#include <Eigen/Dense>
^~~~~~~~~~~~~
Comments (3)
-
reporter -
Oh, sorry! I didn’t have cython/python enabled on my builds.
It’s already fixed, will PR.
-
reporter - changed status to closed
Merged in jlblancoc/gtsam (pull request #383) close issue
#426fix missing Eigen in Cython wrapper→ <<cset 034f69c7aa6d>>
- Log in to comment
@thduynguyen can you replicate, suggest a fix?