8000 Add Support for Voxel Map ROS2 Publishing Via Point Map Conversion by ahpinder · Pull Request #82 · MOLAorg/mola · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add Support for Voxel Map ROS2 Publishing Via Point Map Conversion #82

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

Merged
merged 5 commits into from
Apr 22, 2025

Conversation

ahpinder
Copy link
Contributor
@ahpinder ahpinder commented Apr 21, 2025

This code adds the ability to publish point maps of occupied voxels in the ROS2 bridge, allowing for real-time visualization of 2D map generation. Since it only publishes occupied voxels, it is not overly computationally expensive.
It uses an existing MRPT function to get the points, so no custom, potentially slow point-wise code is involved.

An example RVIZ2 visualization of a map in the process of being created is shown here:
image

Closes #80

Added code to timerPubMap to publish the occupied voxels of a mrpt::maps::CVoxelMap as a point cloud to ROS2, allowing for real-time ROS2 visualization of 2D map capture
@ahpinder ahpinder marked this pull request as ready for review April 21, 2025 02:26
@jlblancoc jlblancoc enabled auto-merge April 21, 2025 10:56
@jlblancoc
Copy link
Member

Thanks, it LGTM!

Merging once CI passes (check the minor failing clang-format-14 failure above and add a new commit fixing it, please)

auto-merge was automatically disabled April 21, 2025 16:02

Head branch was pushed to by a user without write access

@ahpinder
Copy link
Contributor Author

OK, formatting should be fixed now I think

@jlblancoc jlblancoc merged commit 15f05ba into MOLAorg:develop Apr 22, 2025
1 check passed
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.

2D occupancy maps not published to ROS
2 participants
0