-
Notifications
You must be signed in to change notification settings - Fork 636
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)!: add package name prefix of autoware_ #7892
refactor(radar)!: add package name prefix of autoware_ #7892
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
534db99
to
acd324a
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7892 +/- ##
==========================================
- Coverage 28.76% 28.23% -0.54%
==========================================
Files 1593 1615 +22
Lines 116841 117472 +631
Branches 50011 49819 -192
==========================================
- Hits 33608 33164 -444
- Misses 74169 75260 +1091
+ Partials 9064 9048 -16
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
…object_tracker Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
…ar_object_clustering Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
…ware_radar_fusion_to_detected_object Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
…r_crossing_objects_noise_filter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
…_splitter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
39eebb3
to
5c46e85
Compare
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.
I have confirmed that there is no change in logic and that it is a mechanical process.
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
(Note: without checking acting, when it is integrated we need to check acting)
…ation#7892) * refactor: rename radar_object_tracker Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename package from radar_object_tracker to autoware_radar_object_tracker Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename package from radar_object_clustering to autoware_radar_object_clustering Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename package from radar_fusion_to_detected_object to autoware_radar_fusion_to_detected_object Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename radar_crossing_objects_noise_filter to autoware_radar_crossing_objects_noise_filter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename object_velocity_splitter to autoware_object_velocity_splitter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename object_range_splitter to autoware_object_range_splitter Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: update readme 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
How was this PR tested?
Tested in a local recompute environment and the TIER IV Cloud environment.
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.