refactor(run_out): remove unused parameter

[TomohitoAndo]

Signed-off-by: Tomohito Ando tomohito.ando@tier4.jp

Description

Removed unused parameter.

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

Merging #1836 (ccdb112) into main (cb2552b) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1836      +/-   ##
==========================================
- Coverage   10.44%   10.43%   -0.01%     
==========================================
  Files        1221     1221              
  Lines       87728    87808      +80     
  Branches    20493    20541      +48     
==========================================
+ Hits         9160     9161       +1     
- Misses      69091    69167      +76     
- Partials     9477     9480       +3     

Flag	Coverage Δ		*Carryforward flag
differential	4.29% <0.00%> (?)
total	10.42% <0.00%> (ø)		Carriedforward from cb2552b

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

Impacted Files	Coverage Δ
...include/behavior_velocity_planner/planner_data.hpp	0.00% <0.00%> (ø)
...ity_planner/include/scene_module/run_out/utils.hpp	0.00% <ø> (ø)
planning/behavior_velocity_planner/src/node.cpp	0.70% <0.00%> (+0.28%)	⬆️
...ocity_planner/src/scene_module/run_out/manager.cpp	0.00% <ø> (ø)

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

[0x126]

LGTM

[TomohitoAndo]

@taikitanaka3
could you review this PR?
I can't merge without code owner review. 🥺

[taikitanaka3]

@TomohitoAndo
I didn't follow conversation but you don't use assumption of pedestrian velocity anymore?

[TomohitoAndo]

@taikitanaka3
Parameters for that assumption are moved to here, so parameters removed in this PR are no longer needed.

autoware.universe/launch/tier4_planning_launch/config/scenario_planning/lane_driving/behavior_planning/behavior_velocity_planner/run_out.param.yaml

Lines 20 to 21 in ccdb112

	min_vel_kmph: 0.0 # [km/h] minimum velocity for dynamic obstacles
	max_vel_kmph: 5.0 # [km/h] maximum velocity for dynamic obstacles

[taikitanaka3]

LGTM

[TomohitoAndo]

ah, it seems review from @TakaHoribe is necessary...? 🤔

[TakaHoribe]

LGTM