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

fix(autoware_pointcloud_preprocessor): empty input validation (#10115) #1829

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

shmpwk
Copy link
Collaborator

@shmpwk shmpwk commented Feb 17, 2025

Description

Fix crash in the PickupBasedVoxelGridDownsampleFilterComponent when processing empty point cloud data.

cherry-pick autowarefoundation#10115

Related links

Parent Issue:

  • Link

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

…refoundation#10115)

* fix(autoware_pointcloud_preprocessor): fix 0 division

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* style(pre-commit): autofix

* fix float and error throttle

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* style(pre-commit): autofix

* fix

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* fix param validation

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* fix unused var

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* feat add input validatoin

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* fix too cautious floating

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* fix error msg

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* fix
plural

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* fix: set exclusiveMinimum 0.0

Signed-off-by: Shumpei Wakabayashi <[email protected]>

* fix: reomve unnecessary validatoin

Signed-off-by: Shumpei Wakabayashi <[email protected]>

---------

Signed-off-by: Shumpei Wakabayashi <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@shmpwk shmpwk enabled auto-merge (squash) February 17, 2025 08:50
@shmpwk shmpwk merged commit a110d57 into beta/v0.41 Feb 18, 2025
31 of 33 checks passed
@shmpwk shmpwk deleted the fix/downsample_0_input branch February 18, 2025 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants