removed ros2_control configs from lbr_description

lbr-stack · Dec 5, 2023 · 5494616 · 5494616
1 parent 44a341b
commit 5494616
Show file tree

Hide file tree

Showing 12 changed files with 11 additions and 11 deletions.
diff --git a/lbr_bringup/doc/lbr_bringup.rst b/lbr_bringup/doc/lbr_bringup.rst
@@ -57,7 +57,7 @@ and select:
 
 Make sure that the ``update_rate`` in `lbr_controllers.yaml <https://github.com/lbr-stack/lbr_fri_ros2_stack/tree/humble/lbr_ros2_control/config/lbr_controllers.yaml>`_ is greater or equal ``100`` (``FRI send period``).
 
-For using other ``FRI send period``, also change the ``sample_time`` in the `lbr.ros2_control.xacro <https://github.com/lbr-stack/lbr_fri_ros2_stack/blob/humble/lbr_description/ros2_control/lbr.ros2_control.xacro>`_ (automated in the future).
+For using other ``FRI send period``, also change the ``sample_time`` in the `lbr_system_interface.xacro <https://github.com/lbr-stack/lbr_fri_ros2_stack/blob/humble/lbr_ros2_control/config/lbr_system_interface.xacro>`_ (automated in the future).
 
 Standalone Launch
 -----------------

diff --git a/lbr_description/CMakeLists.txt b/lbr_description/CMakeLists.txt
@@ -12,7 +12,7 @@ find_package(ament_cmake_python REQUIRED)
 
 # install
 install(
-  DIRECTORY config gazebo launch meshes ros2_control urdf
+  DIRECTORY config gazebo launch meshes urdf
   DESTINATION share/${PROJECT_NAME}
 )
 

diff --git a/lbr_description/urdf/iiwa14/iiwa14_description.urdf.xacro b/lbr_description/urdf/iiwa14/iiwa14_description.urdf.xacro
@@ -24,7 +24,7 @@
 
     <!-- includes -->
     <xacro:include filename="$(find lbr_description)/gazebo/lbr.gazebo.xacro" />
-    <xacro:include filename="$(find lbr_description)/ros2_control/lbr.ros2_control.xacro" />
+    <xacro:include filename="$(find lbr_ros2_control)/config/lbr_system_interface.xacro" />
 
     <!-- robot as extracted via https://github.com/syuntoku14/fusion2urdf -->
     <xacro:macro name="iiwa14"

diff --git a/lbr_description/urdf/iiwa7/iiwa7_description.urdf.xacro b/lbr_description/urdf/iiwa7/iiwa7_description.urdf.xacro
@@ -24,7 +24,7 @@
 
     <!-- includes -->
     <xacro:include filename="$(find lbr_description)/gazebo/lbr.gazebo.xacro" />
-    <xacro:include filename="$(find lbr_description)/ros2_control/lbr.ros2_control.xacro" />
+    <xacro:include filename="$(find lbr_ros2_control)/config/lbr_system_interface.xacro" />
 
     <!-- robot as extracted via https://github.com/syuntoku14/fusion2urdf -->
     <xacro:macro name="iiwa7"

diff --git a/lbr_description/urdf/med14/med14_description.urdf.xacro b/lbr_description/urdf/med14/med14_description.urdf.xacro
@@ -24,7 +24,7 @@
 
     <!-- includes -->
     <xacro:include filename="$(find lbr_description)/gazebo/lbr.gazebo.xacro" />
-    <xacro:include filename="$(find lbr_description)/ros2_control/lbr.ros2_control.xacro" />
+    <xacro:include filename="$(find lbr_ros2_control)/config/lbr_system_interface.xacro" />
 
     <!-- robot as extracted via https://github.com/syuntoku14/fusion2urdf -->
     <xacro:macro name="med14"

diff --git a/lbr_description/urdf/med7/med7_description.urdf.xacro b/lbr_description/urdf/med7/med7_description.urdf.xacro
@@ -24,7 +24,7 @@
 
     <!-- includes -->
     <xacro:include filename="$(find lbr_description)/gazebo/lbr.gazebo.xacro" />
-    <xacro:include filename="$(find lbr_description)/ros2_control/lbr.ros2_control.xacro" />
+    <xacro:include filename="$(find lbr_ros2_control)/config/lbr_system_interface.xacro" />
 
     <!-- robot as extracted via https://github.com/syuntoku14/fusion2urdf -->
     <xacro:macro name="med7"

diff --git a/lbr_ros2_control/CMakeLists.txt b/lbr_ros2_control/CMakeLists.txt
@@ -72,9 +72,9 @@ target_link_libraries(${PROJECT_NAME}
   FRIClient::FRIClient
 )
 
-pluginlib_export_plugin_description_file(controller_interface broadcasters.xml)
-pluginlib_export_plugin_description_file(controller_interface forward_command_controllers.xml)
-pluginlib_export_plugin_description_file(hardware_interface system_interface.xml)
+pluginlib_export_plugin_description_file(controller_interface plugin_description_files/broadcasters.xml)
+pluginlib_export_plugin_description_file(controller_interface plugin_description_files/forward_command_controllers.xml)
+pluginlib_export_plugin_description_file(hardware_interface plugin_description_files/system_interface.xml)
 
 # Export for downstream usage, see https://docs.ros.org/en/foxy/Guides/Ament-CMake-Documentation.html
 ament_export_targets(

diff --git a/...ption/ros2_control/lbr.ros2_control.xacro → ...control/config/lbr_system_interface.xacro b/...ption/ros2_control/lbr.ros2_control.xacro → ...control/config/lbr_system_interface.xacro
diff --git a/lbr_ros2_control/broadcasters.xml → ...plugin_description_files/broadcasters.xml b/lbr_ros2_control/broadcasters.xml → ...plugin_description_files/broadcasters.xml
diff --git a/...2_control/forward_command_controllers.xml → ...ion_files/forward_command_controllers.xml b/...2_control/forward_command_controllers.xml → ...ion_files/forward_command_controllers.xml
diff --git a/lbr_ros2_control/system_interface.xml → ...in_description_files/system_interface.xml b/lbr_ros2_control/system_interface.xml → ...in_description_files/system_interface.xml
diff --git a/lbr_ros2_control/src/system_interface.cpp b/lbr_ros2_control/src/system_interface.cpp
@@ -9,7 +9,7 @@ SystemInterface::on_init(const hardware_interface::HardwareInfo &system_info) {
     return ret;
   }
 
-  // parameters from lbr.ros2_control.xacro
+  // parameters from config/lbr_system_interface.xacro
   port_id_ = std::stoul(info_.hardware_parameters["port_id"]);
   info_.hardware_parameters["remote_host"] == "INADDR_ANY"
       ? remote_host_ = NULL
@@ -346,7 +346,7 @@ bool SystemInterface::verify_sensors_() {
     return false;
   }
 
-  // check all interfaces are defined in lbr.ros2_control.xacro
+  // check all interfaces are defined in config/lbr_system_interface.xacro
   const auto &lbr_fri_sensor = info_.sensors[0];
   if (lbr_fri_sensor.state_interfaces.size() != LBR_FRI_SENSOR_SIZE) {
     RCLCPP_ERROR(app_node_ptr_->get_logger(),