Skip to content
This repository has been archived by the owner on Mar 27, 2023. It is now read-only.

fix/add arg in order to disable dummy_perception_publisher from command line #144

Merged
merged 1 commit into from
Nov 30, 2021

Conversation

satoshi-ota
Copy link
Contributor

@satoshi-ota satoshi-ota commented Nov 30, 2021

PR Type

  • Improvement

Related Links

Description

When we launch logging_simulator.launch.xml with option percption:=true, no need to launch dummy_perception_publisher, and I want to be able to control enable/disable dummy_perception_publisher from the terminal without having to rewrite lauch file.

Review Procedure

Remarks

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

  • Commits are properly organized and messages are according to the guideline
  • Code follows coding guidelines
  • (Optional) Unit tests have been written for new behavior
  • PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

  • All open points are addressed and tracked via issues or tickets
  • Write release notes

CI Checks

  • Build and test for PR: Required to pass before the merge.
  • Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@satoshi-ota satoshi-ota merged commit 58419a9 into use-autoware-auto-msgs Nov 30, 2021
@satoshi-ota satoshi-ota deleted the fix/add-argument-for-simulation branch November 30, 2021 04:38
@kenji-miyake
Copy link
Contributor

@satoshi-ota Could you explain why you want to change this in the description form, please?

j4tfwm6z pushed a commit that referenced this pull request Feb 1, 2022
kazuki527 pushed a commit to kazuki527/autoware_launch that referenced this pull request Apr 25, 2022
takayuki5168 pushed a commit that referenced this pull request May 26, 2022
takayuki5168 pushed a commit that referenced this pull request May 26, 2022
* fix/rviz perception config (#138)

* auto/fix occupancy grid map topic name (#137)

* fix/rename segmentation namespace (#139)

* fix namespace: rviz config

* fix namespace: planning stack

* rename segmentatino directory

* fix namespace: perception stack

* fix/add arg in order to disable dummy_perception_publisher from command line (#144)

Co-authored-by: Satoshi OTA <44889564+satoshi-ota@users.noreply.github.com>
kosuke55 pushed a commit that referenced this pull request Jan 17, 2023
* feat: update param path

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* fix

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

* organize param files

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
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.

3 participants