KIT-ISAS / lili-om

LiLi-OM is a tightly-coupled, keyframe-based LiDAR-inertial odometry and mapping system for both solid-state-LiDAR and conventional LiDARs.
GNU General Public License v3.0
543 stars 155 forks source link

loop closure performance #40

Open greymfm opened 3 years ago

greymfm commented 3 years ago

Hello, I have evaluated different SLAM software with the Livox Horizon LiDAR (cartographer, hdl_graph_slam, livox_mapping, etc.) and it seems 'lili-om' is the most robust SLAM for such a LiDAR - congratulations!

I tried your code (using 'run_fr_iosb_internal_imu.launch') with some larger recording (Livox Horizon with internal IMU). It works except the loop closure doesn't work at the end of the recording.

Screenshot from 2021-07-12 21-23-16

Screenshot from 2021-07-12 21-25-21

I have uploaded the bag file here (recorded using 'livox_lidar_msg.launch' from 'livox_ros_driver'): https://drive.google.com/file/d/1NheqSfHnh1E00umY0no6I3AqigL7WDFK/view?usp=sharing

Maybe there is something I can try to tune the loop closures? :-)

PS: I plan to add cm-precise RTK-GPS factors (already available as LiDAR time-synchronized NavSatFix messages in the above recording) to the global graph pose optimization to make 'lili-om' even more robust ;-)

Thanks, Alexander

swjtuyang commented 2 years ago

any update?

PS: I plan to add cm-precise RTK-GPS factors (already available as LiDAR time-synchronized NavSatFix messages in the above recording) to the global graph pose optimization to make 'lili-om' even more robust ;-)

kailaili commented 2 years ago

Hi, you may change the parameters related to loop closure in the yaml file, e.g., raise the lc_search_radius. But yeah, adding additional external factors would be very good to achieve guaranteed robustness as sometimes the drift could be too large for closing the loop. I'm looking forward to seeing your further development. :)

schoeller commented 2 years ago

PS: I plan to add cm-precise RTK-GPS factors (already available as LiDAR time-synchronized NavSatFix messages in the above recording) to the global graph pose optimization to make 'lili-om' even more robust ;-)

@greymfm Is there any news on this?