Add allowed_start_tolerance_joints parameter (Port moveit#3287) (backport #3309) #3328
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Allow to specify a per joint allowed start tolerance for the TrajectoryExecutionManager. Port from moveit/moveit#3287 with some changes to be able to dynamically update its values at runtime.
Example of usage
In
moveit_controllers.yaml
:Joints that are not declared in
allowed_start_tolerance_joints
will useallowed_start_tolerance
as the default.Some remarks
I would personally have named this parameter "allowed_start_tolerance_joints" to be next to "allowed_start_tolerance" when sorted alphabetically. However, to maintain continuity with the original PR, I kept it as is.test_execution_manager.cpp
is not used at the moment, right?allowed_start_tolerance
, it would be more intuitive to use the absolute value in that case.Checklist
This is an automatic backport of pull request #3309 done by [Mergify](https://mergify.com).