From 933772a7f794d4119c03acb40c782274d594973e Mon Sep 17 00:00:00 2001
From: scepter914 <scepter914@gmail.com>
Date: Sun, 28 Jan 2024 22:50:38 +0900
Subject: [PATCH 1/5] fix parameter files

Signed-off-by: scepter914 <scepter914@gmail.com>
---
 .../config/radar_tracks_msgs_converter.param.yaml          | 6 ++++++
 common_sensor_launch/launch/ars408.launch.xml              | 7 +++++--
 2 files changed, 11 insertions(+), 2 deletions(-)
 create mode 100644 common_sensor_launch/config/radar_tracks_msgs_converter.param.yaml

diff --git a/common_sensor_launch/config/radar_tracks_msgs_converter.param.yaml b/common_sensor_launch/config/radar_tracks_msgs_converter.param.yaml
new file mode 100644
index 00000000..5a30a6bb
--- /dev/null
+++ b/common_sensor_launch/config/radar_tracks_msgs_converter.param.yaml
@@ -0,0 +1,6 @@
+/**:
+  ros__parameters:
+    update_rate_hz: 20.0
+    use_twist_compensation: true
+    use_twist_yaw_compensation: false
+    static_object_speed_threshold: 1.0
diff --git a/common_sensor_launch/launch/ars408.launch.xml b/common_sensor_launch/launch/ars408.launch.xml
index 0ba70569..6fea93b0 100644
--- a/common_sensor_launch/launch/ars408.launch.xml
+++ b/common_sensor_launch/launch/ars408.launch.xml
@@ -11,6 +11,10 @@
   <arg name="publish_radar_scan" default="false" />
   <arg name="size_x" default="1.8" />
   <arg name="size_y" default="1.8" />
+
+  <!-- parameter path -->
+  <arg name="radar_tracks_msgs_converter_param_path" default="$(find-pkg-share common_sensor_launch)/config/radar_tracks_msgs_converter.param.yaml"/>
+
   <!-- noise filter parameter -->
   <arg name="velocity_y_threshold" default="7.0" />
   <!-- converter parameter -->
@@ -52,7 +56,6 @@
     <arg name="input/odometry" value="$(var input/odometry)"/>
     <arg name="output/radar_detected_objects" value="detected_objects"/>
     <arg name="output/radar_tracked_objects" value="tracked_objects"/>
-    <arg name="update_rate_hz" value="10.0"/>
-    <arg name="use_twist_compensation" value="true"/>
+    <arg name="param_path" value="$(var radar_tracks_msgs_converter_param_path)"/>
   </include>
 </launch>

From 8e69eae69c10a32c74e6c8472f1e6fe6ac7ad345 Mon Sep 17 00:00:00 2001
From: scepter914 <scepter914@gmail.com>
Date: Sun, 28 Jan 2024 23:51:51 +0900
Subject: [PATCH 2/5] fix parameter files

Signed-off-by: scepter914 <scepter914@gmail.com>
---
 .../config/radar_tracks_noise_filter.param.yaml       |  4 ++++
 common_sensor_launch/launch/ars408.launch.xml         | 11 ++++-------
 2 files changed, 8 insertions(+), 7 deletions(-)
 create mode 100644 common_sensor_launch/config/radar_tracks_noise_filter.param.yaml

diff --git a/common_sensor_launch/config/radar_tracks_noise_filter.param.yaml b/common_sensor_launch/config/radar_tracks_noise_filter.param.yaml
new file mode 100644
index 00000000..894e38d8
--- /dev/null
+++ b/common_sensor_launch/config/radar_tracks_noise_filter.param.yaml
@@ -0,0 +1,4 @@
+/**:
+  ros__parameters:
+    velocity_y_threshold: 7.0
+
diff --git a/common_sensor_launch/launch/ars408.launch.xml b/common_sensor_launch/launch/ars408.launch.xml
index 6fea93b0..19451736 100644
--- a/common_sensor_launch/launch/ars408.launch.xml
+++ b/common_sensor_launch/launch/ars408.launch.xml
@@ -11,14 +11,11 @@
   <arg name="publish_radar_scan" default="false" />
   <arg name="size_x" default="1.8" />
   <arg name="size_y" default="1.8" />
-
-  <!-- parameter path -->
-  <arg name="radar_tracks_msgs_converter_param_path" default="$(find-pkg-share common_sensor_launch)/config/radar_tracks_msgs_converter.param.yaml"/>
-
-  <!-- noise filter parameter -->
-  <arg name="velocity_y_threshold" default="7.0" />
   <!-- converter parameter -->
   <arg name="input/odometry" default="/localization/kinematic_state" />
+  <!-- parameter path -->
+  <arg name="radar_tracks_msgs_converter_param_path" default="$(find-pkg-share common_sensor_launch)/config/radar_tracks_msgs_converter.param.yaml"/>
+  <arg name="radar_tracks_noise_filter_param_path" default="$(find-pkg-share common_sensor_launch)/config/radar_tracks_noise_filter.param.yaml"/>
 
   <!-- Socket CAN -->
   <group if="$(var launch_driver)">
@@ -47,7 +44,7 @@
     <arg name="input/tracks" value="objects_raw"/>
     <arg name="output/noise_tracks" value="noise_objects" />
     <arg name="output/filtered_tracks" value="filtered_objects" />
-    <arg name="velocity_y_threshold" value="$(var velocity_y_threshold)" />
+    <arg name="param_path" value="$(var radar_tracks_noise_filter_param_path)"/>
   </include>
 
   <!-- Message converter -->

From 082e10e8b3ce3204a97363a4851aa1bd69f7155d Mon Sep 17 00:00:00 2001
From: scepter914 <scepter914@gmail.com>
Date: Mon, 29 Jan 2024 13:41:36 +0900
Subject: [PATCH 3/5] add a config file of simple_object_merger

Signed-off-by: scepter914 <scepter914@gmail.com>
---
 aip_x2_launch/config/simple_object_merger.param.yaml | 7 +++++++
 aip_x2_launch/launch/radar.launch.xml                | 5 +----
 2 files changed, 8 insertions(+), 4 deletions(-)
 create mode 100644 aip_x2_launch/config/simple_object_merger.param.yaml

diff --git a/aip_x2_launch/config/simple_object_merger.param.yaml b/aip_x2_launch/config/simple_object_merger.param.yaml
new file mode 100644
index 00000000..8497ae8b
--- /dev/null
+++ b/aip_x2_launch/config/simple_object_merger.param.yaml
@@ -0,0 +1,7 @@
+/**:
+  ros__parameters:
+    update_rate_hz: 20.0
+    new_frame_id: "base_link"
+    timeout_threshold: 1.0
+    input_topics: ["/sensing/radar/front_center/detected_objects", "/sensing/radar/front_left/detected_objects", "/sensing/radar/rear_left/detected_objects", "/sensing/radar/rear_center/detected_objects", "/sensing/radar/rear_right/detected_objects", "/sensing/radar/front_right/detected_objects"]
+
diff --git a/aip_x2_launch/launch/radar.launch.xml b/aip_x2_launch/launch/radar.launch.xml
index 73e36ecf..042e9d81 100644
--- a/aip_x2_launch/launch/radar.launch.xml
+++ b/aip_x2_launch/launch/radar.launch.xml
@@ -61,10 +61,7 @@
 
     <include file="$(find-pkg-share simple_object_merger)/launch/simple_object_merger.launch.xml">
       <arg name="output/objects" value="detected_objects"/>
-      <arg name="update_rate_hz" value="20.0"/>
-      <arg name="new_frame_id" value="base_link"/>
-      <arg name="timeout_threshold" value="1.0"/>
-      <arg name="input_topics" value="[/sensing/radar/front_center/detected_objects, /sensing/radar/front_left/detected_objects, /sensing/radar/rear_left/detected_objects, /sensing/radar/rear_center/detected_objects, /sensing/radar/rear_right/detected_objects, /sensing/radar/front_right/detected_objects]"/>
+      <arg name="param_path" value="$(find-pkg-share aip_x2_launch)/config/simple_object_merger.param.yaml"/>
     </include>
 
   </group>

From ef97e34108ed965bf614f6ca1633ead1cbcbec30 Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Mon, 29 Jan 2024 07:54:25 +0000
Subject: [PATCH 4/5] ci(pre-commit): autofix

---
 aip_x2_launch/config/simple_object_merger.param.yaml             | 1 -
 common_sensor_launch/config/radar_tracks_noise_filter.param.yaml | 1 -
 2 files changed, 2 deletions(-)

diff --git a/aip_x2_launch/config/simple_object_merger.param.yaml b/aip_x2_launch/config/simple_object_merger.param.yaml
index 8497ae8b..a817e0b7 100644
--- a/aip_x2_launch/config/simple_object_merger.param.yaml
+++ b/aip_x2_launch/config/simple_object_merger.param.yaml
@@ -4,4 +4,3 @@
     new_frame_id: "base_link"
     timeout_threshold: 1.0
     input_topics: ["/sensing/radar/front_center/detected_objects", "/sensing/radar/front_left/detected_objects", "/sensing/radar/rear_left/detected_objects", "/sensing/radar/rear_center/detected_objects", "/sensing/radar/rear_right/detected_objects", "/sensing/radar/front_right/detected_objects"]
-
diff --git a/common_sensor_launch/config/radar_tracks_noise_filter.param.yaml b/common_sensor_launch/config/radar_tracks_noise_filter.param.yaml
index 894e38d8..0d7ddd34 100644
--- a/common_sensor_launch/config/radar_tracks_noise_filter.param.yaml
+++ b/common_sensor_launch/config/radar_tracks_noise_filter.param.yaml
@@ -1,4 +1,3 @@
 /**:
   ros__parameters:
     velocity_y_threshold: 7.0
-

From 8e8dc7d4c6b76ec29d4c667e8bf2a46b2631ad3f Mon Sep 17 00:00:00 2001
From: scepter914 <scepter914@gmail.com>
Date: Thu, 1 Feb 2024 13:02:44 +0900
Subject: [PATCH 5/5] fix CMakeFile

Signed-off-by: scepter914 <scepter914@gmail.com>
---
 common_sensor_launch/CMakeLists.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/common_sensor_launch/CMakeLists.txt b/common_sensor_launch/CMakeLists.txt
index 0b3dcbd5..60a50fc2 100644
--- a/common_sensor_launch/CMakeLists.txt
+++ b/common_sensor_launch/CMakeLists.txt
@@ -10,5 +10,6 @@ if(BUILD_TESTING)
 endif()
 
 ament_auto_package(INSTALL_TO_SHARE
+  config
   launch
 )