From c6596636550cb42f3c466c5919d8c44c3aad9e39 Mon Sep 17 00:00:00 2001
From: Shohei Sakai <saka1s.jp@gmail.com>
Date: Mon, 11 Mar 2024 18:25:20 +0900
Subject: [PATCH] Revert "feat: keep stop policy v0.11.2 (#1125)"

This reverts commit 23d651f72819dca8de63fabd2c1718a13070a9f6.
---
 .../config/avoidance/avoidance.param.yaml        |  4 ----
 .../utils/avoidance/avoidance_module_data.hpp    |  3 ---
 .../scene_module/avoidance/avoidance_module.cpp  | 16 ++--------------
 .../src/scene_module/avoidance/manager.cpp       |  1 -
 4 files changed, 2 insertions(+), 22 deletions(-)

diff --git a/planning/behavior_path_planner/config/avoidance/avoidance.param.yaml b/planning/behavior_path_planner/config/avoidance/avoidance.param.yaml
index 9ba7fb717b4bb..7c8fa1ac18348 100644
--- a/planning/behavior_path_planner/config/avoidance/avoidance.param.yaml
+++ b/planning/behavior_path_planner/config/avoidance/avoidance.param.yaml
@@ -192,10 +192,6 @@
         stop_buffer: 1.0                                # [m]
 
       policy:
-        # policy for rtc request. select "shift_line" or "maneuver".
-        # "shift_line": request approval for each shift line.
-        # "maneuver": request approval for avoidance maneuver (avoid + return).
-        approval: "shift_line"
         # policy for vehicle slow down behavior. select "best_effort" or "reliable".
         # "best_effort": slow down deceleration & jerk are limited by constraints.
         #                but there is a possibility that the vehicle can't stop in front of the vehicle.
diff --git a/planning/behavior_path_planner/include/behavior_path_planner/utils/avoidance/avoidance_module_data.hpp b/planning/behavior_path_planner/include/behavior_path_planner/utils/avoidance/avoidance_module_data.hpp
index c41829c8d491b..d14a99eb80e73 100644
--- a/planning/behavior_path_planner/include/behavior_path_planner/utils/avoidance/avoidance_module_data.hpp
+++ b/planning/behavior_path_planner/include/behavior_path_planner/utils/avoidance/avoidance_module_data.hpp
@@ -280,9 +280,6 @@ struct AvoidanceParameters
   // policy
   bool use_shorten_margin_immediately{false};
 
-  // policy
-  std::string policy_approval{"shift_line"};
-
   // policy
   std::string policy_deceleration{"best_effort"};
 
diff --git a/planning/behavior_path_planner/src/scene_module/avoidance/avoidance_module.cpp b/planning/behavior_path_planner/src/scene_module/avoidance/avoidance_module.cpp
index 20dfb994ea114..2143e42afd7b6 100644
--- a/planning/behavior_path_planner/src/scene_module/avoidance/avoidance_module.cpp
+++ b/planning/behavior_path_planner/src/scene_module/avoidance/avoidance_module.cpp
@@ -85,11 +85,6 @@ bool isBestEffort(const std::string & policy)
 {
   return policy == "best_effort";
 }
-
-bool perManeuver(const std::string & policy)
-{
-  return policy == "maneuver";
-}
 }  // namespace
 
 AvoidanceModule::AvoidanceModule(
@@ -2321,16 +2316,9 @@ AvoidLineArray AvoidanceModule::findNewShiftLine(const AvoidLineArray & candidat
       break;
     }
 
-    if (is_ignore_shift(candidate)) {
-      continue;
+    if (!is_ignore_shift(candidate)) {
+      return get_subsequent_shift(i);
     }
-
-    if (perManeuver(parameters_->policy_approval)) {
-      return candidates;
-    }
-
-    const auto new_shift_lines = get_subsequent_shift(i);
-    return new_shift_lines;
   }
 
   DEBUG_PRINT("No new shift point exists.");
diff --git a/planning/behavior_path_planner/src/scene_module/avoidance/manager.cpp b/planning/behavior_path_planner/src/scene_module/avoidance/manager.cpp
index 461341ff488e6..871932e1e2232 100644
--- a/planning/behavior_path_planner/src/scene_module/avoidance/manager.cpp
+++ b/planning/behavior_path_planner/src/scene_module/avoidance/manager.cpp
@@ -218,7 +218,6 @@ AvoidanceModuleManager::AvoidanceModuleManager(
   // policy
   {
     std::string ns = "avoidance.policy.";
-    p.policy_approval = getOrDeclareParameter<std::string>(*node, ns + "approval");
     p.policy_deceleration = getOrDeclareParameter<std::string>(*node, ns + "deceleration");
     p.policy_lateral_margin = getOrDeclareParameter<std::string>(*node, ns + "lateral_margin");
     p.use_shorten_margin_immediately =