refactor(avoidance_by_lane_change): update execution condition

[zulfaqar-azmi-t4]

Description

🤖[deprecated] Generated by Copilot at 2147884

This pull request improves the lane change module for path avoidance by simplifying the logic, adding more flexibility, and enhancing the logging. It refactors the specialRequiredCheck function in scene.cpp, extends the lane_change::utils functions in utils.cpp, adds a new processOnEntry function in interface.hpp and interface.cpp, and modifies the logger name in base_class.hpp.

Related links

None

Tests performed

PSIM and evaluator

Notes for reviewers

None

Interface changes

Avoidance by lane change has higher priorities, so if previously, in the case where obstacle + lane change module is activated, now avoidance by lane change will be activated instead.

Effects on system behavior

Avoidance by lane change has higher priorities, so if previously, in the case where obstacle + lane change module is activated, now avoidance by lane change will be activated instead.

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.
• The PR has been properly tested.
• The PR has been reviewed by the code owners.

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.
• The PR is ready for merge.

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

[satoshi-ota]

LGTM 👍 I confirmed that normal LC was kicked and AbLC was launched instead.

simplescreenrecorder-2023-12-19_13.40.13.mp4

[codecov]

Codecov Report

Attention: 261 lines in your changes are missing coverage. Please review.

Comparison is base (320bc2f) 15.29% compared to head (9b9f497) 15.29%.
Report is 6 commits behind head on main.

Files	Patch %	Lines
...ehavior_path_avoidance_module/parameter_helper.hpp	6.43%	7 Missing and 211 partials ⚠️
...path_avoidance_by_lane_change_module/src/scene.cpp	3.70%	25 Missing and 1 partial ⚠️
...th_avoidance_by_lane_change_module/src/manager.cpp	11.11%	0 Missing and 8 partials ⚠️
...ning/behavior_path_planner/src/planner_manager.cpp	16.66%	5 Missing ⚠️
...havior_path_lane_change_module/src/utils/utils.cpp	60.00%	2 Missing ⚠️
...h_avoidance_by_lane_change_module/data_structs.hpp	0.00%	0 Missing and 1 partial ⚠️
..._avoidance_by_lane_change_module/src/interface.cpp	50.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5869   +/-   ##
=======================================
  Coverage   15.29%   15.29%           
=======================================
  Files        1747     1748    +1     
  Lines      120208   120239   +31     
  Branches    36631    36648   +17     
=======================================
+ Hits        18383    18393   +10     
- Misses      81207    81221   +14     
- Partials    20618    20625    +7     

Flag	Coverage Δ		*Carryforward flag
differential	9.46% <8.09%> (?)
total	15.31% <ø> (+0.01%)	⬆️	Carriedforward from 320bc2f

*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.