-
Notifications
You must be signed in to change notification settings - Fork 368
Add motion_primitives_forward_controller
for interfacing motion primitive messages with hardware interfaces
#1636
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
base: master
Are you sure you want to change the base?
Conversation
…time --> added gitignore, to not push it to github --> remove it after everything is done
… the hardware interface can receive a new motion primitive — replaces the previous, more complex handling via execution_status
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a lot of stuff coming from templates in this PR and you either used some AI tool for coding or you are very good at writing AI-style comments.
Please try removing all commented code or fix them and remove all comments that add nothing to the code itself (which in this code is almost all comments...)
...mitives_forward_controller/include/motion_primitives_forward_controller/visibility_control.h
Outdated
Show resolved
Hide resolved
motion_primitives_forward_controller/motion_primitives_forward_controller.rolling.repos
Outdated
Show resolved
Hide resolved
motion_primitives_forward_controller/test/test_motion_primitives_forward_controller.cpp
Outdated
Show resolved
Hide resolved
...roller/include/motion_primitives_forward_controller/motion_primitives_forward_controller.hpp
Outdated
Show resolved
Hide resolved
…ls according to other controllers in the repo
… rtw template to fit the controller implementation
…tation from rtw template to fit the controller implementation
…ontroller namespace
Thanks @bmagyar for your review! I've addressed and resolved all the comments. Looking forward to any further feedback! |
This PR adds the
motion_primitives_forward_controller
, a controller for forwarding motion primitive commands to motion-primitive-capable hardware interfaces.It was developed alongside the
motion_primitive_ur_driver
and is intended to support future hardware interfaces for additional robot manufacturers.The corresponding PR for integrating the
motion_primitive_ur_driver
into the official Universal Robots ROS 2 driver can be found here: UniversalRobots/Universal_Robots_ROS2_Driver#1341.The controller subscribes to
MotionPrimitive.msg
from theindustrial_robot_motion_interfaces
package. TheMotionPrimitive.msg
has been extended with additional helper types:STOP_MOTION
: Immediately interrupts execution and clears all queued primitives.MOTION_SEQUENCE_START
andMOTION_SEQUENCE_END
: Define a sequence of primitives to be executed as a blended motion block.The controller uses the following interfaces:
Command Interfaces
motion_type
: Type of motion primitive (e.g., LINEAR_JOINT, LINEAR_CARTESIAN, CIRCULAR_CARTESIAN, etc.)q1
–q6
: Target joint positions for joint-based motionpos_x
,pos_y
,pos_z
: Target Cartesian positionpos_qx
,pos_qy
,pos_qz
,pos_qw
: Orientation quaternion of the target posepos_via_x
,pos_via_y
,pos_via_z
: Intermediate via-point position for circular motionpos_via_qx
,pos_via_qy
,pos_via_qz
,pos_via_qw
: Orientation quaternion of via-pointblend_radius
: Blending radius for smooth transitionsvelocity
: Desired motion velocityacceleration
: Desired motion accelerationmove_time
: Optional duration for time-based executionState Interfaces
execution_status
: Indicates the current execution state of the primitive.ready_for_new_primitive
: Boolean flag indicating whether the interface is ready to receive a new motion primitiveI'd appreciate any feedback or suggestions – thanks in advance!