-
Notifications
You must be signed in to change notification settings - Fork 691
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
fix(probabilistic_occupancy_grid_map): fix robot_pose to refer base_link_frame #6485
fix(probabilistic_occupancy_grid_map): fix robot_pose to refer base_link_frame #6485
Conversation
Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Hi, |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6485 +/- ##
=======================================
Coverage 14.98% 14.98%
=======================================
Files 1839 1839
Lines 127213 127213
Branches 38132 38132
=======================================
Hits 19069 19069
Misses 86809 86809
Partials 21335 21335
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…ink_frame (autowarefoundation#6485) fix: fix robot_pose to refer base_link_frame Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
…ink_frame (autowarefoundation#6485) fix: fix robot_pose to refer base_link_frame Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
…ink_frame (#6485) fix: fix robot_pose to refer base_link_frame Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com> Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
…ink_frame (autowarefoundation#6485) fix: fix robot_pose to refer base_link_frame Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
Description
As reported in https://github.com/orgs/autowarefoundation/discussions/4202#discussioncomment-8564823, we found probabilistic occupancy grid map is published not on the baes_link but on the raw pointcloud frame.
This PR fixes the issue.
Related links
https://github.com/orgs/autowarefoundation/discussions/4202#discussioncomment-8564823
Tests performed
@tseco15 Could you check this PR in your environment and check if the problem is fixed?
Notes for reviewers
Interface changes
Effects on system behavior
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.