Skip to content

Commit

Permalink
rename planPullOver
Browse files Browse the repository at this point in the history
Signed-off-by: kosuke55 <[email protected]>
  • Loading branch information
kosuke55 committed Nov 13, 2023
1 parent 0602a72 commit 0be67be
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -412,8 +412,8 @@ class GoalPlannerModule : public SceneModuleInterface
void returnToLaneParking();

// plan pull over path
BehaviorModuleOutput planRunning();
BehaviorModuleOutput planPullOver();
BehaviorModuleOutput planPullOverAsOutput();
BehaviorModuleOutput planPullOverAsCandidate();
void selectSafePullOverPath();
std::vector<PullOverPath> sortPullOverPathCandidatesByGoalPriority(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -575,7 +575,7 @@ void GoalPlannerModule::generateGoalCandidates()
BehaviorModuleOutput GoalPlannerModule::plan()
{
if (goal_planner_utils::isAllowedGoalModification(planner_data_->route_handler)) {
return planRunning();
return planPullOver();

Check warning on line 578 in planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp

View check run for this annotation

Codecov / codecov/patch

planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp#L578

Added line #L578 was not covered by tests
}

fixed_goal_planner_->setPreviousModuleOutput(getPreviousModuleOutput());
Expand Down Expand Up @@ -879,13 +879,13 @@ void GoalPlannerModule::decideVelocity()
}
}

BehaviorModuleOutput GoalPlannerModule::planRunning()
BehaviorModuleOutput GoalPlannerModule::planPullOver()

Check warning on line 882 in planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp

View check run for this annotation

Codecov / codecov/patch

planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp#L882

Added line #L882 was not covered by tests
{
if (!hasDecidedPath()) {
return planPullOverAsCandidate();

Check warning on line 885 in planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp

View check run for this annotation

Codecov / codecov/patch

planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp#L885

Added line #L885 was not covered by tests
}

return planPullOver();
return planPullOverAsOutput();

Check warning on line 888 in planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp

View check run for this annotation

Codecov / codecov/patch

planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp#L888

Added line #L888 was not covered by tests
}

BehaviorModuleOutput GoalPlannerModule::planPullOverAsCandidate()

Check warning on line 891 in planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp

View check run for this annotation

Codecov / codecov/patch

planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp#L891

Added line #L891 was not covered by tests
Expand All @@ -896,7 +896,7 @@ BehaviorModuleOutput GoalPlannerModule::planPullOverAsCandidate()
}

BehaviorModuleOutput output{};

Check warning on line 898 in planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp

View check run for this annotation

Codecov / codecov/patch

planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp#L898

Added line #L898 was not covered by tests
const BehaviorModuleOutput pull_over_output = planPullOver();
const BehaviorModuleOutput pull_over_output = planPullOverAsOutput();
output.modified_goal = pull_over_output.modified_goal;
output.path = std::make_shared<PathWithLaneId>(generateStopPath());

Expand All @@ -917,7 +917,7 @@ BehaviorModuleOutput GoalPlannerModule::planPullOverAsCandidate()
return output;
}

Check warning on line 918 in planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp

View check run for this annotation

Codecov / codecov/patch

planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp#L917-L918

Added lines #L917 - L918 were not covered by tests

BehaviorModuleOutput GoalPlannerModule::planPullOver()
BehaviorModuleOutput GoalPlannerModule::planPullOverAsOutput()

Check warning on line 920 in planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp

View check run for this annotation

Codecov / codecov/patch

planning/behavior_path_planner/src/scene_module/goal_planner/goal_planner_module.cpp#L920

Added line #L920 was not covered by tests
{
// if pull over path candidates generation is not finished, use previous module output
if (thread_safe_data_.get_pull_over_path_candidates().empty()) {
Expand Down

0 comments on commit 0be67be

Please sign in to comment.