JafarAbdi / moveit2

:robot: MoveIt for ROS 2.0, Alpha
https://moveit.ros.org/
BSD 3-Clause "New" or "Revised" License
1 stars 2 forks source link

Enforce joint position limits on internal joint state before making trajectory #13

Closed AdamPettinger closed 3 years ago

AdamPettinger commented 3 years ago

Description

This switchs the order that composeJointTrajMessage and enforcePositionLimits is called so that enforcePositionLimits works on a JointState and uses internal_jointstate which holds position and velocity information regardless of what should be published. Previously: it was operating on the JointTrajectory and had to take publishing booleans into consideration

Tested locally, including being able to back away from joint position limit. Shameless plug for new Servo tutorial which includes using a keyboard as input (how I tested the joint limits)