Merge branch 'master' into change_interface_export_variant

.github/workflows/humble-abi-compatibility.yml

@@ -8,8 +8,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/humble-abi-compatibility.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/humble-abi-compatibility.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers-not-released.humble.repos'

.github/workflows/humble-binary-build.yml

@@ -11,8 +11,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/humble-binary-build.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/humble-binary-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers-not-released.humble.repos'
@@ -23,8 +24,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/humble-binary-build.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/humble-binary-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers-not-released.humble.repos'

.github/workflows/humble-coverage-build.yml

@@ -8,9 +8,10 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
+      - '**.yaml'
       - '.github/workflows/humble-coverage-build.yml'
       - 'codecov.yml'
-      - '**.yaml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers.humble.repos'
@@ -21,9 +22,10 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
+      - '**.yaml'
       - '.github/workflows/humble-coverage-build.yml'
       - 'codecov.yml'
-      - '**.yaml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers.humble.repos'

.github/workflows/humble-debian-build.yml

@@ -8,8 +8,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/humble-debian-build.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/humble-debian-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers.humble.repos'

.github/workflows/humble-rhel-semi-binary-build.yml

@@ -8,8 +8,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/humble-rhel-semi-binary-build.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/humble-rhel-semi-binary-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers.humble.repos'

.github/workflows/humble-semi-binary-build.yml

@@ -10,8 +10,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/humble-semi-binary-build.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/humble-semi-binary-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers.humble.repos'
@@ -22,8 +23,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/humble-semi-binary-build.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/humble-semi-binary-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers.humble.repos'

.github/workflows/humble-source-build.yml

@@ -8,6 +8,8 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
+      - '**.yaml'
       - '.github/workflows/humble-source-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'

.github/workflows/iron-binary-build.yml

@@ -13,8 +13,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/iron-binary-build.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/iron-binary-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers-not-released.iron.repos'
@@ -25,8 +26,9 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
-      - '.github/workflows/iron-binary-build.yml'
+      - '**.py'
       - '**.yaml'
+      - '.github/workflows/iron-binary-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
       - 'ros2_controllers-not-released.iron.repos'

.github/workflows/iron-coverage-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/iron-coverage-build.yml'
       - 'codecov.yml'
@@ -21,6 +22,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/iron-coverage-build.yml'
       - 'codecov.yml'

.github/workflows/iron-debian-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/iron-debian-build.yml'
       - '**/package.xml'

.github/workflows/iron-semi-binary-build.yml

@@ -12,6 +12,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/iron-semi-binary-build.yml'
       - '**/package.xml'

.github/workflows/iron-source-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/iron-source-build.yml'
       - '**/package.xml'

.github/workflows/jazzy-binary-build.yml

@@ -13,6 +13,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/jazzy-binary-build.yml'
       - '**/package.xml'

.github/workflows/jazzy-debian-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/jazzy-debian-build.yml'
       - '**/package.xml'

.github/workflows/jazzy-rhel-semi-binary-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/jazzy-rhel-semi-binary-build.yml'
       - '**/package.xml'

.github/workflows/jazzy-semi-binary-build.yml

@@ -12,6 +12,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/jazzy-semi-binary-build.yml'
       - '**/package.xml'
@@ -24,6 +25,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/jazzy-semi-binary-build.yml'
       - '**/package.xml'

.github/workflows/jazzy-source-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/jazzy-source-build.yml'
       - '**/package.xml'

.github/workflows/rolling-binary-build.yml

@@ -13,6 +13,8 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
+      - '**.yaml'
       - '.github/workflows/rolling-binary-build.yml'
       - '**/package.xml'
       - '**/CMakeLists.txt'
@@ -24,6 +26,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/rolling-binary-build.yml'
       - '**/package.xml'

.github/workflows/rolling-coverage-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/rolling-coverage-build.yml'
       - 'codecov.yml'
@@ -21,6 +22,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/rolling-coverage-build.yml'
       - 'codecov.yml'

.github/workflows/rolling-debian-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/rolling-debian-build.yml'
       - '**/package.xml'

.github/workflows/rolling-rhel-semi-binary-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/rolling-rhel-semi-binary-build.yml'
       - '**/package.xml'

.github/workflows/rolling-semi-binary-build.yml

@@ -12,6 +12,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/rolling-semi-binary-build.yml'
       - '**/package.xml'
@@ -24,6 +25,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/rolling-semi-binary-build.yml'
       - '**/package.xml'

.github/workflows/rolling-source-build.yml

@@ -8,6 +8,7 @@ on:
       - '**.hpp'
       - '**.h'
       - '**.cpp'
+      - '**.py'
       - '**.yaml'
       - '.github/workflows/rolling-source-build.yml'
       - '**/package.xml'

.pre-commit-config.yaml

