-
Notifications
You must be signed in to change notification settings - Fork 938
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
fixed REALTIME planned path trajectory playback #1683
Conversation
Thanks for helping in improving MoveIt |
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.
Thanks a lot for your contribution! The fix works for me. I don't think we need any tests for this. I just noticed that the trajectory slider somehow doesn't work nicely together with "Loop Animation" enabled, but this is unrelated to this PR.
@henningkayser what problems are you seeing with loop animation enabled? I just ran a few simple tests and don't see any issues. The tool was able to loop the entire animation and was also able to pickup looping when starting from any point in the trajectory. If I know what to look for I might have some time to fix it. |
@MarqRazz actually, I expected the TrajectorySlider to have the same behavior as the MotionPlanning slider (syncing to the loop), which obviously is not the case. So nothing to fix here ;) |
Congrats on getting your first MoveIt pull request merged and improving open source robotics! |
* Add new error types (moveit_msgs moveit#146) * Add default case * Small change to the default case Co-authored-by: Tyler Weaver <maybe@tylerjw.dev> Co-authored-by: Tyler Weaver <maybe@tylerjw.dev>
Description
When using the REALTIME mode to display the planned path the program would stop at the last or second to last waypoint and is unable to re-play the trajectory when using the trajectory slider plugin.
This pull request fixes the REALTIME playback mode to always finish playing the full trajectory and has the ability to properly pause and continue playback using the trajectory slider plugin.
Please let me know if you would like me to add any tests, features or changes.
Thanks,
-Marq