-
Notifications
You must be signed in to change notification settings - Fork 673
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
refactor(ndt_scan_matcher): fixed ndt_scan_matcher.launch.xml #6041
refactor(ndt_scan_matcher): fixed ndt_scan_matcher.launch.xml #6041
Conversation
Signed-off-by: Shintaro Sakoda <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for refactoring 🙆
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6041 +/- ##
=======================================
Coverage 15.22% 15.23%
=======================================
Files 1767 1767
Lines 121695 121694 -1
Branches 37020 37020
=======================================
Hits 18534 18534
+ Misses 82349 82348 -1
Partials 20812 20812
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
a0da8c4
into
autowarefoundation:main
…refoundation#6041) Fixed ndt_scan_matcher.launch.xml Signed-off-by: Shintaro Sakoda <[email protected]>
…refoundation#6041) Fixed ndt_scan_matcher.launch.xml Signed-off-by: Shintaro Sakoda <[email protected]>
…refoundation#6041) Fixed ndt_scan_matcher.launch.xml Signed-off-by: Shintaro Sakoda <[email protected]>
…refoundation#6041) Fixed ndt_scan_matcher.launch.xml Signed-off-by: Shintaro Sakoda <[email protected]>
…refoundation#6041) Fixed ndt_scan_matcher.launch.xml Signed-off-by: Shintaro Sakoda <[email protected]>
Description
The topic name settings in
ndt_scan_matcher.launch.xml
have been changed.Detail
It is recommended to use abstract topic names for topics described in launch within a package. ekf_localizer.launch.xml is a good example. Following this, I made the same changes to
ndt_scan_matcher
.The specific topic name is deleted in the launch file in the
ndt_scan_matcher
package. Instead, set the name used by autoware inlaunch/tier4_localization_launch/launch/pose_twist_estimator/ndt_scan_matcher.launch.xml
.As a minor change, I also aligned the order in which topic names are written in these files.
Also,
input_map_points_topic
has been removed as it is no longer used. Originally it should have been removed at #5873.Tests performed
It has been confirmed that the
logging_simulator
runs with the same accuracy as before on AWSIM data with GT.Effects on system behavior
There are no 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.