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

Performance in KITTI or velodyne-HDL64e ? #11

Closed
lshGame opened this issue Jul 5, 2019 · 5 comments
Closed

Performance in KITTI or velodyne-HDL64e ? #11

lshGame opened this issue Jul 5, 2019 · 5 comments

Comments

@lshGame
Copy link

lshGame commented Jul 5, 2019

Hello, big thanks for sharing this code.
I made a ros bag from KITTI raw data(2011_09_30_drive_0027_extract). This bag includes cam0, 100Hz imu and velodyne-HDL64e. I have tested this bag with VINS-Mono and lio-mapping(64_scans_test.launch), the result is good.
But the result of lio-mapping(test_outdoor.launch & map_4D_indoor.launch) is bad. lio-mapping is able to initialize but can't reach high level of accuracy in velodyne-HDL64e.
Screenshot from 2019-07-05 09-49-48

@hyye
Copy link
Owner

hyye commented Jul 6, 2019

From our experience on the outdoor planar motion tests, two main issues can be the prior extrinsic parameters (if no auto rotation calibration is applied) and initialization.
Sometimes, initialization may suffer from “bad motion”, bad extrinsic parameters or wrong data, yet it still claims to be successful.
For further optimization, extrinsic parameters are important to the initial states. Straight / planar motion is not friendly to the auto extrinsic parameter estimation.
Maybe you can share the tools to build the KITTI data. I can have a try when I have time.

@weisongwen
Copy link

@, hi haoyang, Thanks for sharing your nice work. The prior factor is the prior for extrinsic parameters?

Best,
Welson,

@hyye
Copy link
Owner

hyye commented Jul 8, 2019

@weisongwen, yes it is. This is only needed when translation parts of the extrinsic parameters cannot be well constrained.

@weisongwen
Copy link

Hi @hyye , quite clear, Thanks.

@hyye
Copy link
Owner

hyye commented Jul 23, 2019

@lshGame, I tested the KITTI 00 sequence with the commit 8485a4d. It should work.
You may try to run test_outdoor_64.launch and map_4D.launch to see the results w/o and w/ mapping.
The estimator will have some drift at the beginning; but after several seconds, it converges.

@KitKat7 KitKat7 closed this as completed Aug 5, 2019
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