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.

Sign up for GitHub

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

Jump to bottom

Iris: set base_link as mobile base in odometry publisher #64

Merged
merged 1 commit into from
Sep 17, 2023
Merged

Iris: set base_link as mobile base in odometry publisher #64

merged 1 commit into from
Sep 17, 2023

Conversation

srmainwaring
Copy link
Collaborator

@srmainwaring srmainwaring commented Sep 16, 2023 •
edited
Loading

Update the odom and robot frames for the OdometryPublisher plugin to conform with ROS REP 105.

  • Remove the frame prefix in <odom_frame>.
  • Change <robot_base_frame > to base_link.

Required to fix: ArduPilot/ardupilot_gz#30

@srmainwaring srmainwaring added the bug Something isn't working label Sep 16, 2023
@srmainwaring srmainwaring self-assigned this Sep 16, 2023
@srmainwaring srmainwaring mentioned this pull request Sep 16, 2023
Merged
@srmainwaring srmainwaring merged commit 8df6bd0 into ArduPilot:ros2 Sep 17, 2023
3 checks passed
@srmainwaring srmainwaring deleted the prs/pr-iris-odom-pub branch September 17, 2023 21:36
@srmainwaring srmainwaring mentioned this pull request Sep 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pedro-fuoco pedro-fuoco pedro-fuoco approved these changes

@Ryanf55 Ryanf55 Ryanf55 approved these changes

Assignees

@srmainwaring srmainwaring

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@srmainwaring @Ryanf55 @pedro-fuoco