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
Add info messages to pick and place routine #1004
Merged
davetcoleman
merged 3 commits into
moveit:kinetic-devel
from
felixvd:add-pick-place-debug-message
Jul 31, 2018
Merged
Add info messages to pick and place routine #1004
davetcoleman
merged 3 commits into
moveit:kinetic-devel
from
felixvd:add-pick-place-debug-message
Jul 31, 2018
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The current behavior silently adds an arbitrary wait time to the pipeline. Users should define an appropriate wait time for their own gripper. This message helps them find the issue, which is otherwise not trivial to debug.
felixvd
added a commit
to felixvd/moveit_tutorials
that referenced
this pull request
Jul 30, 2018
The [video in the Pick and Place tutorial](http://docs.ros.org/kinetic/api/moveit_tutorials/html/doc/pick_place/pick_place_tutorial.html) is 45 seconds long. For 28 of those 45 seconds, the robot is standing still. This is because a default wait time of 7 seconds is added after every opening and closing of the gripper. This change fixes that time to 0.5 s and makes the movement look smooth. I also propose notifying the user about this behavior in [this PR](moveit/moveit#1004).
mlautman
approved these changes
Jul 30, 2018
davetcoleman
approved these changes
Jul 31, 2018
davetcoleman
pushed a commit
that referenced
this pull request
Jul 31, 2018
* Add debug messages to pick and place routine The current behavior silently adds an arbitrary wait time to the pipeline. Users should define an appropriate wait time for their own gripper. This message helps them find the issue, which is otherwise not trivial to debug. * Apply clang-format * Fix auto-formatting
cherry picked to melodic |
mlautman
pushed a commit
to moveit/moveit_tutorials
that referenced
this pull request
Aug 6, 2018
The [video in the Pick and Place tutorial](http://docs.ros.org/kinetic/api/moveit_tutorials/html/doc/pick_place/pick_place_tutorial.html) is 45 seconds long. For 28 of those 45 seconds, the robot is standing still. This is because a default wait time of 7 seconds is added after every opening and closing of the gripper. This change fixes that time to 0.5 s and makes the movement look smooth. I also propose notifying the user about this behavior in [this PR](moveit/moveit#1004).
Thanks @felixvd ! |
mayman99
pushed a commit
to mayman99/moveit
that referenced
this pull request
Aug 25, 2018
* Add debug messages to pick and place routine The current behavior silently adds an arbitrary wait time to the pipeline. Users should define an appropriate wait time for their own gripper. This message helps them find the issue, which is otherwise not trivial to debug. * Apply clang-format * Fix auto-formatting
pull bot
pushed a commit
to shadow-robot/moveit
that referenced
this pull request
Sep 3, 2020
* Add debug messages to pick and place routine The current behavior silently adds an arbitrary wait time to the pipeline. Users should define an appropriate wait time for their own gripper. This message helps them find the issue, which is otherwise not trivial to debug. * Apply clang-format * Fix auto-formatting
cohen39
pushed a commit
to cohen39/pick_place_forked
that referenced
this pull request
Oct 26, 2021
The [video in the Pick and Place tutorial](http://docs.ros.org/kinetic/api/moveit_tutorials/html/doc/pick_place/pick_place_tutorial.html) is 45 seconds long. For 28 of those 45 seconds, the robot is standing still. This is because a default wait time of 7 seconds is added after every opening and closing of the gripper. This change fixes that time to 0.5 s and makes the movement look smooth. I also propose notifying the user about this behavior in [this PR](moveit/moveit#1004).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
The current pick and place pipeline silently assigns a default duration of an arbitrary (and long) 7 seconds to each action of the gripper. This PR adds a message alerting the user to this behavior, and prompts them to define an appropriate wait time for their own gripper. This helps them identify a problem which is otherwise not trivial to debug, due to the structure of the planning stages. Case in point: it was an issue in the official pick and place tutorial.
I suppose this should have been a PR on melodic-devel and cherry-picked to Kinetic. Let me know if anything needs changing.
Checklist