qqqGpe / DC-Loc

24 stars 4 forks source link

Do I need to install teaser? #1

Closed SubMishMar closed 2 years ago

SubMishMar commented 2 years ago

I am getting this error. Screenshot from 2022-10-31 22-07-56

qqqGpe commented 2 years ago

No, you don't have to install teaser, just delete this line and try again.

SubMishMar commented 2 years ago

Oh ok. I installed it and it compiles. Let me remove this line. Thanks! In addition to this I also needed to add #include"iostream" to utility.h , and I think you need to modify the complete path to eigen includes with something generic in your CMakeLists.txt. Perhaps using ${EIGEN3_INCLUDE_DIR} is a better idea.

Thanks for the paper and the code. I like your work!

SubMishMar commented 2 years ago

Also I had to add /usr/lib/x86_64-linux-gnu/libfmt.a to target_link_libraries() for DCRML - but this is probably something that you have no control over, I am not sure.

qqqGpe commented 2 years ago

Oh ok. I installed it and it compiles. Let me remove this line. Thanks! In addition to this I also needed to add #include"iostream" to utility.h , and I think you need to modify the complete path to eigen includes with something generic in your CMakeLists.txt. Perhaps using ${EIGEN3_INCLUDE_DIR} is a better idea.

Thanks for the paper and the code. I like your work!

Thanks for the suggestion.

qqqGpe commented 2 years ago

Also I had to add /usr/lib/x86_64-linux-gnu/libfmt.a to target_link_libraries() for DCRML - but this is probably something that you have no control over, I am not sure.

I didn't meet the same problem as you like, maybe this problem is caused by the old version of 'ceres'.