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

imu's params overwritten #2

Closed
narutojxl opened this issue Sep 4, 2020 · 2 comments
Closed

imu's params overwritten #2

narutojxl opened this issue Sep 4, 2020 · 2 comments

Comments

@narutojxl
Copy link

narutojxl commented Sep 4, 2020

Hi author @icameling,
The most details in paper are realized and encapsulated in modified Kontiki and lidar_surfel_point.h, Maybe i will take some more time to learn this Toolkit to better understand paper. Thanks for your this excellent work! :)
In CalibParamManager ctor, acc, gyro and laser weight used in trajectory_manager.cpp is overwritten by const value.

// fine-tuned parameter
global_opt_gyro_weight = 28.0;
global_opt_acce_weight = 18.5;
global_opt_lidar_weight = 10.0; 

When use my own imu, its price is about dozens of RMB, how to set three params? Thanks a lot!
Best

@icameling
Copy link
Member

Now the output error statistics are the original errors, such as point-to-surfel, the errors between IMU measurements and derivatives of estimated trajectory, they are all unweighted.
If you use different types of IMUs, we suggest to output the weighted error, that is, the normalized error, and check out if the magnitude of the error and the noise amplitude are consistent.

@narutojxl
Copy link
Author

Thanks for your detailed reply :), close it now.

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

2 participants