Remove rviz static transform publishers hack

ouster-ros/launch/record.independent.launch.xml

@@ -74,7 +74,6 @@
   <include if="$(var viz)" file="$(find-pkg-share ouster_ros)/launch/rviz.launch.xml">
     <arg name="ouster_ns" value="$(var ouster_ns)"/>
     <arg name="rviz_config" value="$(var rviz_config)"/>
-    <arg name="_enable_static_tf_publishers" value="false"/>
   </include>
 
   <let name="_use_bag_file_name" value="$(eval '\'$(var bag_file)\' != \'b\'')"/>

ouster-ros/launch/replay.independent.launch.xml

@@ -51,7 +51,6 @@
   <include if="$(var viz)" file="$(find-pkg-share ouster_ros)/launch/rviz.launch.xml">
     <arg name="ouster_ns" value="$(var ouster_ns)"/>
     <arg name="rviz_config" value="$(var rviz_config)"/>
-    <arg name="_enable_static_tf_publishers" value="false"/>
     <arg name="sensor_frame" value="$(var sensor_frame)"/>
     <arg name="lidar_frame" value="$(var lidar_frame)"/>
     <arg name="imu_frame" value="$(var imu_frame)"/>

ouster-ros/launch/rviz.launch.py

@@ -24,30 +24,6 @@ def generate_launch_description():
     ouster_ns_arg = DeclareLaunchArgument(
         'ouster_ns', default_value='ouster')
 
-    enable_static_tf = LaunchConfiguration('_enable_static_tf_publishers')
-    enable_static_tf_arg = DeclareLaunchArgument(
-        '_enable_static_tf_publishers', default_value='false')
-
-    # NOTE: the two static tf publishers are rather a workaround to let rviz2
-    #     get going and not complain while waiting for the actual sensor frames
-    #     to be published that is when running rviz2 using a parent launch file
-    # TODO: need to be able to propagate the modified frame names from the
-    #       parameters file to RVIZ launch py.
-    sensor_imu_tf = Node(
-        package="tf2_ros",
-        executable="static_transform_publisher",
-        name="stp_sensor_imu",
-        namespace=ouster_ns,
-        condition=IfCondition(enable_static_tf),
-        arguments=["0", "0", "0", "0", "0", "0", "os_sensor", "os_imu"])
-    sensor_ldr_tf = Node(
-        package="tf2_ros",
-        executable="static_transform_publisher",
-        name="stp_sensor_lidar",
-        namespace=ouster_ns,
-        condition=IfCondition(enable_static_tf),
-        arguments=["0", "0", "0", "0", "0", "0", "os_sensor", "os_lidar"])
-
     rviz_node = Node(
         package='rviz2',
         namespace=ouster_ns,
@@ -59,8 +35,5 @@ def generate_launch_description():
     return LaunchDescription([
         ouster_ns_arg,
         rviz_config_arg,
-        enable_static_tf_arg,
-        sensor_imu_tf,
-        sensor_ldr_tf,
         rviz_node
     ])

ouster-ros/launch/rviz.launch.xml

@@ -4,8 +4,6 @@
     description="Override the default namespace of all ouster nodes"/>
   <arg name="rviz_config" default="$(find-pkg-share ouster_ros)/config/viz.rviz"
     description="optional rviz config file"/>
-  <arg name="_enable_static_tf_publishers" default="false"
-    description="boolean value to enable static tf publishers"/>
 
   <arg name="sensor_frame" default="os_sensor" description="value can not be empty"/>
   <arg name="lidar_frame" default="os_lidar" description="value can not be empty"/>
@@ -15,15 +13,6 @@
     <push-ros-namespace namespace="$(var ouster_ns)"/>
     <node pkg="rviz2" exec="rviz2" name="rviz2" output="screen"
       launch-prefix="bash -c 'sleep 5; $0 $@'" args="-d $(var rviz_config)"/>
-    <!-- NOTE: this is rather a workaround to let rviz2 get going and not complain while waiting
-      for the actual sensor frames to be published that is when when using the same launch file
-      to run rviz2 -->
-    <node if="$(var _enable_static_tf_publishers)"
-      pkg="tf2_ros" exec="static_transform_publisher" name="stp_sensor_imu"
-      args="0 0 0 0 0 0 $(var sensor_frame) $(var imu_frame)"/>
-    <node  if="$(var _enable_static_tf_publishers)"
-      pkg="tf2_ros" exec="static_transform_publisher" name="stp_sensor_lidar"
-      args="0 0 0 0 0 0 $(var sensor_frame) $(var lidar_frame)"/>
   </group>
 
 </launch>

ouster-ros/launch/sensor.composite.launch.py

@@ -77,8 +77,7 @@ def generate_launch_description():
         Path(ouster_ros_pkg_dir) / 'launch' / 'rviz.launch.py'
     rviz_launch = IncludeLaunchDescription(
         PythonLaunchDescriptionSource([str(rviz_launch_file_path)]),
-        condition=IfCondition(rviz_enable),
-        launch_arguments={'_enable_static_tf_publishers': 'false'}.items()
+        condition=IfCondition(rviz_enable)
     )
 
     # HACK: to configure and activate the the sensor since state transition

ouster-ros/launch/sensor.composite.launch.xml

@@ -79,7 +79,6 @@
   <include if="$(var viz)" file="$(find-pkg-share ouster_ros)/launch/rviz.launch.xml">
     <arg name="ouster_ns" value="$(var ouster_ns)"/>
     <arg name="rviz_config" value="$(var rviz_config)"/>
-    <arg name="_enable_static_tf_publishers" value="false"/>
     <arg name="sensor_frame" value="$(var sensor_frame)"/>
     <arg name="lidar_frame" value="$(var lidar_frame)"/>
     <arg name="imu_frame" value="$(var imu_frame)"/>

ouster-ros/launch/sensor.independent.launch.py

@@ -105,8 +105,7 @@ def generate_launch_description():
         Path(ouster_ros_pkg_dir) / 'launch' / 'rviz.launch.py'
     rviz_launch = IncludeLaunchDescription(
         PythonLaunchDescriptionSource([str(rviz_launch_file_path)]),
-        condition=IfCondition(rviz_enable),
-        launch_arguments={'_enable_static_tf_publishers': 'false'}.items()
+        condition=IfCondition(rviz_enable)
     )
 
     return launch.LaunchDescription([

ouster-ros/launch/sensor.independent.launch.xml

@@ -77,7 +77,6 @@
   <include if="$(var viz)" file="$(find-pkg-share ouster_ros)/launch/rviz.launch.xml">
     <arg name="ouster_ns" value="$(var ouster_ns)"/>
     <arg name="rviz_config" value="$(var rviz_config)"/>
-    <arg name="_enable_static_tf_publishers" value="false"/>
     <arg name="sensor_frame" value="$(var sensor_frame)"/>
     <arg name="lidar_frame" value="$(var lidar_frame)"/>
     <arg name="imu_frame" value="$(var imu_frame)"/>

ouster-ros/launch/sensor_mtp.launch.xml

@@ -87,7 +87,6 @@
   <include if="$(var viz)" file="$(find-pkg-share ouster_ros)/launch/rviz.launch.xml">
     <arg name="ouster_ns" value="$(var ouster_ns)"/>
     <arg name="rviz_config" value="$(var rviz_config)"/>
-    <arg name="_enable_static_tf_publishers" value="false"/>
     <arg name="sensor_frame" value="$(var sensor_frame)"/>
     <arg name="lidar_frame" value="$(var lidar_frame)"/>
     <arg name="imu_frame" value="$(var imu_frame)"/>