Skip to content

Conversation

urfeex
Copy link
Contributor

@urfeex urfeex commented Jul 7, 2025

These messages should reflect the motion primitives supported by most industrial robots.They are meant to being extended in the future in order to support a larger variety of motion implementations.

The motivation is to support executing robot motions as they are traditionally programmed: A series of primitives such as LIN, PTP, CIRC.

This PR should support ros-controls/ros2_controllers#1636

These messages should reflect the motion primitives supported by most industrial
robots.They are meant to being extended in the future in order to support
a larger variety of motion implementations.

The motivation is to support executing robot motions as they are
traditionally programmed: A series of primitives such as `LIN`,
`PTP`, `CIRC`.
mathias31415 added a commit to b-robotized-forks/ros2_controllers that referenced this pull request Jul 8, 2025
mathias31415 added a commit to b-robotized-forks/Universal_Robots_ROS2_Driver_MotionPrimitive that referenced this pull request Jul 8, 2025
bmagyar
bmagyar previously approved these changes Jul 17, 2025
Copy link
Member

@bmagyar bmagyar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great start, I only have that one question re. documenting a teeeeny bit more

Copy link
Contributor

@christophfroehlich christophfroehlich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@saikishor saikishor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Co-authored-by: Sai Kishor Kothakota <[email protected]>
@christophfroehlich christophfroehlich merged commit 16742c2 into ros-controls:master Jul 19, 2025
10 checks passed
@saikishor saikishor added the backport-jazzy Triggers PR backport to ROS 2 jazzy. label Jul 19, 2025
mergify bot pushed a commit that referenced this pull request Jul 19, 2025
saikishor pushed a commit that referenced this pull request Jul 19, 2025
(cherry picked from commit 16742c2)

Co-authored-by: Felix Exner <[email protected]>
@urfeex urfeex deleted the motion_primitives branch July 21, 2025 08:08
mathias31415 added a commit to b-robotized-forks/Universal_Robots_ROS2_Driver_MotionPrimitive that referenced this pull request Jul 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-jazzy Triggers PR backport to ROS 2 jazzy.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants