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

chore(ground_segmentation_launch): change max_z of cropbox filter to vehicle_height #6549

Conversation

badai-nguyen
Copy link
Contributor

@badai-nguyen badai-nguyen commented Mar 6, 2024

Description

Related_link

TIER IV INTERNAL LINK

Tests performed

  • Confirmed by logging_simulator that the correct max_z and min_z were parsed to `obstacle_segmentation/crop_box_filter' node as desired.
    image

Effects on system behavior

Not applicable.

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.

@github-actions github-actions bot added the component:launch Launch files, scripts and initialization tools. (auto-assigned) label Mar 6, 2024
@badai-nguyen badai-nguyen marked this pull request as ready for review March 6, 2024 05:05
@yukkysaito
Copy link
Contributor

@badai-nguyen
image
I ran the tutorial and checked the polygon in the crop box and it looks like the polygon height is zero.

@yukkysaito
Copy link
Contributor

@badai-nguyen here is parameter dump result 🙏
image

Signed-off-by: badai-nguyen <[email protected]>
@badai-nguyen
Copy link
Contributor Author

badai-nguyen commented Mar 6, 2024

@yukkysaito I'm so sorry for my typo mistake 🙇 . I fixed https://github.com/autowarefoundation/autoware.universe/pull/6549/commits
image

@yukkysaito
Copy link
Contributor

image
I change the vehicle height to 10.5, it is properly reflected

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

@badai-nguyen badai-nguyen added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Mar 6, 2024
@badai-nguyen badai-nguyen merged commit 0e1761a into autowarefoundation:main Mar 6, 2024
28 checks passed
HansRobo pushed a commit that referenced this pull request Mar 12, 2024
…vehicle_height (#6549)

* chore(ground_segmentation_launch): change max_z of cropbox filter to vehicle_height

Signed-off-by: badai-nguyen <[email protected]>

* fix: typo

Signed-off-by: badai-nguyen <[email protected]>

---------

Signed-off-by: badai-nguyen <[email protected]>
Signed-off-by: Kotaro Yoshimoto <[email protected]>
kaigohirao pushed a commit to kaigohirao/autoware.universe that referenced this pull request Mar 22, 2024
…vehicle_height (autowarefoundation#6549)

* chore(ground_segmentation_launch): change max_z of cropbox filter to vehicle_height

Signed-off-by: badai-nguyen <[email protected]>

* fix: typo

Signed-off-by: badai-nguyen <[email protected]>

---------

Signed-off-by: badai-nguyen <[email protected]>
Signed-off-by: kaigohirao <[email protected]>
badai-nguyen added a commit to tier4/autoware.universe that referenced this pull request Apr 22, 2024
…vehicle_height (autowarefoundation#6549)

* chore(ground_segmentation_launch): change max_z of cropbox filter to vehicle_height

Signed-off-by: badai-nguyen <[email protected]>

* fix: typo

Signed-off-by: badai-nguyen <[email protected]>

---------

Signed-off-by: badai-nguyen <[email protected]>
badai-nguyen added a commit to tier4/autoware.universe that referenced this pull request Apr 23, 2024
…vehicle_height (autowarefoundation#6549)

* chore(ground_segmentation_launch): change max_z of cropbox filter to vehicle_height

Signed-off-by: badai-nguyen <[email protected]>

* fix: typo

Signed-off-by: badai-nguyen <[email protected]>

---------

Signed-off-by: badai-nguyen <[email protected]>
karishma1911 pushed a commit to Interplai/autoware.universe that referenced this pull request Jun 3, 2024
…vehicle_height (autowarefoundation#6549)

* chore(ground_segmentation_launch): change max_z of cropbox filter to vehicle_height

Signed-off-by: badai-nguyen <[email protected]>

* fix: typo

Signed-off-by: badai-nguyen <[email protected]>

---------

Signed-off-by: badai-nguyen <[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.

2 participants