refactor(avoidance): parameterize magic number

[satoshi-ota]

Description

🤖 Generated by Copilot at f2348f0

This pull request refactors the avoidance module of the behavior path planner to use a new parameter lateral_avoid_check_threshold for checking the lateral shift of the ego vehicle during avoidance maneuvers. It also removes the unused variable avoiding_now and replaces it with the isShifted method. The change affects the data structures, the logic, the configuration, and the debugging of the avoidance module.

Please approve ⬇️ before this PR.
autowarefoundation/autoware_launch#426

Tests performed

• PASS TIER IV INTERNAL SCENARIOS

Effects on system behavior

Nothing. Just refactoring.

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.

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -0.07 ⚠️

Comparison is base (9dad706) 15.06% compared to head (164f934) 15.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4116      +/-   ##
==========================================
- Coverage   15.06%   15.00%   -0.07%     
==========================================
  Files        1508     1512       +4     
  Lines      103774   104408     +634     
  Branches    31705    32059     +354     
==========================================
+ Hits        15630    15662      +32     
- Misses      71124    71675     +551     
- Partials    17020    17071      +51     

Flag	Coverage Δ		*Carryforward flag
differential	13.51% <0.00%> (?)
total	15.06% <ø> (+<0.01%)	⬆️	Carriedforward from 5183938

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

Impacted Files	Coverage Δ
..._planner/utils/avoidance/avoidance_module_data.hpp	0.00% <ø> (ø)
...e/behavior_path_planner/utils/avoidance/helper.hpp	24.27% <0.00%> (-0.24%)	⬇️
..._path_planner/src/marker_utils/avoidance/debug.cpp	4.59% <0.00%> (+0.01%)	⬆️
...er/src/scene_module/avoidance/avoidance_module.cpp	11.94% <0.00%> (-0.06%)	⬇️
...ath_planner/src/scene_module/avoidance/manager.cpp	6.81% <0.00%> (-0.13%)	⬇️
...behavior_velocity_out_of_lane_module/src/debug.cpp	55.10% <ø> (+1.10%)	⬆️

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[kyoichi-sugahara]

LGTM