Completing ros-navigation#4259 (ros-navigation#4273)

* changed slam launch file according to the comments for the PR ros-navigation#4211 Signed-off-by: Ibrahim Özcan <ibrahim.ozcan@rwth-aachen.de> * Fixing litning problems --------- Signed-off-by: Ibrahim Özcan <ibrahim.ozcan@rwth-aachen.de> Co-authored-by: Ibrahim Özcan <ibrahim.ozcan@rwth-aachen.de>
Manos-G · Aug 1, 2024 · 3741063 · 3741063
1 parent a3c6eea
commit 3741063
Showing 1 changed file with 22 additions and 15 deletions.
diff --git a/nav2_bringup/launch/slam_launch.py b/nav2_bringup/launch/slam_launch.py
@@ -21,7 +21,7 @@
 from launch.conditions import IfCondition, UnlessCondition
 from launch.launch_description_sources import PythonLaunchDescriptionSource
 from launch.substitutions import LaunchConfiguration
-from launch_ros.actions import Node, SetParameter
+from launch_ros.actions import Node, SetParameter, SetRemap
 from launch_ros.descriptions import ParameterFile
 from nav2_common.launch import HasNodeParams, RewrittenYaml
 
@@ -88,7 +88,6 @@ def generate_launch_description():
     )
 
     # Nodes launching commands
-
     start_map_server = GroupAction(
         actions=[
             SetParameter('use_sim_time', use_sim_time),
@@ -119,19 +118,28 @@ def generate_launch_description():
         source_file=params_file, node_name='slam_toolbox'
     )
 
-    start_slam_toolbox_cmd = IncludeLaunchDescription(
-        PythonLaunchDescriptionSource(slam_launch_file),
-        launch_arguments={'use_sim_time': use_sim_time}.items(),
-        condition=UnlessCondition(has_slam_toolbox_params),
-    )
+    start_slam_toolbox_cmd = GroupAction(
+
+        actions=[
+            # Remapping required to have a slam session subscribe & publish in optional namespaces
+            SetRemap(src='/scan', dst='scan'),
+            SetRemap(src='/tf', dst='tf'),
+            SetRemap(src='/tf_static', dst='tf_static'),
+            SetRemap(src='/map', dst='map'),
+
+            IncludeLaunchDescription(
+                PythonLaunchDescriptionSource(slam_launch_file),
+                launch_arguments={'use_sim_time': use_sim_time}.items(),
+                condition=UnlessCondition(has_slam_toolbox_params),
+            ),
 
-    start_slam_toolbox_cmd_with_params = IncludeLaunchDescription(
-        PythonLaunchDescriptionSource(slam_launch_file),
-        launch_arguments={
-            'use_sim_time': use_sim_time,
-            'slam_params_file': params_file,
-        }.items(),
-        condition=IfCondition(has_slam_toolbox_params),
+            IncludeLaunchDescription(
+                PythonLaunchDescriptionSource(slam_launch_file),
+                launch_arguments={'use_sim_time': use_sim_time,
+                                  'slam_params_file': params_file}.items(),
+                condition=IfCondition(has_slam_toolbox_params),
+            )
+        ]
     )
 
     ld = LaunchDescription()
@@ -149,6 +157,5 @@ def generate_launch_description():
 
     # Running SLAM Toolbox (Only one of them will be run)
     ld.add_action(start_slam_toolbox_cmd)
-    ld.add_action(start_slam_toolbox_cmd_with_params)
 
     return ld