From 442e5bad2c9c93bdca7f126829e90f97785208d1 Mon Sep 17 00:00:00 2001
From: Jakub Delicat <jakub.delicat@husarion.com>
Date: Wed, 8 May 2024 18:35:47 +0200
Subject: [PATCH] removed simulation-engine from s3 | fix with no namespace

Signed-off-by: Jakub Delicat <jakub.delicat@husarion.com>
---
 launch/gz_components.launch.py     |  4 +-
 urdf/slamtec_rplidar_s3.urdf.xacro | 73 +++++++++++++++---------------
 2 files changed, 38 insertions(+), 39 deletions(-)

diff --git a/launch/gz_components.launch.py b/launch/gz_components.launch.py
index 98cf262..aaa0f70 100644
--- a/launch/gz_components.launch.py
+++ b/launch/gz_components.launch.py
@@ -41,9 +41,9 @@ def get_launch_description(name: str, package: str, namespace: str, component: y
     device_namespace = get_value(component, "device_namespace")
     robot_namespace = namespace
 
-    if robot_namespace[0] != "/":
+    if len(robot_namespace) and robot_namespace[0] != "/":
         robot_namespace = "/" + robot_namespace
-    if device_namespace[0] != "/":
+    if len(device_namespace) and device_namespace[0] != "/":
         device_namespace = "/" + device_namespace
 
     return IncludeLaunchDescription(
diff --git a/urdf/slamtec_rplidar_s3.urdf.xacro b/urdf/slamtec_rplidar_s3.urdf.xacro
index f79b96a..629e506 100644
--- a/urdf/slamtec_rplidar_s3.urdf.xacro
+++ b/urdf/slamtec_rplidar_s3.urdf.xacro
@@ -65,44 +65,43 @@
 
     <link name="${prefix}laser" />
 
-    <xacro:if value="${simulation_engine == 'ignition-gazebo'}">
-      <gazebo reference="${prefix}laser">
-        <!-- gpu_lidar has to be set, CPU lidar doesn't work in ignition -
-        https://github.com/gazebosim/gz-sensors/issues/26 -->
-        <sensor type="gpu_lidar" name="${ns}${prefix}rplidar_s3_sensor">
+    <gazebo reference="${prefix}laser">
+      <!-- gpu_lidar has to be set, CPU lidar doesn't work in ignition -
+      https://github.com/gazebosim/gz-sensors/issues/26 -->
+      <sensor type="gpu_lidar" name="${ns}${prefix}rplidar_s3_sensor">
 
-          <topic>${ns}${device_ns}scan</topic>
-          <frame_id>${ns}${prefix}laser</frame_id>
-          <ignition_frame_id>${ns}${prefix}laser</ignition_frame_id>
+        <topic>${ns}${device_ns}scan</topic>
+        <frame_id>${ns}${prefix}laser</frame_id>
+        <ignition_frame_id>${ns}${prefix}laser</ignition_frame_id>
+
+        <update_rate>10.0</update_rate>
+        <ray>
+          <scan>
+            <horizontal>
+              <samples>3000</samples>
+              <resolution>1</resolution>
+              <min_angle>-${pi}</min_angle>
+              <max_angle>${pi}</max_angle>
+            </horizontal>
+          </scan>
+          <range>
+            <min>0.05</min>
+            <max>40.0</max>
+            <resolution>0.01</resolution>
+          </range>
+          <noise>
+            <type>gaussian</type>
+            <mean>0.0</mean>
+            <stddev>0.03</stddev>
+          </noise>
+        </ray>
+        <always_on>1</always_on>
+        <visualize>false</visualize>
+        <ros>
+          <namespace>${ns}${device_ns}</namespace>
+        </ros>
+      </sensor>
+    </gazebo>
 
-          <update_rate>10.0</update_rate>
-          <ray>
-            <scan>
-              <horizontal>
-                <samples>3000</samples>
-                <resolution>1</resolution>
-                <min_angle>-${pi}</min_angle>
-                <max_angle>${pi}</max_angle>
-              </horizontal>
-            </scan>
-            <range>
-              <min>0.05</min>
-              <max>40.0</max>
-              <resolution>0.01</resolution>
-            </range>
-            <noise>
-              <type>gaussian</type>
-              <mean>0.0</mean>
-              <stddev>0.03</stddev>
-            </noise>
-          </ray>
-          <always_on>1</always_on>
-          <visualize>false</visualize>
-          <ros>
-            <namespace>${ns}${device_ns}</namespace>
-          </ros>
-        </sensor>
-      </gazebo>
-    </xacro:if>
   </xacro:macro>
 </robot>