Merge pull request #51 from husarion/ros2-improvements

Ros2-improvements
husarion · Jun 12, 2024 · b39e2e2 · b39e2e2
2 parents 602be76 + da9b19b
commit b39e2e2
Show file tree

Hide file tree

Showing 17 changed files with 29 additions and 98 deletions.
diff --git a/config/gz_orbbec_astra_remappings.yaml b/config/gz_orbbec_astra_remappings.yaml
@@ -1,31 +1,27 @@
 ---
 # https://github.com/orbbec/OrbbecSDK_ROS2#all-available-topics
-- ros_topic_name: <robot_namespace><device_namespace>/<camera_name>/color/camera_info
-  gz_topic_name: <robot_namespace><device_namespace>/color/camera_info
+- topic_name: <robot_namespace><device_namespace>/color/camera_info
   ros_type_name: sensor_msgs/msg/CameraInfo
   gz_type_name: ignition.msgs.CameraInfo
   lazy: true
 
-- ros_topic_name: <robot_namespace><device_namespace>/<camera_name>/color/image_raw
-  gz_topic_name: <robot_namespace><device_namespace>/color/image_raw
+- topic_name: <robot_namespace><device_namespace>/color/image_raw
   ros_type_name: sensor_msgs/msg/Image
   gz_type_name: ignition.msgs.Image
   lazy: true
 
-- ros_topic_name: <robot_namespace><device_namespace>/<camera_name>/depth/camera_info
-  gz_topic_name: <robot_namespace><device_namespace>/depth/camera_info
+- topic_name: <robot_namespace><device_namespace>/depth/camera_info
   ros_type_name: sensor_msgs/msg/CameraInfo
   gz_type_name: ignition.msgs.CameraInfo
   lazy: true
 
-- ros_topic_name: <robot_namespace><device_namespace>/<camera_name>/depth/image_raw
-  gz_topic_name: <robot_namespace><device_namespace>/depth/image_raw
+- topic_name: <robot_namespace><device_namespace>/depth/image_raw
   ros_type_name: sensor_msgs/msg/Image
   gz_type_name: ignition.msgs.Image
   lazy: true
 
-- ros_topic_name: <robot_namespace><device_namespace>/<camera_name>/depth/points
-  gz_ros_topic_name: <robot_namespace><device_namespace>/depth/image_raw/points
+- ros_topic_name: <robot_namespace><device_namespace>/depth/points
+  gz_topic_name: <robot_namespace><device_namespace>/depth/image_raw/points
   ros_type_name: sensor_msgs/msg/PointCloud2
   gz_type_name: ignition.msgs.PointCloudPacked
   lazy: true
diff --git a/config/gz_ouster_os_remappings.yaml b/config/gz_ouster_os_remappings.yaml
@@ -4,3 +4,4 @@
     gz_topic_name: <robot_namespace><device_namespace>/ouster/points/points
     ros_type_name: sensor_msgs/msg/PointCloud2
     gz_type_name: ignition.msgs.PointCloudPacked
+    lazy: true
diff --git a/config/gz_slamtec_rplidar_remappings.yaml b/config/gz_slamtec_rplidar_remappings.yaml
@@ -2,3 +2,4 @@
   - topic_name: <robot_namespace><device_namespace>/scan
     ros_type_name: sensor_msgs/msg/LaserScan
     gz_type_name: ignition.msgs.LaserScan
+    lazy: true
diff --git a/config/gz_velodyne_remappings.yaml b/config/gz_velodyne_remappings.yaml
@@ -4,3 +4,4 @@
     gz_topic_name: <robot_namespace><device_namespace>/velodyne_points/points
     ros_type_name: sensor_msgs/msg/PointCloud2
     gz_type_name: ignition.msgs.PointCloudPacked
+    lazy: true
diff --git a/launch/gz_orbbec_astra.launch.py b/launch/gz_orbbec_astra.launch.py
@@ -26,25 +26,15 @@
 def fix_depth_image_tf(context, *args, **kwargs):
     robot_namespace = LaunchConfiguration("robot_namespace").perform(context)
     device_namespace = LaunchConfiguration("device_namespace").perform(context)
