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

refactor(lane_change): add missing safety check parameter #1300

Conversation

zulfaqar-azmi-t4
Copy link
Contributor

@zulfaqar-azmi-t4 zulfaqar-azmi-t4 commented Jan 16, 2025

Description

Parameterized incoming object yaw threshold.

Related PR.

Change type Parameter Name Type Default Value Description
Added collision_check.th_incoming_object_yaw double 2.3562 Objects with a heading difference from the ego exceeding this value are excluded from the safety check.
Added execution.extended_polygon_policy string rectangle Policy used to determine polygon's shape for safety check.
Added cancel.extended_polygon_policy string rectangle Policy used to determine polygon's shape for safety check.
Added stuck.extended_polygon_policy string rectangle Policy used to determine polygon's shape for safety check.
Added parked.extended_polygon_policy string rectangle Policy used to determine polygon's shape for safety check.

How was this PR tested?

  1. PSIM
  2. Evaluator/TIER IV internal link

Notes for reviewers

None.

Effects on system behavior

None.

Copy link

github-actions bot commented Jan 16, 2025

Thank you for contributing to the Autoware project!

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

Please ensure:

Signed-off-by: Zulfaqar Azmi <[email protected]>
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 changed the title refactor(lane_change): parameterize incoming object angle for filter refactor(lane_change): add missing safety check parameter Jan 16, 2025
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 merged commit ece815a into autowarefoundation:main Jan 16, 2025
13 of 16 checks passed
@zulfaqar-azmi-t4 zulfaqar-azmi-t4 deleted the lc-filter-parameterize-obj-angle branch January 16, 2025 14:12
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.

4 participants