fix(shape_estimation): preserve irregular large size vehicle detected by camera_lidar_fusion as unknown

[badai-nguyen]

Description

• Current shape_estimation limits the size of known objects (vehicles) below some threshold and it will ignore some supper large trailer or crane.
• Adjust the size threshold to cover such irregular size vehicles might cause the degredation of estimation performance overall.
• This PR to reverse such irregular size vehicle as Unknown object.

Related Link

TIER IV INTERNAL LINK

Tests performed

Confirmed by logging_simulator with internal rosbag that the camera_lidar_fusion result of left-side crane detection (/perception/object_recognition/detection/clustering/camera_lidar_fusion/objects) has been improved.

• Before change (Green): crane was lost some times
• After change (Purple): more stable
• Screencast from 2024年03月12日 19時51分17秒.webm

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.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

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.

[yukkysaito]

LGTM

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 14.92%. Comparing base (fcebec6) to head (1d2827f).
Report is 5 commits behind head on main.

Files	Patch %	Lines
perception/shape_estimation/src/node.cpp	0.00%	5 Missing ⚠️
...erception/shape_estimation/lib/shape_estimator.cpp	0.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6598      +/-   ##
==========================================
+ Coverage   14.79%   14.92%   +0.13%     
==========================================
  Files        1920     1915       -5     
  Lines      132349   131181    -1168     
  Branches    39339    39305      -34     
==========================================
  Hits        19578    19578              
+ Misses      90935    89770    -1165     
+ Partials    21836    21833       -3     

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <0.00%> (?)
total	14.92% <0.00%> (+0.13%)	⬆️	Carriedforward from e77d669

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[YoshiRi]

Please wait for this to be merged.

[technolojin]

I confirmed the issue is resolved, via the issue data replay and recompute.
Thank you.

[YoshiRi]

Sorry, could you change variable name?

[YoshiRi]

LGTM. Please change autoware_launch before merge this.

[badai-nguyen]

LGTM. Please change autoware_launch before merge this.

@YoshiRi Since no shape_estimation releated parameters registered in autoware_launch so It's unneccessary.
BTW, I also tested on CI and confirmed there is no degradation. https://evaluation.tier4.jp/evaluation/reports/0df16ed5-ce10-5bd7-acb5-b8ee07b6e378?project_id=prd_jt
So I will merge this PR.