fix(behavior_path_planner): fix planner data copy

[purewater0901]

Signed-off-by: yutaka [email protected]

Description

Since it uses shared_ptr for planner_data_, it is not appropriate to copy it while using mutex. In this PR, I created a new shared_ptr to copy planner_data_ to planner_data. In the new code, the owner of the planner_data_ is one, so I confirmed there is no copy.

Related Issues

• planner_data access outside mutex lock inside behavior_path_planner_node #2495
• behavior_path_planner needs to lock planner_data_ mutex before calling getTurnSignal #2422

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.

[TakaHoribe]

yeah, I know. tmux is too complicated for human being.

[TakaHoribe]

LGTM

[codecov]

Codecov Report

Base: 10.55% // Head: 10.50% // Decreases project coverage by -0.05% ⚠️

Coverage data is based on head (5fdb13d) compared to base (0c6c46b).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2501      +/-   ##
==========================================
- Coverage   10.55%   10.50%   -0.06%     
==========================================
  Files        1264     1264              
  Lines       87353    88052     +699     
  Branches    20943    21388     +445     
==========================================
+ Hits         9224     9253      +29     
- Misses      68247    68854     +607     
- Partials     9882     9945      +63     

Flag	Coverage Δ		*Carryforward flag
differential	3.14% <ø> (?)
total	10.56% <0.00%> (+<0.01%)	⬆️	Carriedforward from a26b69d

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

Impacted Files	Coverage Δ
...or_path_planner/src/behavior_path_planner_node.cpp	0.32% <0.00%> (+0.13%)	⬆️
...planning_evaluator/src/planning_evaluator_node.cpp	34.37% <0.00%> (-2.09%)	⬇️
planning/behavior_path_planner/src/utilities.cpp	3.66% <0.00%> (-0.06%)	⬇️
...vehicle_model/sim_model_ideal_steer_acc_geared.cpp	75.00% <0.00%> (+14.28%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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