feat(trajectory_validator): filter collision objects by shape - #3248
Conversation
|
@satoshi-ota Thank you for contributing to TIER IV E2E branch! Edit the checklist below and check the boxes as you complete the items. Things to check before mergingPlease check the following points before merging:
After mergingPlease check the following points after merging:
|
2514af5 to
cdef173
Compare
6d9848b to
ab88aaa
Compare
yuki-takagi-66
left a comment
There was a problem hiding this comment.
動いてはいることは理解できましたが、パースアルゴリズムが紛らわしすぎるように思います。
何かしら設計変更をお願いしたいです。
| return static_cast<OutT>(label_value.empty() ? params_struct.base : label_value); | ||
| } else if constexpr (std::is_same_v<OutT, std::vector<std::string>>) { | ||
| return static_cast<OutT>( | ||
| label_value.size() == 1U && label_value.front() == kCollisionCheckParamBaseKey |
There was a problem hiding this comment.
”base”はkey stringです。value側にいれるものではありません。
相当confusingです。
| }, | ||
| "target_shapes_override": { | ||
| "type": "array", | ||
| "default": ["base"], |
There was a problem hiding this comment.
”base”はkey stringです。value側にいれるものではありません。
相当confusingです。
yuki-takagi-66
left a comment
There was a problem hiding this comment.
メモ:空文字と空配列とで、「対象shape_typeなし」と「有効値なし」とを区別している
|
Tick the box to add this pull request to the merge queue (same as
|
9cdb387 to
66dc9ae
Compare
66dc9ae to
69b1fe5
Compare
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
|
…autoware_universe#3248) * feat(trajectory_validator): filter collision objects by shape * fix(trajectory_validator): filter target shapes during assessment * chore(trajectory_validator): set collision shape defaults in config * refactor(trajectory_validator): pass shape type to target filter * fix(trajectory_validator): fallback shape filters to base * fix(trajectory_validator): fallback shape filters without base value * fix: pre commit Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com> --------- Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>



Summary
This PR adds shape-type based filtering to the trajectory validator collision check filter.
Objects can now be included or excluded from DRAC/RSS collision assessment per object class based on
autoware_perception_msgs/msg/Shape.type.Changes
bbox,polygon[""]because ROS 2 parameter YAML cannot infer the type of[].bbox=true/falsepolygon=true/false[bbox][bbox, polygon]validator.param.yaml.Shape.typevalues without throwing, and emit a warning instead.[base]fallback and[""]disabled-class configuration.Verification
TEST CASE1
TEST CASE2
TEST CASE3