-    tf_prefix = LaunchConfiguration("tf_prefix").perform(context)
-    camera_name = LaunchConfiguration("camera_name").perform(context)
-
-    device_namespace_ext = device_namespace + "/"
-    if device_namespace == "":
-        device_namespace_ext = ""
-
-    tf_prefix_ext = tf_prefix + "_"
-    if tf_prefix == "":
-        tf_prefix_ext = ""
-
-    parent_frame = tf_prefix_ext + camera_name + "_depth_optical_frame"
-    child_frame = (
-        "panther/base_link//"
-        + device_namespace_ext
-        + tf_prefix_ext
-        + camera_name
-        + "_orbbec_astra_depth"
-    )
+
+    if robot_namespace.startswith('/'):
+        robot_namespace = robot_namespace[1:] + "/"
+
+    if device_namespace.startswith('/'):
+        device_namespace = device_namespace[1:]
+
+    parent_frame = robot_namespace + device_namespace + "_depth_optical_frame"
+    child_frame = "panther/base_link/" + robot_namespace + device_namespace + "_orbbec_astra_depth"
 
     static_transform_publisher = Node(
         package="tf2_ros",
@@ -66,15 +56,13 @@ def generate_launch_description():
 
     robot_namespace = LaunchConfiguration("robot_namespace")
     device_namespace = LaunchConfiguration("device_namespace")
-    camera_name = LaunchConfiguration("camera_name")
     gz_bridge_name = LaunchConfiguration("gz_bridge_name")
 
     namespaced_gz_bridge_config_path = ReplaceString(
         source_file=gz_bridge_config_path,
         replacements={
             "<robot_namespace>": robot_namespace,
             "<device_namespace>": device_namespace,
-            "<camera_name>": camera_name,
         },
     )
 
@@ -84,24 +72,12 @@ def generate_launch_description():
         description="Sensor namespace that will appear before all non absolute topics and TF frames, used for distinguishing multiple cameras on the same robot.",
     )
 
-    declare_tf_prefix = DeclareLaunchArgument(
-        "tf_prefix",
-        default_value="",
-        description="Prefix added for all links of device. Here used as fix for static transform publisher.",
-    )
-
     declare_robot_namespace = DeclareLaunchArgument(
         "robot_namespace",
         default_value=EnvironmentVariable("ROBOT_NAMESPACE", default_value=""),
         description="Namespace which will appear in front of all topics (including /tf and /tf_static).",
     )
 
