Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extending jtc and adding ctg #15

Open
wants to merge 63 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
63 commits
Select commit Hold shift + click to select a range
3809db7
Update mergify.yml (#974)
christophfroehlich Jan 15, 2024
d5f1eab
[PID] Remove joint_jog include (#975)
christophfroehlich Jan 15, 2024
2f7a755
Bump actions/upload-artifact from 4.0.0 to 4.1.0 (#976)
dependabot[bot] Jan 17, 2024
833ed7f
[JTC] Convert lambda to class functions (#945)
christophfroehlich Jan 22, 2024
d696767
Bump actions/upload-artifact from 4.1.0 to 4.2.0 (#984)
dependabot[bot] Jan 23, 2024
3ab3759
Update mergify.yml (#982)
christophfroehlich Jan 23, 2024
0bcb77b
[CI] debian + RHEL updates (#985)
christophfroehlich Jan 25, 2024
9f7e9e9
Revert "[JTC] Remove read_only from 'joints', 'state_interfaces' and …
Noel215 Jan 25, 2024
7fee940
Revert "[ForceTorqueSensorBroadcaster] Create ParamListener and get p…
saikishor Jan 29, 2024
8d732f1
[tricycle_controller] Use generate_parameter_library (#957)
christophfroehlich Jan 29, 2024
ab68351
Bump ros-tooling/action-ros-ci from 0.3.5 to 0.3.6 (#994)
dependabot[bot] Jan 30, 2024
d375e43
Bump codecov/codecov-action from 3.1.4 to 3.1.5 (#993)
dependabot[bot] Jan 30, 2024
021ccab
Bump actions/upload-artifact from 4.2.0 to 4.3.0 (#992)
dependabot[bot] Jan 30, 2024
99fadce
[JTC] Invalidate empty trajectory messages (#902)
christophfroehlich Jan 30, 2024
f375c69
Let sphinx add parameter description with nested structures to docume…
christophfroehlich Jan 31, 2024
f2f36c0
Add tests for `interface_configuration_type` consistently (#899)
christophfroehlich Jan 31, 2024
2705ca8
[diff_drive] Remove unused parameter and add simple validation #abi-b…
christophfroehlich Jan 31, 2024
67dbf7b
[JTC] Fill action error_strings (#887)
christophfroehlich Jan 31, 2024
08ea6cf
Update changelogs
bmagyar Jan 31, 2024
fd8b91f
4.5.0
bmagyar Jan 31, 2024
2817f27
Fix tests for using new `get_node_options` API (#840)
saikishor Feb 1, 2024
737a45b
Use correct ref for scheduled workflows (#1013)
christophfroehlich Feb 5, 2024
8bf379b
Add test_depend on `hardware_interface_testing` (#1018)
christophfroehlich Feb 5, 2024
c786604
Add test_depend on `hardware_interface_testing` also for diff_drive (…
christophfroehlich Feb 5, 2024
5cde4fc
Bump codecov/codecov-action from 3.1.5 to 4.0.1 (#1023)
dependabot[bot] Feb 6, 2024
89109a0
Bump actions/upload-artifact from 4.3.0 to 4.3.1 (#1024)
dependabot[bot] Feb 6, 2024
1266941
[CI] Improvements and Cleanups (#1028)
christophfroehlich Feb 9, 2024
e1647c7
Make some methods virtual for extending of JTC. Extend caculation of …
destogl Dec 4, 2023
7dfd7ea
Add structures for joint limits in the controller.
destogl Dec 5, 2023
c0a38ac
Added Cartesian Trajectory Generator (CTG).
destogl Dec 5, 2023
133de37
Add reseting of individual DoFs to state values in JTC when using in …
destogl Apr 6, 2023
45443c5
Fix missing .cpp extension in CMakeLists for CTG
bijoua29 Apr 7, 2023
484e0f6
Added structures for joint limits.
destogl Jul 25, 2022
94c4c2c
Integrate SimpleJointLimiter in to JTC.
destogl May 4, 2023
79039e2
Adjust default value of limiters.
destogl May 4, 2023
59c542f
JTC extending for overriding.
destogl Dec 6, 2023
3aaf388
Adjust method names to the JTC updates and manage errors in feedback …
destogl Dec 6, 2023
27dcfb8
Remove unused variable
bijoua29 May 8, 2023
fdb6e35
Remove deprecated rclcpp/qos_event.hpp
bijoua29 May 12, 2023
5ba276e
Remove unused free functions for resetting controlled feedback messages
bijoua29 May 12, 2023
b90c3e3
Use time from start passed in from input reference if non-zero in CTG
bijoua29 Aug 30, 2023
e6fe00b
Modify CTG to expect twist in measured state input to be in body frame
bijoua29 Sep 5, 2023
177a242
Handle NaNs in measured state in CTG
bijoua29 Sep 12, 2023
e178549
Update JTC to be more extensible.
destogl Dec 7, 2023
2dfedc8
Updated CTG to have custom state message.
destogl Dec 7, 2023
9119f51
Fix stair step in JTC when new trajectory message received
bijoua29 Sep 25, 2023
e506ba6
Reset initial value of last commanded time to 0.
destogl Sep 26, 2023
72874be
Use previous state for limiters.
destogl Sep 28, 2023
3f10cd1
Use values, if present in reset dofs service, instead of fb to reset
bijoua29 Oct 18, 2023
7b2b089
Fix bug when two reset dofs req sent without being processed in between
bijoua29 Oct 20, 2023
e4054ad
Update reset dofs output messages in JTC
bijoua29 Oct 20, 2023
e5edfe5
Remove service for setting joint limits in CTG.
destogl Nov 1, 2023
d2717a3
Cleanup rucking code and add simple joint limiter.
destogl Nov 16, 2023
e31691f
Fixup JTC and Trajectory to compile.
destogl Dec 7, 2023
753d83b
Fixup CTG to compile.
destogl Dec 7, 2023
2350810
Added support for save integral term parameter
Whalex451 Apr 20, 2023
f7d7466
Fix JTC crashes
destogl Dec 18, 2023
b203869
Fixup one crash of JTC.
destogl Dec 18, 2023
cc5fc7a
Move resetting of the axis values from lambda to a method.
destogl Jan 31, 2024
1ed60eb
Adjust variable name to avoid shadowing.
destogl Jan 31, 2024
54b195f
Bugfixing.
destogl Feb 1, 2024
34efa25
Increasing code readability in JTC
destogl Feb 9, 2024
1ca51d1
Adjusting CTG with limiters.
destogl Feb 9, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/mergify.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,19 +32,19 @@ pull_request_rules:
- author=mergify[bot]
actions:
comment:
message: This pull request is in conflict. Could you fix it @bmagyar @dstogl @christophfroehlich?
message: This pull request is in conflict. Could you fix it @bmagyar @destogl @christophfroehlich?

- name: development targets master branch
conditions:
- base!=master
- author!=bmagyar
- author!=dstogl
- author!=destogl
- author!=christophfroehlich
- author!=mergify[bot]
- author!=dependabot[bot]
actions:
comment:
message: |
@{{author}}, all pull requests must be targeted towards the `master` development branch.
Once merged into `master`, it is possible to backport to @{{base}}, but it must be in `master`
Once merged into `master`, it is possible to backport to `{{base}}`, but it must be in `master`
to have these changes reflected into new distributions.
6 changes: 3 additions & 3 deletions .github/workflows/ci-coverage-build-humble.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
with:
required-ros-distributions: ${{ env.ROS_DISTRO }}
- uses: actions/checkout@v4
- uses: ros-tooling/[email protected].5
- uses: ros-tooling/[email protected].6
with:
target-ros2-distro: ${{ env.ROS_DISTRO }}
import-token: ${{ secrets.GITHUB_TOKEN }}
Expand Down Expand Up @@ -55,12 +55,12 @@ jobs:
}
}
colcon-mixin-repository: https://raw.githubusercontent.com/colcon/colcon-mixin-repository/master/index.yaml
- uses: codecov/codecov-action@v3.1.4
- uses: codecov/codecov-action@v4.0.1
with:
file: ros_ws/lcov/total_coverage.info
flags: unittests
name: codecov-umbrella
- uses: actions/upload-artifact@v4.0.0
- uses: actions/upload-artifact@v4.3.1
with:
name: colcon-logs-coverage-humble
path: ros_ws/log
6 changes: 3 additions & 3 deletions .github/workflows/ci-coverage-build-iron.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
with:
required-ros-distributions: ${{ env.ROS_DISTRO }}
- uses: actions/checkout@v4
- uses: ros-tooling/[email protected].5
- uses: ros-tooling/[email protected].6
with:
target-ros2-distro: ${{ env.ROS_DISTRO }}
import-token: ${{ secrets.GITHUB_TOKEN }}
Expand Down Expand Up @@ -55,12 +55,12 @@ jobs:
}
}
colcon-mixin-repository: https://raw.githubusercontent.com/colcon/colcon-mixin-repository/master/index.yaml
- uses: codecov/codecov-action@v3.1.4
- uses: codecov/codecov-action@v4.0.1
with:
file: ros_ws/lcov/total_coverage.info
flags: unittests
name: codecov-umbrella
- uses: actions/upload-artifact@v4.0.0
- uses: actions/upload-artifact@v4.3.1
with:
name: colcon-logs-coverage-iron
path: ros_ws/log
6 changes: 3 additions & 3 deletions .github/workflows/ci-coverage-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ jobs:
with:
required-ros-distributions: ${{ env.ROS_DISTRO }}
- uses: actions/checkout@v4
- uses: ros-tooling/[email protected].5
- uses: ros-tooling/[email protected].6
with:
target-ros2-distro: ${{ env.ROS_DISTRO }}
import-token: ${{ secrets.GITHUB_TOKEN }}
Expand Down Expand Up @@ -55,12 +55,12 @@ jobs:
}
}
colcon-mixin-repository: https://raw.githubusercontent.com/colcon/colcon-mixin-repository/master/index.yaml
- uses: codecov/codecov-action@v3.1.4
- uses: codecov/codecov-action@v4.0.1
with:
file: ros_ws/lcov/total_coverage.info
flags: unittests
name: codecov-umbrella
- uses: actions/upload-artifact@v4.0.0
- uses: actions/upload-artifact@v4.3.1
with:
name: colcon-logs-coverage-rolling
path: ros_ws/log
70 changes: 26 additions & 44 deletions .github/workflows/ci-ros-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,30 @@ name: ROS Lint
on:
pull_request:

env:
package-name:
ackermann_steering_controller
admittance_controller
bicycle_steering_controller
diff_drive_controller
effort_controllers
force_torque_sensor_broadcaster
forward_command_controller
gripper_controllers
imu_sensor_broadcaster
joint_state_broadcaster
joint_trajectory_controller
pid_controller
position_controllers
range_sensor_broadcaster
ros2_controllers
ros2_controllers_test_nodes
rqt_joint_trajectory_controller
steering_controllers_library
tricycle_controller
tricycle_steering_controller
velocity_controllers

jobs:
ament_lint:
name: ament_${{ matrix.linter }}
Expand All @@ -19,28 +43,7 @@ jobs:
with:
distribution: rolling
linter: ${{ matrix.linter }}
package-name:
ackermann_steering_controller
admittance_controller
bicycle_steering_controller
diff_drive_controller
effort_controllers
force_torque_sensor_broadcaster
forward_command_controller
gripper_controllers
imu_sensor_broadcaster
joint_state_broadcaster
joint_trajectory_controller
pid_controller
position_controllers
range_sensor_broadcaster
ros2_controllers
ros2_controllers_test_nodes
rqt_joint_trajectory_controller
steering_controllers_library
tricycle_controller
tricycle_steering_controller
velocity_controllers
package-name: ${{ env.package-name }}


ament_lint_100:
Expand All @@ -58,25 +61,4 @@ jobs:
distribution: rolling
linter: cpplint
arguments: "--linelength=100 --filter=-whitespace/newline"
package-name:
ackermann_steering_controller
admittance_controller
bicycle_steering_controller
diff_drive_controller
effort_controllers
force_torque_sensor_broadcaster
forward_command_controller
gripper_controllers
imu_sensor_broadcaster
joint_state_broadcaster
joint_trajectory_controller
pid_controller
position_controllers
range_sensor_broadcaster
ros2_controllers
ros2_controllers_test_nodes
rqt_joint_trajectory_controller
steering_controllers_library
tricycle_controller
tricycle_steering_controller
velocity_controllers
package-name: ${{ env.package-name }}
2 changes: 0 additions & 2 deletions .github/workflows/humble-binary-build-main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ name: Humble Binary Build - main

on:
workflow_dispatch:
branches:
- humble
pull_request:
branches:
- humble
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/humble-binary-build-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ name: Humble Binary Build - testing

on:
workflow_dispatch:
branches:
- humble
pull_request:
branches:
- humble
Expand Down
20 changes: 20 additions & 0 deletions .github/workflows/humble-debian-build.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
name: Debian Humble Source Build
on:
workflow_dispatch:
pull_request:
branches:
- humble
schedule:
# Run every day to detect flakiness and broken dependencies
- cron: '03 1 * * *'


jobs:
humble_debian:
name: Humble debian build
uses: ./.github/workflows/reusable-debian-build.yml
with:
ros_distro: humble
upstream_workspace: ros2_controllers.humble.repos
ref_for_scheduled_build: humble
skip_packages: rqt_joint_trajectory_controller
28 changes: 7 additions & 21 deletions .github/workflows/humble-rhel-binary-build.yml
Original file line number Diff line number Diff line change
@@ -1,33 +1,19 @@
name: Humble RHEL Binary Build
name: RHEL Humble Binary Build
on:
workflow_dispatch:
branches:
- humble
pull_request:
branches:
- humble
push:
branches:
- humble
schedule:
# Run every day to detect flakiness and broken dependencies
- cron: '03 1 * * *'


jobs:
humble_rhel_binary:
name: Humble RHEL binary build
runs-on: ubuntu-latest
env:
ROS_DISTRO: humble
container: ghcr.io/ros-controls/ros:humble-rhel
steps:
- uses: actions/checkout@v4
with:
path: src/ros2_controllers
- run: |
rosdep update
rosdep install -iy --from-path src/ros2_controllers
source /opt/ros/${{ env.ROS_DISTRO }}/setup.bash
colcon build
colcon test
uses: ./.github/workflows/reusable-rhel-binary-build.yml
with:
ros_distro: humble
upstream_workspace: ros2_controllers.humble.repos
ref_for_scheduled_build: humble
skip_packages: rqt_joint_trajectory_controller
2 changes: 0 additions & 2 deletions .github/workflows/humble-semi-binary-build-main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@ name: Humble Semi-Binary Build - main

on:
workflow_dispatch:
branches:
- humble
pull_request:
branches:
- humble
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/humble-semi-binary-build-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,6 @@ name: Humble Semi-Binary Build - testing

on:
workflow_dispatch:
branches:
- humble
pull_request:
branches:
- humble
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/humble-source-build.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
name: Humble Source Build
on:
workflow_dispatch:
branches:
- humble
push:
branches:
- humble
Expand Down
4 changes: 0 additions & 4 deletions .github/workflows/iron-binary-build-main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@ name: Iron Binary Build - main

on:
workflow_dispatch:
branches:
- iron
- '*feature*'
- '*feature/**'
pull_request:
branches:
- iron
Expand Down
4 changes: 0 additions & 4 deletions .github/workflows/iron-binary-build-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@ name: Iron Binary Build - testing

on:
workflow_dispatch:
branches:
- iron
- '*feature*'
- '*feature/**'
pull_request:
branches:
- iron
Expand Down
20 changes: 20 additions & 0 deletions .github/workflows/iron-debian-build.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
name: Debian Iron Source Build
on:
workflow_dispatch:
pull_request:
branches:
- iron
schedule:
# Run every day to detect flakiness and broken dependencies
- cron: '03 1 * * *'


jobs:
iron_debian:
name: Iron debian build
uses: ./.github/workflows/reusable-debian-build.yml
with:
ros_distro: iron
upstream_workspace: ros2_controllers.iron.repos
ref_for_scheduled_build: iron
skip_packages: rqt_joint_trajectory_controller
24 changes: 8 additions & 16 deletions .github/workflows/iron-rhel-binary-build.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name: Iron RHEL Binary Build
name: RHEL Iron Binary Build
on:
workflow_dispatch:
push:
pull_request:
branches:
- iron
schedule:
Expand All @@ -12,17 +12,9 @@ on:
jobs:
iron_rhel_binary:
name: Iron RHEL binary build
runs-on: ubuntu-latest
env:
ROS_DISTRO: iron
container: ghcr.io/ros-controls/ros:iron-rhel
steps:
- uses: actions/checkout@v4
with:
path: src/ros2_controllers
- run: |
rosdep update
rosdep install -iy --from-path src/ros2_controllers
source /opt/ros/${{ env.ROS_DISTRO }}/setup.bash
colcon build
colcon test
uses: ./.github/workflows/reusable-rhel-binary-build.yml
with:
ros_distro: iron
upstream_workspace: ros2_controllers.iron.repos
ref_for_scheduled_build: iron
skip_packages: rqt_joint_trajectory_controller
4 changes: 0 additions & 4 deletions .github/workflows/iron-semi-binary-build-main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,6 @@ name: Iron Semi-Binary Build - main

on:
workflow_dispatch:
branches:
- iron
- '*feature*'
- '*feature/**'
pull_request:
branches:
- iron
Expand Down
4 changes: 0 additions & 4 deletions .github/workflows/iron-semi-binary-build-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,6 @@ name: Iron Semi-Binary Build - testing

on:
workflow_dispatch:
branches:
- iron
- '*feature*'
- '*feature/**'
pull_request:
branches:
- iron
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/iron-source-build.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
name: Iron Source Build
on:
workflow_dispatch:
branches:
- iron
push:
branches:
- iron
Expand Down
Loading
Loading