-
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
refactor(radar_tracks_msgs_converter, simple_object_merger, radar_tracks_noise_filter)!: add package name prefix of autoware_ #8173
Conversation
…r_tracks_msgs_converter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
…ct_merger Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
…e_radar_tracks_noise_filter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
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.
LGTM
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8173 +/- ##
==========================================
- Coverage 29.11% 29.07% -0.05%
==========================================
Files 1607 1612 +5
Lines 118177 118482 +305
Branches 50851 50905 +54
==========================================
+ Hits 34410 34446 +36
- Misses 74528 74789 +261
- Partials 9239 9247 +8
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…cks_noise_filter)!: add package name prefix of autoware_ (autowarefoundation#8173) * refactor: rename radar_tracks_msgs_converter package to autoware_radar_tracks_msgs_converter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename simple_object_merger package to autoware_simple_object_merger Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename sensing/radar_tracks_noise_filter to sensing/autoware_radar_tracks_noise_filter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> --------- Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
…cks_noise_filter)!: add package name prefix of autoware_ (autowarefoundation#8173) * refactor: rename radar_tracks_msgs_converter package to autoware_radar_tracks_msgs_converter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename simple_object_merger package to autoware_simple_object_merger Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename sensing/radar_tracks_noise_filter to sensing/autoware_radar_tracks_noise_filter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> --------- Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
Description
This PR adds the autoware_ prefix to the package name.
Changes are as following.
CMakeLists.txt
,package.xml
README.md
$(find-pkg-share ...)
.github/CODEOWNERS
This PR do not contains any logic change.
Related links
Part of: autowarefoundation/autoware#4569
Launcher update tier4/aip_launcher#267
How was this PR tested?
(radar_tracks_msgs_converter, radar_tracks_noise_filter)Tested in a local recompute environment and the TIER IV Cloud environment.
(simple_object_merger) A specific deploy that uses this package was used for local test.
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.