-
Notifications
You must be signed in to change notification settings - Fork 955
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
Adding IMU doesn't seem to work #6
Comments
Hey! Have you tried to check the right name of the Imu topic ? |
Hey, same problem with IMU here. Using just pointcloud data seems to work well. But trying to add an Imu is freaking me out. Output becomes completely crazy. Imu is aligned with my Velodyne and both use ROS axis references (x-front, y-left, z-up). Any ideas of what could be wrong? Thanks in advance! |
Hey!The same problem with IMU. |
Hello everyone, |
Hey @laizwithzed Check imu data consistency (for instance, check that timestamps are correct. Also I had sometimes two imu messages with the same header. Driver node problems...) If imu axis are aligned correctly with the lidar, try to record a Ros bag with lidar+imu and then play it back slower (use "--clock" param as well as "-r 0.5" to make it slower). I realized that my computer wasn't powerful enough to process everything in real-time but it performed correctly when I was using a rosbag with slower speed. Tell me if you found this helpful, |
Hi @rcabg, Thanks for your comments, but unfortunately it didn't work. I had to transform the orientation axis in my code to allign the IMU data and Velodyne points. To do so, I applied a rotation+translation transform in the coordinates from the topic /an_device/Imu/ (in ScanRegistration.cpp) and published the transformed coordinates in the topic /imu/data/ (that I'm using in LaserMapping.cpp). I tried your suggestion to play the bag file at a slower speed, but it didn't work, as the algorithm goes crazy in slow motion. Did you change something in your code? |
Hi @laizwithzed No, just lidar+imu alignment. Also, does your linear acc in axis Z include gravity? It has to. It could be weird, but I've seen some IMUs that don't. |
Hi @rcabg,
In fact, my IMU data didn't contain any acceleration at first and it seems
to be the cause of the problem, as the algorithm worked fine with new bag
files including acceleration data.
Best,
Laiz
…On Mon, Aug 20, 2018 at 1:16 PM Rafael Caballero González < ***@***.***> wrote:
Hi @laizwithzed <https://github.com/laizwithzed>
No, just lidar+imu alignment. Also, does your linear acc in axis Z include
gravity? It has to. It could be weird, but I've seen some IMUs that don't.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/Amhimv9SvCnds0ZKRvfZQn_LGe9AsCxGks5uSpqPgaJpZM4I676W>
.
|
Hi @rcabg , |
@laizwithzed how do you do the IMU? what are the axes convention you use? |
Hi guys! I just wanted to check in to keep the conversation going, since we seem to be having similar issues. My team is working on creating an autonomous rover using a Velodyne Puck VLP-16, Zed camera, and a BNO080 IMU. We're evaluating LOAM for now, to see if we can perhaps fuse the output with a camera based SLAM algorithm later (ORB-SLAM2 is showing promising results). As we're new to ROS and SLAM, please point out any rookie mistakes that we might be making. Unfortunately, we are also struggling with getting LOAM working together with the IMU, even at standstill. I'll make sure to provide a detailed description of what we're seeing the next time we have it all set up. So far, we have tried:
These are the steps we are planning on doing at the beginning of next week:
Please stay in touch, and hopefully we can solve this together! |
@luhcforgh looking forward for this. I am actually working on integratin lidar with IMU too, and facing problem like this! |
@w4rlock999 - We have decided to switch over to LeGO-LOAM (https://github.com/RobustFieldAutonomyLab/LeGO-LOAM/) since it performed and the author of this repo admitted to never testing it with an IMU in another issue thread. According to their research paper, the performance should be even better on the KITTI datasets, and it also features loop closure. So far, we are getting good performance with just the LIDAR, have managed to run a bag with IMU data successfully, and now we're trying to work out the IMU matching to the LIDAR frame there instead. Good luck if you decide to stick to this repo! |
@luhcforgh Have you measured the performance on Kitti Dataset? |
@luhcforgh what dataset did you use that contain imu data to test the lego loam? Can i have it to test it ? |
@BrunoEduardoCSantos I'm unsure whether we actually did that, and we have since moved on to LeGO-LOAM since loop closure is a big perk for our application. @w4rlock999 We tried these datasets, which include IMU data. Don't forget to set the flag indicating that you are running bagged data, as described at the bottom of the LeGO-LOAM readme. https://github.com/RobustFieldAutonomyLab/jackal_dataset_20170608 If you want to continue the discussion regarding LeGO-LOAM and IMU data, we have an active thread here instead: |
@luhcforgh @BrunoEduardoCSantos update for all, i actually near to success in using loam_velodyne with IMU, but not with laboshinl's package, but daobilige-su 's package. the only problem is that the yaw, rotation on vertical axis, is really off, i think it's the problem with the pointcloud movement estimation, but the other rotation an translation is OK. |
@w4rlock999 How is your work going? Plz share your results or problems when integrating loam_velodyne with IMU |
When I try to add an IMU topic it doesn't seem to change the output.
Anyone get this working with an IMU?
Thanks,
Matt
The text was updated successfully, but these errors were encountered: