fix(behavior_path_planner): sort keep last modules considering priority

[kosuke55]

Description

sort keep last modules considering priority.
when the parameter is following dynamic_avoiadnce shoulder be last.

    goal_planner:
      enable_rtc: false
      enable_simultaneous_execution_as_approved_module: true
      enable_simultaneous_execution_as_candidate_module: false
      keep_last: true
      priority: 1
      max_module_size: 1

    dynamic_avoidance:
      enable_rtc: false
      enable_simultaneous_execution_as_approved_module: true
      enable_simultaneous_execution_as_candidate_module: true
      keep_last: true
      priority: 7
      max_module_size: 1

before

after

Tests performed

psim

2024/01/25 https://evaluation.tier4.jp/evaluation/reports/2744b0ff-2c3b-5816-94fd-0f0ffe27b725/?project_id=prd_jt

https://evaluation.tier4.jp/evaluation/reports/254e158d-7800-583e-b078-7a9529a53b86?project_id=prd_jt

Effects on system behavior

Not applicable.

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: 20 lines in your changes are missing coverage. Please review.

Comparison is base (ccd852b) 14.58% compared to head (1a0c61b) 14.58%.
Report is 2 commits behind head on main.

Files	Patch %	Lines
...ning/behavior_path_planner/src/planner_manager.cpp	0.00%	20 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6174      +/-   ##
==========================================
- Coverage   14.58%   14.58%   -0.01%     
==========================================
  Files        1871     1871              
  Lines      127545   127556      +11     
  Branches    37311    37315       +4     
==========================================
  Hits        18606    18606              
- Misses      88019    88030      +11     
  Partials    20920    20920              

Flag	Coverage Δ		*Carryforward flag
differential	9.45% <0.00%> (?)
total	14.58% <ø> (+<0.01%)	⬆️	Carriedforward from ccd852b

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

[kosuke55]

seems to need to inherit modified goal

[kosuke55]

seems to need to inherit modified goal

fixed by #6179
https://evaluation.tier4.jp/evaluation/reports/254e158d-7800-583e-b078-7a9529a53b86?project_id=prd_jt