search

rpng / open_vins

An open source platform for visual-inertial navigation research.
https://docs.openvins.com
GNU General Public License v3.0
2.13k stars 630 forks source link

Small Clang Fixes #226

Closed smnogar closed 2 years ago

smnogar commented 2 years ago

Small tweaks to fix clang build errors/warnings. Notably the ROS2Visualizer was causing some Eigen assertions to fail, breaking the build for ov_msckf.

goldbattle commented 2 years ago

What version of Eigen are you using? I think https://github.com/rpng/open_vins/issues/121 had this same issue, so this might support newer versions. Mine is pretty ancient:

pkg-config --modversion eigen3
3.3.7
smnogar commented 2 years ago

Brew likes to stay at the cutting edge, so I'm on 3.4.0. I tried my changes on a docker ubuntu 20.04 image and it pulled 3.3.7, so it should work there too.