-    declare_camera_name = DeclareLaunchArgument(
-        "camera_name",
-        default_value="camera",
-        description="Name of the camera. It will appear before all tfs and topics.",
-    )
-
     declare_gz_bridge_name = DeclareLaunchArgument(
         "gz_bridge_name",
         default_value="gz_bridge",
@@ -121,8 +97,6 @@ def generate_launch_description():
         [
             declare_device_namespace,
             declare_robot_namespace,
-            declare_tf_prefix,
-            declare_camera_name,
             declare_gz_bridge_name,
             gz_bridge,
             OpaqueFunction(function=fix_depth_image_tf),

diff --git a/urdf/components.urdf.xacro b/urdf/components.urdf.xacro
@@ -28,12 +28,12 @@
       <xacro:property name="device_namespace" value="${component['device_namespace']}"
         scope="parent" />
 
-      <xacro:if value="${robot_namespace == ''}">
-        <xacro:property name="ns" value="None" scope="parent" />
-      </xacro:if>
-      <xacro:unless value="${robot_namespace == ''}">
-        <xacro:property name="ns" value="${robot_namespace}" scope="parent" />
-      </xacro:unless>
+        <xacro:if value="${robot_namespace == ''}">
+          <xacro:property name="ns" value="None" scope="parent" />
+        </xacro:if>
+        <xacro:unless value="${robot_namespace == ''}">
+          <xacro:property name="ns" value="${robot_namespace}" scope="parent" />
+        </xacro:unless>
 
       <xacro:if value="${type == 'CAM01'}">
         <xacro:include

diff --git a/urdf/intel_realsense_d435.urdf.xacro b/urdf/intel_realsense_d435.urdf.xacro
@@ -192,9 +192,7 @@
           <near>0.02</near>
           <far>300.0</far>
         </clip>
-        <ros>
-          <namespace>${ns}${device_namespace}</namespace>
-        </ros>
+
       </sensor>
 
       <sensor type="depth_camera" name="${ns}${device_namespace}_intel_realsense_d435_depth">
@@ -222,9 +220,7 @@
             <stddev>0.005</stddev>
           </noise>
         </camera>
-        <ros>
-          <namespace>${ns}${device_namespace}</namespace>
-        </ros>
+
       </sensor>
     </gazebo>
   </xacro:macro>

diff --git a/urdf/kinova.urdf.xacro b/urdf/kinova.urdf.xacro
@@ -186,9 +186,6 @@
           <visualize>true</visualize>
           <topic>${ns}${device_namespace}/wrist_mounted_camera</topic>
           <enable_metrics>false</enable_metrics>
-          <ros>
-            <namespace>${ns}${device_namespace}</namespace>
-          </ros>
         </sensor>
       </gazebo>
 

diff --git a/urdf/orbbec_astra.urdf.xacro b/urdf/orbbec_astra.urdf.xacro
@@ -100,7 +100,7 @@
         <topic>${ns}${device_namespace}/color/image_raw</topic>
         <visualize>false</visualize>
 
-        <ignition_frame_id>${device_namespace}_color_optical_frame</ignition_frame_id>
+        <ignition_frame_id>${ns}${device_namespace}_color_optical_frame</ignition_frame_id>
         <horizontal_fov>${60.0/180.0*pi}</horizontal_fov>
         <image>
           <width>640</width>
@@ -111,9 +111,6 @@
           <near>0.6</near>
           <far>8.0</far>
         </clip>
-        <ros>
-          <namespace>${ns}${device_namespace}</namespace>
-        </ros>
       </sensor>
 
       <sensor type="depth_camera" name="${ns}${device_namespace}_orbbec_astra_depth">
@@ -124,7 +121,7 @@
         <topic>${ns}${device_namespace}/depth/image_raw</topic>
         <visualize>false</visualize>
 
-        <ignition_frame_id>${device_namespace}_depth_optical_frame</ignition_frame_id>
+        <ignition_frame_id>${ns}${device_namespace}_depth_optical_frame</ignition_frame_id>
         <camera>
           <horizontal_fov>${60.0/180.0*pi}</horizontal_fov>
           <image>
@@ -142,9 +139,6 @@
             <stddev>0.005</stddev>
           </noise>
         </camera>
-        <ros>
-          <namespace>${ns}${device_namespace}</namespace>
-        </ros>
       </sensor>
     </gazebo>
 

diff --git a/urdf/ouster.urdf.xacro b/urdf/ouster.urdf.xacro
@@ -146,11 +146,6 @@
         <frame_id>${ns}${prefix}os_lidar</frame_id>
         <ignition_frame_id>${ns}${prefix}os_lidar</ignition_frame_id>
         <always_on>true</always_on>
-
-        <ros>
-          <namespace>${ns}${device_ns}</namespace>
-        </ros>
-
       </sensor>
     </gazebo>
   </xacro:macro>

diff --git a/urdf/slamtec_rplidar_a2.urdf.xacro b/urdf/slamtec_rplidar_a2.urdf.xacro
@@ -123,9 +123,6 @@
         </ray>
         <always_on>true</always_on>
         <visualize>false</visualize>
-        <ros>
-          <namespace>${ns}${device_ns}</namespace>
-        </ros>
       </sensor>
     </gazebo>
   </xacro:macro>

diff --git a/urdf/slamtec_rplidar_a3.urdf.xacro b/urdf/slamtec_rplidar_a3.urdf.xacro
@@ -99,9 +99,6 @@
         </ray>
         <always_on>true</always_on>
         <visualize>false</visualize>
-        <ros>
-          <namespace>${ns}${device_ns}</namespace>
-        </ros>
       </sensor>
     </gazebo>
 

diff --git a/urdf/slamtec_rplidar_s1.urdf.xacro b/urdf/slamtec_rplidar_s1.urdf.xacro
@@ -105,9 +105,6 @@
         </ray>
         <always_on>true</always_on>
         <visualize>false</visualize>
-        <ros>
-          <namespace>${ns}${device_ns}</namespace>
-        </ros>
       </sensor>
     </gazebo>
 

diff --git a/urdf/slamtec_rplidar_s2.urdf.xacro b/urdf/slamtec_rplidar_s2.urdf.xacro
@@ -97,9 +97,6 @@
         </ray>
         <always_on>1</always_on>
         <visualize>false</visualize>
-        <ros>
-          <namespace>${ns}${device_ns}</namespace>
-        </ros>
       </sensor>
     </gazebo>
 

diff --git a/urdf/slamtec_rplidar_s3.urdf.xacro b/urdf/slamtec_rplidar_s3.urdf.xacro
@@ -97,9 +97,6 @@
         </ray>
         <always_on>1</always_on>
         <visualize>false</visualize>
-        <ros>
-          <namespace>${ns}${device_ns}</namespace>
-        </ros>
       </sensor>
     </gazebo>
 

diff --git a/urdf/stereolabs_zed.urdf.xacro b/urdf/stereolabs_zed.urdf.xacro
@@ -199,9 +199,6 @@
             <far>300.0</far>
           </clip>
         </camera>
-        <ros>
-          <namespace>${ns}</namespace>
-        </ros>
       </sensor>
 
       <sensor type="depth_camera" name="${ns}${device_namespace}_stereolabs_zed_depth">
@@ -228,9 +225,7 @@
             <stddev>0.03</stddev>
           </noise>
         </camera>
-        <ros>
-          <namespace>${ns}${device_namespace}</namespace>
-        </ros>
+
       </sensor>
     </gazebo>
 

diff --git a/urdf/velodyne_puck.urdf.xacro b/urdf/velodyne_puck.urdf.xacro
@@ -103,10 +103,6 @@
         <frame_id>${ns}${prefix}velodyne</frame_id>
         <ignition_frame_id>${ns}${prefix}velodyne</ignition_frame_id>
         <always_on>true</always_on>
-
-        <ros>
-          <namespace>${ns}${device_ns}</namespace>
-        </ros>
       </sensor>
     </gazebo>
   </xacro:macro>