fix(bpp): overwrite turn signal by latter module

[satoshi-ota]

Description

Sometimes, behavior path planner module outputs unnecessary turn signal while multiple modules are running simlutaneously.

Related ticket: https://tier4.atlassian.net/browse/RT1-5329

simplescreenrecorder-2024-03-07_13.16.32.mp4

Currently, turn signal decider manages multiple turn singal infomation (e.g. lane change, avoidance, turn right/left...) based on this logic. But I think it's better to overwrite turn signal simply by latter module one in terms of the signal management among multiple modules because, in most case, the latter module's path modufication is dominant.

In this PR, I added a new function to overwrite turn signal when the ego's in the section that calculated by latter module.

• former module:
  ------|--- turn signal section A ---------|--------------------------------------------------------
• latter module:
  ----------------------------------------|------ turn signal section B ------|----------------------
• overwrite:
  ------|--- turn signal section A --|------ turn signal section B ------|----------------------

Avoidance -> LC

simplescreenrecorder-2024-03-05_14.34.26.mp4

LC -> Avoidance

simplescreenrecorder-2024-03-05_14.35.24.mp4

Tests performed

• PASS TIER IV INTERNAL SCENARIOS

Effects on system behavior

Fix unnecessary turn signal.

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

Attention: Patch coverage is 0% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 14.75%. Comparing base (7081a61) to head (5546035).
Report is 20 commits behind head on main.

Files	Patch %	Lines
...or_path_planner_common/src/turn_signal_decider.cpp	0.00%	14 Missing ⚠️
...ing/behavior_path_lane_change_module/src/scene.cpp	0.00%	3 Missing ⚠️
...nning/behavior_path_avoidance_module/src/scene.cpp	0.00%	1 Missing ⚠️
...th_goal_planner_module/src/goal_planner_module.cpp	0.00%	1 Missing ⚠️
...behavior_path_lane_change_module/src/interface.cpp	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6543      +/-   ##
==========================================
- Coverage   14.80%   14.75%   -0.06%     
==========================================
  Files        1917     1917              
  Lines      132086   132963     +877     
  Branches    39262    39446     +184     
==========================================
+ Hits        19553    19614      +61     
- Misses      90715    91501     +786     
- Partials    21818    21848      +30     

Flag	Coverage Δ		*Carryforward flag
differential	10.74% <0.00%> (?)
total	14.80% <ø> (+<0.01%)	⬆️	Carriedforward from 7081a61

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

[satoshi-ota]

Close this PR. I'll create new PR if need.