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 orientation #82

Closed
andre-nguyen opened this issue Aug 22, 2018 · 6 comments
Closed

IMU orientation #82

andre-nguyen opened this issue Aug 22, 2018 · 6 comments

Comments

@andre-nguyen
Copy link

@laboshinl @facontidavide @StefanGlaser

Hi guys, tagging you since you seem to know what's going on.

In the current state of the code, what is the expected orientation of the IMU?

On the original NSH indoor outdoor bag file everything works fine. However, for the gates oscillating motion bag that also contains IMU data I do not get good performance.

As the original author noted:

The input messages from the Velodyne and IMU follow the convention of x- pointing to the front, y- pointing to the left, and z- pointing upward. The IMU messages should be projected to align with the Velodyne frame before sending in.

Does that bag include the projection? Has the projection been included in this code?

Thanks!

@andre-nguyen
Copy link
Author

nvm got it

@anushridixit
Copy link

anushridixit commented Oct 2, 2018

@andre-nguyen Has it been included? What did you find out?

@andre-nguyen
Copy link
Author

The code is just very confusing because its all hard coded to bring things into a camera frame. So just align your axes together and everything will work out. Except the map and pose estimates will have x pointing to upwards Z.

@anushridixit
Copy link

I'm not sure how to do that. Is this post-processing? Or while recording the bag file? Thanks a lot!

@andre-nguyen
Copy link
Author

which ever works for you. You can align your axes by putting your lidar with x forward and your imu with x forward, or you can just rotate it right before feeding it into loam and also rotate the output.

@improve100
Copy link

improve100 commented Jan 8, 2019

@andre-nguyen If I put the imu level, can I use the static tf rotation to get the code running? I tried it and put the i imu accelerate x in the direction of z, the map is still very bad.

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

3 participants