autowarefoundation / autoware_launch

Apache License 2.0
29 stars 261 forks source link

fix(obstacle_cruise_planner): improve cruise planner handling to a VRU maneuver overlapping with ego vehicle trajectory #1128

Closed ahmeddesokyebrahim closed 3 weeks ago

ahmeddesokyebrahim commented 1 month ago

Description

Fixes :

Related links

Parent Issue :

Tests performed

Before this PR :

https://github.com/autowarefoundation/autoware.universe/assets/36835765/b913905d-eacb-45dc-87d0-1d6effd8d960

https://github.com/autowarefoundation/autoware.universe/assets/36835765/f5367470-0f39-446e-8432-b2780a1098c5

After this PR :

https://github.com/user-attachments/assets/842ef857-156d-4463-8d0d-0bbdd01b7e7e

https://github.com/user-attachments/assets/47e581e6-676b-4ced-b584-2baa3c8d14f1

Notes for reviewers

Interface changes

ROS Topic Changes

N.A

ROS Parameter Changes

Parameter Name Default Value Update Description
obstacle_velocity_threshold_from_cruise_to_stop 1.0 m/s Lowering the this threshold below obstacle_velocity_threshold_from_stop_to_cruise to prevent chattering between cruise and stop
obstacle_velocity_threshold_from_stop_to_cruise 1.5 m/s Lowering the the velocity threshold hysteresis from cruise to obstacle stop so that obstacle_cruise_planner is not performing high -ve acceleration once the VRU is overlapping with ego trajectory and allowing the ego vehicle to cruise the front VRU not always triggering obstacle stop for it

Effects on system behavior

Enhancing the obstacle_cruise_planner to effectively handle scenarios where vulnerable road users (VRUs) suddenly maneuver and overlap with the ego vehicle's trajectory in dense urban operational design domains (ODDs) will improve deceleration and velocity adjustments, resulting in more human-like driving behavior.

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.

After all checkboxes are checked, anyone who has write access can merge the PR.

ahmeddesokyebrahim commented 3 weeks ago

Closing this PR as the intended issue and scenario/s will be covered by this PR