Skip to content

Add odometry plugin to mobile_robot demos#531

Draft
Amronos wants to merge 15 commits into
ros-controls:rollingfrom
Amronos:add-odometry-plugin
Draft

Add odometry plugin to mobile_robot demos#531
Amronos wants to merge 15 commits into
ros-controls:rollingfrom
Amronos:add-odometry-plugin

Conversation

@Amronos

@Amronos Amronos commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

Depends on #511

Added gz-sim-odometry-publisher-system to the URDF/SDF descriptions, which allows Gazebo to publish the "real" odometry on a topic. This topic can then be used to verify how accurately the controllers are working.

@Amronos Amronos requested a review from ahcorde as a code owner March 27, 2025 08:56
@Amronos

Amronos commented Mar 27, 2025

Copy link
Copy Markdown
Contributor Author

Converting to draft as this should not be merged yet.

@Amronos Amronos marked this pull request as draft March 27, 2025 08:57
@Amronos Amronos force-pushed the add-odometry-plugin branch from fb2d96d to 9a9a1a3 Compare March 27, 2025 09:23
@mergify

mergify Bot commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

⚠️ The sha of the head commit of this PR conflicts with #511. Mergify cannot evaluate rules on this PR. ⚠️

@ahcorde ahcorde left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you mind to fix conflicts?

@Amronos

Amronos commented May 21, 2025

Copy link
Copy Markdown
Contributor Author

do you mind to fix conflicts?

I will rebase this PR on top of #511 once it gets completed.

@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

This PR is stale because it has been open for 45 days with no activity. Please tag a maintainer for help on completing this PR, or close it if you think it has become obsolete.

@github-actions github-actions Bot added the stale label May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants