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

General Advice for running LidarSlam and the IMU #96

Open
Tom-Hickling opened this issue Aug 15, 2024 · 3 comments
Open

General Advice for running LidarSlam and the IMU #96

Tom-Hickling opened this issue Aug 15, 2024 · 3 comments
Assignees
Labels
question Further information is requested

Comments

@Tom-Hickling
Copy link

We are using your package for odometry for a drone in an indoor environment. Is there any advice on settings for the package to get the best out of it in this sort of environment? We are using the IMU addition, and we were wondering what the scan_time needs to be changed to, as you mentioned changing it in the read-me when using the IMU. We were also wondering if you need to do the transformation between the IMU location and the lidar.

@rsasaki0109
Copy link
Owner

Is 'scan_time' the same as 'scan_period'? Please set the 'scan_period' to the time interval of the LiDAR scan. For example, if it’s 10Hz, then the 'scan_period' should be 0.1 sec. The IMU values need to be transformed to match the LiDAR origin. This package doesn’t implement that, but you can use functions like the one in the link below:
https://github.com/rsasaki0109/transfrom_velocity/blob/main/src/transfrom_IMU.cpp
If you plan to use an IMU, you might want to consider using 'li_slam_ros2' instead of this package."

@rsasaki0109 rsasaki0109 self-assigned this Aug 21, 2024
@rsasaki0109 rsasaki0109 added the question Further information is requested label Aug 21, 2024
@Tom-Hickling
Copy link
Author

Thanks for the info. We're having issues maintaining an accurate z-position estimate. We think this is due to the enclosed nature of our environment. The mapping seems to work fine. There is just a problem when using the lidarslam for the odometry. Do you have any insight into this?

@rsasaki0109
Copy link
Owner

Integrating the IMU can help reduce drift in the z-direction

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants