From f66ac5620c1a3a3cad461a3d0b543b5658e5e5a4 Mon Sep 17 00:00:00 2001
From: Yoshiki Obinata <mqcmd196@hotmail.co.jp>
Date: Tue, 20 Apr 2021 13:05:00 +0900
Subject: [PATCH 1/3] rename rosbag launch files

---
 .../jsk_spot_startup/launch/{play.launch => rosbag_play.launch}   | 0
 .../launch/{record.launch => rosbag_record.launch}                | 0
 2 files changed, 0 insertions(+), 0 deletions(-)
 rename jsk_spot_robot/jsk_spot_startup/launch/{play.launch => rosbag_play.launch} (100%)
 rename jsk_spot_robot/jsk_spot_startup/launch/{record.launch => rosbag_record.launch} (100%)

diff --git a/jsk_spot_robot/jsk_spot_startup/launch/play.launch b/jsk_spot_robot/jsk_spot_startup/launch/rosbag_play.launch
similarity index 100%
rename from jsk_spot_robot/jsk_spot_startup/launch/play.launch
rename to jsk_spot_robot/jsk_spot_startup/launch/rosbag_play.launch
diff --git a/jsk_spot_robot/jsk_spot_startup/launch/record.launch b/jsk_spot_robot/jsk_spot_startup/launch/rosbag_record.launch
similarity index 100%
rename from jsk_spot_robot/jsk_spot_startup/launch/record.launch
rename to jsk_spot_robot/jsk_spot_startup/launch/rosbag_record.launch

From 3453a41fddc0886129694de86c3919d89661b290 Mon Sep 17 00:00:00 2001
From: Yoshiki Obinata <mqcmd196@hotmail.co.jp>
Date: Tue, 20 Apr 2021 14:55:49 +0900
Subject: [PATCH 2/3] add duration, start etc...

---
 .../jsk_spot_startup/launch/rosbag_play.launch      | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/jsk_spot_robot/jsk_spot_startup/launch/rosbag_play.launch b/jsk_spot_robot/jsk_spot_startup/launch/rosbag_play.launch
index 302ada90c7..83fe1fa3bb 100644
--- a/jsk_spot_robot/jsk_spot_startup/launch/rosbag_play.launch
+++ b/jsk_spot_robot/jsk_spot_startup/launch/rosbag_play.launch
@@ -1,11 +1,18 @@
 <launch>
     <arg name="rosbag" doc="rosbag file path" />
     <arg name="use_decompress" default="true" />
-    <arg name="use_rviz" default="true" />
+    <arg name="loop" default="true" />
+    <arg name="gui" default="true" />
+    <arg name="loop_flag" value="--loop" if="$(arg loop)" />
+    <arg name="loop_flag" value="" unless="$(arg loop)" />
+    <arg name="start_time" default="0" />
+    <arg name="duration_time" default="0" />
+    <arg name="duration" default="--duration $(arg duration_time)" if="$(eval duration_time != 0)" />
+    <arg name="duration" default="" if="$(eval duration_time == 0)" />
 
     <include
         file="$(find jsk_spot_startup)/launch/rviz.launch"
-        if="$(arg use_rviz)"
+        if="$(arg gui)"
         />
 
     <param name="/use_sim_time" value="true" />
@@ -24,7 +31,7 @@
         pkg="rosbag"
         type="play"
         name="$(anon rosbag)"
-        args="$(arg rosbag) --clock"
+        args="$(arg rosbag) $(arg loop_flag) --clock --start $(arg start_time) $(arg duration)"
         launch-prefix="xterm -e"
         />
     <node

From 9ed80e67d9df0ffdca7b343783e999d2e7786516 Mon Sep 17 00:00:00 2001
From: Yoshiki Obinata <mqcmd196@hotmail.co.jp>
Date: Tue, 20 Apr 2021 15:13:36 +0900
Subject: [PATCH 3/3] add xterm to exec_depend for using rosbag play

---
 jsk_spot_robot/jsk_spot_startup/package.xml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/jsk_spot_robot/jsk_spot_startup/package.xml b/jsk_spot_robot/jsk_spot_startup/package.xml
index 309df2e161..7effcb44ae 100644
--- a/jsk_spot_robot/jsk_spot_startup/package.xml
+++ b/jsk_spot_robot/jsk_spot_startup/package.xml
@@ -21,6 +21,7 @@
   <exec_depend>spot_driver</exec_depend>
   <exec_depend>respeaker_ros</exec_depend>
   <exec_depend>cmd_vel_smoother</exec_depend>
+  <exec_depend>xterm</exec_depend>
 
   <test_depend>rostest</test_depend>