control_msgs icon indicating copy to clipboard operation
control_msgs copied to clipboard

Add indices to FollowJointTrajectory to store trajectory index being executed

Open Levi-Armstrong opened this issue 3 years ago • 11 comments

This is to address issue #65

Open Question: Should it be storing an index for each joint or just a single index for all joints? I believe it is the former.

Levi-Armstrong avatar Jul 27 '22 13:07 Levi-Armstrong

Open Question: Should it be storing an index for each joint or just a single index for all joints? I believe it is the former.

I think it is sufficient to have one index for all joints because the controllers usually take cases about individual points (all joints) they are passing through.

What is the exact purpose of these indices? Do we need a more descriptive field name?

destogl avatar Sep 15 '22 21:09 destogl

Currently we use this for dynamic planning. So based on the environment if we are approaching a collision object we use this index to then look forward some time in the trajectory and replan a new segment and splice it in on the controller side. Another use case is if the tolerance violation is triggered during execution this index is useful for restarting the trajectory.

Levi-Armstrong avatar Sep 16 '22 00:09 Levi-Armstrong

@Levi-Armstrong is this still required? Could you please document the new field please? The current name gives 0 clues to the user as to what they should be expecting those indices to correspond to and why.

bmagyar avatar Apr 01 '23 14:04 bmagyar

@bmagyar

is this still required?

I would say yes. This is the index from the provided trajectory that the controller is executing toward. Though I don't think this needs to be a vector; do you agree?

Could you please document the new field please?

I will update the documentation.

Levi-Armstrong avatar Dec 06 '23 17:12 Levi-Armstrong