@@ -50,13 +50,13 @@ repos:
       args: ["--ignore=D100,D101,D102,D103,D104,D105,D106,D107,D203,D212,D404"]
 
   - repo: https://github.com/psf/black
-    rev: 24.4.2
+    rev: 24.8.0
     hooks:
       - id: black
         args: ["--line-length=99"]
 
   - repo: https://github.com/pycqa/flake8
-    rev: 7.1.0
+    rev: 7.1.1
     hooks:
     - id: flake8
       args: ["--extend-ignore=E501"]
@@ -133,7 +133,7 @@ repos:
         exclude: CHANGELOG\.rst|\.(svg|pyc|drawio)$
 
   - repo: https://github.com/python-jsonschema/check-jsonschema
-    rev: 0.29.1
+    rev: 0.29.2
     hooks:
       - id: check-github-workflows
         args: ["--verbose"]

ackermann_steering_controller/CHANGELOG.rst

@@ -2,6 +2,17 @@
 Changelog for package ackermann_steering_controller
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
+4.14.0 (2024-09-11)
+-------------------
+
+4.13.0 (2024-08-22)
+-------------------
+* Fixes tests to work with use_global_arguments NodeOptions parameter  (`#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>`_)
+* Contributors: Sai Kishor Kothakota
+
+4.12.1 (2024-08-14)
+-------------------
+
 4.12.0 (2024-07-23)
 -------------------
 * Add missing includes (`#1226 <https://github.com/ros-controls/ros2_controllers/issues/1226>`_)

ackermann_steering_controller/CMakeLists.txt

@@ -58,10 +58,8 @@ if(BUILD_TESTING)
   find_package(ros2_control_test_assets REQUIRED)
 
 
-  add_rostest_with_parameters_gmock(test_load_ackermann_steering_controller
-    test/test_load_ackermann_steering_controller.cpp
-    ${CMAKE_CURRENT_SOURCE_DIR}/test/ackermann_steering_controller_params.yaml
-  )
+  add_definitions(-DTEST_FILES_DIRECTORY="${CMAKE_CURRENT_SOURCE_DIR}/test")
+  ament_add_gmock(test_load_ackermann_steering_controller test/test_load_ackermann_steering_controller.cpp)
   ament_target_dependencies(test_load_ackermann_steering_controller
     controller_manager
     hardware_interface

ackermann_steering_controller/package.xml

@@ -2,7 +2,7 @@
 <?xml-model href="http://download.ros.org/schema/package_format3.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
 <package format="3">
   <name>ackermann_steering_controller</name>
-  <version>4.12.0</version>
+  <version>4.14.0</version>
   <description>Steering controller for Ackermann kinematics. Rear fixed wheels are powering the vehicle and front wheels are steering it.</description>
   <license>Apache License 2.0</license>
   <maintainer email="[email protected]">Bence Magyar</maintainer>

ackermann_steering_controller/test/test_load_ackermann_steering_controller.cpp

@@ -29,12 +29,15 @@ TEST(TestLoadAckermannSteeringController, load_controller)
 
   controller_manager::ControllerManager cm(
     executor, ros2_control_test_assets::minimal_robot_urdf, true, "test_controller_manager");
+  const std::string test_file_path =
+    std::string(TEST_FILES_DIRECTORY) + "/ackermann_steering_controller_params.yaml";
 
-  ASSERT_NE(
-    cm.load_controller(
-      "test_ackermann_steering_controller",
-      "ackermann_steering_controller/AckermannSteeringController"),
-    nullptr);
+  cm.set_parameter({"test_ackermann_steering_controller.params_file", test_file_path});
+  cm.set_parameter(
+    {"test_ackermann_steering_controller.type",
+     "ackermann_steering_controller/AckermannSteeringController"});
+
+  ASSERT_NE(cm.load_controller("test_ackermann_steering_controller"), nullptr);
 }
 
 int main(int argc, char ** argv)

admittance_controller/CHANGELOG.rst

@@ -2,6 +2,20 @@
 Changelog for package admittance_controller
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
+4.14.0 (2024-09-11)
+-------------------
+
+4.13.0 (2024-08-22)
+-------------------
+* Fix segfault at reconfigure of AdmittanceController (`#1248 <https://github.com/ros-controls/ros2_controllers/issues/1248>`_)
+* Fixes tests to work with use_global_arguments NodeOptions parameter  (`#1256 <https://github.com/ros-controls/ros2_controllers/issues/1256>`_)
+* Contributors: Lennart Nachtigall, Sai Kishor Kothakota
+
+4.12.1 (2024-08-14)
+-------------------
+* Fix admittance controller interface read/write logic (`#1232 <https://github.com/ros-controls/ros2_controllers/issues/1232>`_)
+* Contributors: Nikola Banović
+
 4.12.0 (2024-07-23)
 -------------------
 * Change the subscription timeout in the tests to 5ms (`#1219 <https://github.com/ros-controls/ros2_controllers/issues/1219>`_)