Apply suggestions from code review

Co-authored-by: Yuma Nihei <[email protected]>
tier4 · Sep 25, 2024 · 60a0bed · 60a0bed
1 parent 85a41aa
commit 60a0bed
Show file tree

Hide file tree

Showing 6 changed files with 7 additions and 8 deletions.
diff --git a/...ability_detector/include/autoware/pose_instability_detector/pose_instability_detector.hpp b/...ability_detector/include/autoware/pose_instability_detector/pose_instability_detector.hpp
@@ -1,4 +1,4 @@
-// Copyright 2023- Autoware Foundation
+// Copyright 2024- Autoware Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.

diff --git a/localization/pose_instability_detector/launch/pose_instability_detector.launch.xml b/localization/pose_instability_detector/launch/pose_instability_detector.launch.xml
@@ -4,13 +4,11 @@
 
   <!-- Topics -->
   <arg name="input_odometry" default="~/input/odometry"/>
-  <arg name="input_pose" default="~/input/pose"/>
   <arg name="input_twist" default="~/input/twist"/>
 
   <node pkg="pose_instability_detector" exec="pose_instability_detector_node" name="$(var node_name)" output="both">
     <remap from="~/input/odometry" to="$(var input_odometry)"/>
     <remap from="~/input/twist" to="$(var input_twist)"/>
-    <remap from="~/input/pose" to="$(var input_pose)"/>
     <param from="$(var param_file)"/>
   </node>
 </launch>
diff --git a/localization/pose_instability_detector/schema/pose_instability_detector.schema.json b/localization/pose_instability_detector/schema/pose_instability_detector.schema.json
@@ -19,7 +19,7 @@
         },
         "heading_velocity_maximum": {
           "type": "number",
-          "default": 16.667,
+          "default": 2.778,
           "minimum": 0.0,
           "description": "The maximum of heading velocity (m/s)."
         },
@@ -73,7 +73,8 @@
         }
       },
       "required": [
-        "timer_period",
+        "window_length",
+        "output_x_y_yaw_only",
         "heading_velocity_maximum",
         "heading_velocity_scale_factor_tolerance",
         "angular_velocity_maximum",

diff --git a/localization/pose_instability_detector/src/pose_instability_detector.cpp b/localization/pose_instability_detector/src/pose_instability_detector.cpp
@@ -1,4 +1,4 @@
-// Copyright 2023- Autoware Foundation
+// Copyright 2024- Autoware Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.

diff --git a/localization/pose_instability_detector/test/test.cpp b/localization/pose_instability_detector/test/test.cpp
@@ -1,4 +1,4 @@
-// Copyright 2023- Autoware Foundation
+// Copyright 2024- Autoware Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.

diff --git a/localization/pose_instability_detector/test/test_message_helper_node.hpp b/localization/pose_instability_detector/test/test_message_helper_node.hpp
@@ -1,4 +1,4 @@
-// Copyright 2023- Autoware Foundation
+// Copyright 2024- Autoware Foundation
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.