Skip to content

Implement robot_localization#71

Draft
EnzoRap117 wants to merge 17 commits intodevelopfrom
feature/integrate-IMU-into-rtabmap
Draft

Implement robot_localization#71
EnzoRap117 wants to merge 17 commits intodevelopfrom
feature/integrate-IMU-into-rtabmap

Conversation

@EnzoRap117
Copy link
Copy Markdown

No description provided.

@zbrotherton
Copy link
Copy Markdown

zbrotherton commented Feb 25, 2026

This covers much much more than IMU btw, its more of an EKF PR

Linking relevant issues

@zbrotherton zbrotherton linked an issue Feb 25, 2026 that may be closed by this pull request
@EnzoRap117
Copy link
Copy Markdown
Author

Yeah, I realized that after I named the branch though. Will probably replace this branch with one with a more fitting name when I get the chance.

@EnzoRap117
Copy link
Copy Markdown
Author

Currently, this modified version of sim_rtabmap causes rviz to fail to launch automatically when starting the sim, likely due to some failure related to the LiDAR data based on the errors being thrown when running rviz manually.

Still working through how to best implement robot_localization with rtabmap, a better name for this branch would be "implement robot_localization"

@EnzoRap117
Copy link
Copy Markdown
Author

Based on what I read here, I'll try have it so icp_odometry and stereo_odometry can be fed into robot_localization as separate odometry sources and having that act as the odometry source. http://official-rtab-map-forum.206.s1.nabble.com/Combining-visual-and-icp-odometry-td7623.html

@EnzoRap117
Copy link
Copy Markdown
Author

Right now:

Currently trying to change the the rtabmap nodes of icp_odometry and stereo_odometry to rename their publish topics to something other than 'odom' so they don't conflict with robot_localization

Need to feed the data of icp_odometry and stereo_odometry into robot_localization


Right now: Figure out how to do the above

Currently, 'odom' is being published to by icp_odometry

  • This is bad
    • The only raw sensor data is currently icp_odometry in the topic /rtabmap/odom_sensor_data/raw
  • Instead
    • icp_odometry and stereo_odometry should be used as inputs for robot_localization
    • robot_localizaiton should be the only thing publishing to odom

@EnzoRap117
Copy link
Copy Markdown
Author

The code for launching the icp & stereo nodes is copied from the .launch file of rtabmap itself in commit 10. They still need to re-written, ideally in a way that minimizes repetition to avoid topics being set same values multiple times

 into feature/implement-robot_localization-rtabmap
- Still fails to launch simulation
- Styled it off of rtabmap's launch arguments & robot localization
- probably doesn't work at all
 into feature/implement-robot_localization-rtabmap
 into feature/implement-robot_localization-rtabmap
 into feature/implement-robot_localization-rtabmap
@EnzoRap117 EnzoRap117 changed the title Feature/integrate imu into rtabmap implement robot_localization Apr 3, 2026
@EnzoRap117 EnzoRap117 changed the title implement robot_localization Implement robot_localization Apr 3, 2026
@EnzoRap117 EnzoRap117 self-assigned this Apr 3, 2026
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.

Implement robot_localization for use with RTABMap

2 participants