Skip to content
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: add autoware_ prefix to diagnostic_graph_utils in tier4_system_component.launch.xml #741

Merged

Conversation

mitsudome-r
Copy link
Collaborator

Description

This adds autoware_ prefix to diagnostic_graph_utils package used in tier4_system_component.launch.xml
For the details, please refer to autowarefoundation/autoware.universe#9968

How was this PR tested?

Made sure that autoware_launch launches correctly:
ros2 launch autoware_launch autoware.launch.xml vehicle_model:=sample_vehicle sensor_model:=aip_xx1 map_path:=/home/mitsudome-r/autoware_map/nishishinjuku_autoware_map/

Notes for reviewers

Please merge this with autowarefoundation/autoware.universe#9968.

Effects on system behavior

None.

…component.launch.xml

Signed-off-by: Ryohsuke Mitsudome <[email protected]>
Copy link

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

Copy link

@sasakisasaki sasakisasaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick Comment

Thank you for fixing!

Reason for Approval

This PR is for the update by this prefix PR. Thus, if let's we merge this PR and the prefix PR at the same time, I believe it'll be working 👍

(EDITED): The prefix PR is tested appropreately. That's why I said "let's merge this PR and the prefix PR at the same time" above.

@mitsudome-r mitsudome-r merged commit 36f53a2 into tier4:awf-latest Jan 24, 2025
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants