Fix :angle-vector-make-trajectory to ignore start-offset-time #365
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.
MERGE AFTER #361 and #363All dependencies are merged.(This PR includes the commits of #361 and #363 )
Include #375
start-offset-time
should only affect time stamp of trajectory, which is set in:send-trajectory
.We don't need to use it in
:angle-vector-make-trajectory
.When sending list of angle-vector having non-zero
start-offset-time
with:angle-vector-motion-plan
, total time of trajectory becomes more thantotal-time
.This bug becomes caught by 69ec3ad.This bug becomes caught by Add test for start-offset-time with avs #375 and this PR is rebased on that.
See commit messages for other information.
Minor but important fix
init-pose
intest-angle-vector-sequence-motion-plan
because usingreset-manip-pose
moves torso a lot and very slowstart-offset-time
in tests