-
Notifications
You must be signed in to change notification settings - Fork 683
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
feat: apply autoware_
prefix for dummy_perception_publisher
#9987
feat: apply autoware_
prefix for dummy_perception_publisher
#9987
Conversation
Note: * In this commit, I did not organize a folder structure. The folder structure will be organized in the next some commits. * The changes will follow the Autoware's guideline as below: - https://autowarefoundation.github.io/autoware-documentation/main/contributing/coding-guidelines/ros-nodes/directory-structure/#package-folder Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
…are` * Fixes due to this changes for .hpp/.cpp files will be applied in the next commit Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
* To follow the previous commit Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
…blisher` Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
…my-perception-publisher
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.
+ rosdep install -y --from-paths src --ignore-src --rosdistro humble
ERROR: the following packages/stacks could not have their rosdep keys resolved
to system dependencies:
tier4_simulator_launch: Cannot locate rosdep definition for [dummy_perception_publisher]
Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
@SakodaShintaro I have fixed the error with the launch files |
Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9987 +/- ##
==========================================
- Coverage 29.79% 29.43% -0.36%
==========================================
Files 1434 1431 -3
Lines 108244 108139 -105
Branches 42930 42848 -82
==========================================
- Hits 32247 31829 -418
- Misses 72868 73238 +370
+ Partials 3129 3072 -57
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
[ERROR] [launch]: Caught exception in launch (see debug for traceback): "package 'autowaredummy_perception_publisher' not found,
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp> Co-authored-by: SakodaShintaro <rgbygscrsedppbwg@gmail.com>
…-for-simulator-dummy-perception-publisher Signed-off-by: Shintaro Sakoda <shintaro.sakoda@tier4.jp>
…-for-simulator-dummy-perception-publisher Signed-off-by: Shintaro Sakoda <shintaro.sakoda@tier4.jp>
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.
The cause is unknown, but the following error occurred when running planning_simulator.
1737621638.1476393 [ERROR] [autoware_dummy_perception_publisher_node-48]: process has died [pid 1898099, exit code 127, cmd '/home/shintarosakoda/autoware/install/autoware_dummy_perception_publisher/lib/autoware_dummy_perception_publisher/autoware_dummy_perception_publisher_node --ros-args -r __node:=autoware_dummy_perception_publisher -r __ns:=/simulation -p use_sim_time:=False -p wheel_radius:=0.383 -p wheel_width:=0.235 -p wheel_base:=2.79 -p wheel_tread:=1.64 -p front_overhang:=1.0 -p rear_overhang:=1.1 -p left_overhang:=0.128 -p right_overhang:=0.128 -p vehicle_height:=2.5 -p max_steer_angle:=0.7 --params-file /tmp/launch_params_v2k5w2vv --params-file /tmp/launch_params_d62dolct --params-file /tmp/launch_params_q6r_drmg --params-file /tmp/launch_params_eorxccef --params-file /tmp/launch_params_jq3q2sfe --params-file /tmp/launch_params_ufo07lgu --params-file /tmp/launch_params_b6k9morh -r output/dynamic_object:=/perception/object_recognition/detection/labeled_clusters -r output/objects_pose:=debug/object_pose -r output/points_raw:=/perception/obstacle_segmentation/pointcloud -r input/object:=dummy_perception_publisher/object_info -r input/reset:=input/reset -r output/debug/ground_truth_objects:=debug/ground_truth_objects'].
In addition, there may be other code that needs to be fixed, even though it may not be related to the bug.
…my-perception-publisher Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
…on-publisher' of github.com:sasakisasaki/autoware.universe into feat-apply-autoware-prefix-for-simulator-dummy-perception-publisher Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Thank you for your report, now I'm investigating. |
* The node name does not need `autoware_` prefix Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Did you try building with the clean environment? If you don't want to rebuild the full stack, it might just work if you delete both dummy_perception_publisher and autoware_dummy_perception_publisher packages: |
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.
A clean build solved the problem
I forgot to attach the evaluator result (TIER IV Internal link): |
Were you guys able to run planning simulator demo after these changes? For me this node fails at start. I've tried removing this package from the build/install folders and rebuilding but it persisted. Now I'm doing a complete recompilation of entire autoware folder to make sure, will report the test results tomorrow. |
I just did a full clean build and was able to run planning_simulator successfully. |
@SakodaShintaro thanks, complete rebuild solved it for me as well ✅ |
Description
This pull request addresses the issue of inconsistent naming by adding the
autoware_
prefix to relevant components. This change ensures uniformity across the codebase and aligns with the project's naming conventions.Related Links
How was this PR tested?
Currently, we only verified that the build/launch succeeds. Further checks will be performed by using the updated module. This PR will be ready for review after finishing all the necessary checks/tests.
Directory Name
autoware_
as a prefix to the directory name.packages.xml
autoware_
to thename
element.CMakeLists.txt
project
name toautoware_***
.autoware::
toPLUGIN
.Header Files (.hpp)
#ifndef
guards withAUTOWARE__
.autoware::
tonamespace
.Source Files (.cpp)
autoware::
insideRCLCPP_COMPONENTS_REGISTER_NODE
.Launch Files
autoware_
beforefind-pkg-share
.node pkg="<pkgname>"
toautoware_<pkgname>
.Include
autoware
folder as necessary and update include statements accordingly.Verification Points
find-pkg-share
in autowaresensor_launch
as seen ingnss_poser
Launched Commands
Notes for Reviewers
None.
Interface Changes
None.
Effects on System Behavior
All the developers who were using
dummy_perception_publisher
needs to modify their code. This is maybe kind of non quick-win task.