fix(behavior_path_planner): fix simultaneous execution

[kosuke55]

Description

The value of isSimultaneousExecutable* cannot be changed dynamically when the module is not requested.
For example, we want the value of fixed_goal_planner to always be true, but if there is another module ahead of it in the first place, the module will not be requeted and the value cannot be changed.
In this PR, the value is changed to be changed on the manager side.
Also, complex functions were removed.

Related links

Tests performed

psim

LC -> fixed_goal_planner

fixed_goal_planner-2023-08-14_16.34.26.mp4

back pull out -> avoidance

back_pull_out_to_avoidance-2023-08-14_18.08.42.mp4

tier4 scenario simulator 1584/1584

https://github.com/tier4/sim_evaluation_tools_test/issues/109

Notes for reviewers

Interface changes

none

Effects on system behavior

none

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.

[codecov]

Codecov Report

Patch coverage: 1.72% and project coverage change: +0.02% 🎉

Comparison is base (e0e9fcf) 14.82% compared to head (9dc0f33) 14.84%.
Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4622      +/-   ##
==========================================
+ Coverage   14.82%   14.84%   +0.02%     
==========================================
  Files        1529     1529              
  Lines      105813   105832      +19     
  Branches    32244    32249       +5     
==========================================
+ Hits        15690    15716      +26     
+ Misses      72966    72929      -37     
- Partials    17157    17187      +30     

Flag	Coverage Δ		*Carryforward flag
differential	13.23% <1.72%> (?)
total	14.83% <ø> (+<0.01%)	⬆️	Carriedforward from e0e9fcf

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

Files Changed	Coverage Δ
...path_planner/scene_module/goal_planner/manager.hpp	100.00% <ø> (ø)
...th_planner/scene_module/scene_module_interface.hpp	12.23% <ø> (+0.34%)	⬆️
...er/scene_module/scene_module_manager_interface.hpp	28.70% <0.00%> (+3.70%)	⬆️
...ath_planner/scene_module/start_planner/manager.hpp	100.00% <ø> (ø)
...cene_module/start_planner/start_planner_module.hpp	0.00% <0.00%> (ø)
.../scene_module/goal_planner/goal_planner_module.cpp	6.54% <0.00%> (-0.07%)	⬇️
...planner/src/scene_module/start_planner/manager.cpp	9.63% <0.00%> (-2.67%)	⬇️
...cene_module/start_planner/start_planner_module.cpp	4.03% <ø> (+0.03%)	⬆️
...avior_path_planner/src/utils/goal_planner/util.cpp	3.84% <0.00%> (-0.64%)	⬇️
..._planner/src/scene_module/goal_planner/manager.cpp	8.55% <11.11%> (+0.16%)	⬆️

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kosuke55]

@satoshi-ota
evaluator result looks good

tier4 scenario simulator 1584/1584
https://github.com/tier4/sim_evaluation_tools_test/issues/109

[satoshi-ota]

LGTM 👍 Thanks.