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

Made it work in ros2 humbe #25

Open
wants to merge 34 commits into
base: main
Choose a base branch
from

Conversation

Wataru-Oshima-Tokyo
Copy link

Hello CHENXXX
Thank you for releasing this repository!
I have makde ig lio work in ros2 humble and added some logics such as below.

  1. added tf for robot frame to imu frame and robot rame to lidar frame.
  2. added a function to aovid getting an orthogonal error caused by a very small error which could happen some dataset.
  3. move to InRadius() function into pointcloud_preprocess to make it more efficient where it does not have to run though all the points again before undistoring and downsampling.
  4. added a new node to subscribe a keyframe and add a service to save the map.
  5. some other changes such as given parameters or frames...

It would be highly apprecisated if you could add a new branch in case people want to use your highly-accurate lio in ros2!

Thank you!
Wataru Oshima

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

Successfully merging this pull request may close these issues.

1 participant