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(tier4_perception_launch): add missing arg use_multi_channel_tracker_merger #7705

Conversation

technolojin
Copy link
Contributor

@technolojin technolojin commented Jun 26, 2024

Description

add missing arg use_multi_channel_tracker_merger to fix the following launch error

[ERROR] [launch]: Caught exception in launch (see debug for traceback): launch configuration 'use_multi_channel_tracker_merger' does not exist

Related links

#7459

Parent Issue:

  • Link

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

@technolojin technolojin self-assigned this Jun 26, 2024
@github-actions github-actions bot added the component:launch Launch files, scripts and initialization tools. (auto-assigned) label Jun 26, 2024
Copy link

github-actions bot commented Jun 26, 2024

Thank you for contributing to the Autoware project!

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

Please ensure:

@technolojin technolojin marked this pull request as ready for review June 26, 2024 07:28
@technolojin technolojin added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Jun 26, 2024
Copy link
Contributor

@yukkysaito yukkysaito 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

@YoshiRi YoshiRi left a comment

Choose a reason for hiding this comment

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

LGTM

This commit adds the `use_multi_channel_tracker_merger` argument to the simulator launch file. The argument is set to `false` by default. This change enables the use of the multi-channel tracker merger in the simulator.

Signed-off-by: Taekjin LEE <[email protected]>
@technolojin technolojin requested a review from YoshiRi June 26, 2024 07:42
@technolojin technolojin enabled auto-merge (squash) June 26, 2024 07:43
@technolojin technolojin disabled auto-merge June 26, 2024 07:57
@technolojin technolojin enabled auto-merge (squash) June 26, 2024 07:57
@technolojin technolojin merged commit 23d857d into autowarefoundation:main Jun 26, 2024
25 checks passed
@technolojin technolojin deleted the fix/perception-tracker-launch-missing-arg branch June 26, 2024 08:47
simon-eisenmann-driveblocks pushed a commit to simon-eisenmann-driveblocks/autoware.universe that referenced this pull request Jun 26, 2024
…ker_merger (autowarefoundation#7705)

* feat(tier4_perception_launch): add missing arg use_multi_channel_tracker_merger

Signed-off-by: Taekjin LEE <[email protected]>

* feat: add use_multi_channel_tracker_merger argument to simulator launch

This commit adds the `use_multi_channel_tracker_merger` argument to the simulator launch file. The argument is set to `false` by default. This change enables the use of the multi-channel tracker merger in the simulator.

Signed-off-by: Taekjin LEE <[email protected]>

---------

Signed-off-by: Taekjin LEE <[email protected]>
Signed-off-by: Simon Eisenmann <[email protected]>
mitukou1109 pushed a commit to mitukou1109/autoware.universe that referenced this pull request Jul 2, 2024
…ker_merger (autowarefoundation#7705)

* feat(tier4_perception_launch): add missing arg use_multi_channel_tracker_merger

Signed-off-by: Taekjin LEE <[email protected]>

* feat: add use_multi_channel_tracker_merger argument to simulator launch

This commit adds the `use_multi_channel_tracker_merger` argument to the simulator launch file. The argument is set to `false` by default. This change enables the use of the multi-channel tracker merger in the simulator.

Signed-off-by: Taekjin LEE <[email protected]>

---------

Signed-off-by: Taekjin LEE <[email protected]>
palas21 pushed a commit to palas21/autoware.universe that referenced this pull request Jul 12, 2024
…ker_merger (autowarefoundation#7705)

* feat(tier4_perception_launch): add missing arg use_multi_channel_tracker_merger

Signed-off-by: Taekjin LEE <[email protected]>

* feat: add use_multi_channel_tracker_merger argument to simulator launch

This commit adds the `use_multi_channel_tracker_merger` argument to the simulator launch file. The argument is set to `false` by default. This change enables the use of the multi-channel tracker merger in the simulator.

Signed-off-by: Taekjin LEE <[email protected]>

---------

Signed-off-by: Taekjin LEE <[email protected]>
Signed-off-by: palas21 <[email protected]>
tby-udel pushed a commit to tby-udel/autoware.universe that referenced this pull request Jul 14, 2024
…ker_merger (autowarefoundation#7705)

* feat(tier4_perception_launch): add missing arg use_multi_channel_tracker_merger

Signed-off-by: Taekjin LEE <[email protected]>

* feat: add use_multi_channel_tracker_merger argument to simulator launch

This commit adds the `use_multi_channel_tracker_merger` argument to the simulator launch file. The argument is set to `false` by default. This change enables the use of the multi-channel tracker merger in the simulator.

Signed-off-by: Taekjin LEE <[email protected]>

---------

Signed-off-by: Taekjin LEE <[email protected]>
KhalilSelyan pushed a commit that referenced this pull request Jul 22, 2024
…ker_merger (#7705)

* feat(tier4_perception_launch): add missing arg use_multi_channel_tracker_merger

Signed-off-by: Taekjin LEE <[email protected]>

* feat: add use_multi_channel_tracker_merger argument to simulator launch

This commit adds the `use_multi_channel_tracker_merger` argument to the simulator launch file. The argument is set to `false` by default. This change enables the use of the multi-channel tracker merger in the simulator.

Signed-off-by: Taekjin LEE <[email protected]>

---------

Signed-off-by: Taekjin LEE <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:launch Launch files, scripts and initialization tools. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants