Add fields to store planning time in pick-and-place#43
Add fields to store planning time in pick-and-place#43130s merged 1 commit intomoveit:melodic-develfrom
Conversation
|
This will break ABI in a difficult way for Kinetic. I believe this should be merged into ROS-M, which we haven't branched for yet. We are having a maintainer meeting on March 20th and will be discussing branching. Would you like to join this discussion? |
|
It makes sense to me to put this off until ROS-M. |
|
I agree, we should not merge this into kinetic. |
|
@aochiai please email me so I can invite you. |
9c0ef4b to
343a726
Compare
|
@aochiai I have created the new branch |
| # The performed grasp, if attempt was successful | ||
| Grasp grasp | ||
|
|
||
| # The amount of time in seconds it took to complete the plan |
There was a problem hiding this comment.
This seems ambiguous. Is it the time to plan or the time to execute the plan? Maybe reword to "it took to generate a plan"
| # The successful place location, if any | ||
| PlaceLocation place_location | ||
|
|
||
| # The amount of time in seconds it took to complete the plan |
There was a problem hiding this comment.
This seems ambiguous. Is it the time to plan or the time to execute the plan? Maybe reword to "it took to generate a plan"
|
@aochiai Can you rewrite the comment to make it less ambiguous in another PR? |
Add new fields to store planning time of the pick-and-place capability.
MoveGroup.actionalready has this.