Skip to content
This repository was archived by the owner on Mar 28, 2025. It is now read-only.

Conversation

@asana17
Copy link
Contributor

@asana17 asana17 commented Feb 14, 2023

Signed-off-by: asana17 akihiro.sakurai@tier4.jp

Description

The node dummy_diag_publisher is modified in autowarefoundation/autoware_universe#2840. I modified awf:autoware.universe/launch/tier4_system_launch/system.launch.xml because this node needs some parameter YAML files to launch. For this reason, we also need to add some param files to awf:autoware_launch and awf:sample_sensor_kit_launch, corresponding to the change above. Temporarily, I placed an empty parameter YAML file.

For more information, please look at the links below.
autowarefoundation/autoware_universe#2840
autowarefoundation/autoware_universe#2840 (review)
autowarefoundation/autoware_launch#206

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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: asana17 <akihiro.sakurai@tier4.jp>
@asana17 asana17 requested a review from kminoda February 14, 2023 07:01
@asana17 asana17 changed the title add param file for dummy_diag_publisher feat(sample_sensor_kit_launch): add param file for dummy_diag_publisher Feb 14, 2023
Signed-off-by: asana17 <akihiro.sakurai@tier4.jp>
Copy link
Contributor

@kminoda kminoda left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@kminoda kminoda left a comment

Choose a reason for hiding this comment

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

LGTM

@asana17 asana17 merged commit 06330e6 into autowarefoundation:main Feb 16, 2023
@asana17 asana17 deleted the feature/add-dummy-diag-param branch February 16, 2023 08:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants