Skip to content

Reopen #60: Add capability of execute known trajectory with a ROS action#85

Closed
wkentaro wants to merge 20 commits intomoveit:indigo-develfrom
wkentaro:execute-path-action
Closed

Reopen #60: Add capability of execute known trajectory with a ROS action#85
wkentaro wants to merge 20 commits intomoveit:indigo-develfrom
wkentaro:execute-path-action

Conversation

@wkentaro
Copy link
Copy Markdown
Contributor

@wkentaro wkentaro commented Aug 19, 2016

Reopen #60

@wkentaro
Copy link
Copy Markdown
Contributor Author

Renamed class, action, capability name considering moveit/moveit_msgs#27 .

@rhaschke
Copy link
Copy Markdown
Contributor

Reopened against Kinetic in #94. I suggest to drop the feature for Indigo.

@rhaschke rhaschke closed this Aug 20, 2016
davetcoleman pushed a commit that referenced this pull request Aug 23, 2016
* Add capability to execute trajectory as a ROS action

* cleanup instantiation of MoveGroupInterface

- unified names of execute_trajectory service and action capabilities
- consider overall (wall) timeout for waiting for action servers
- output deprecation warning once during instantiation

* addressed Dave's comments

- use MOVEIT_DEPRECATED instead of ROS_DEPRECATED
- use separated function waitForExecuteActionOrService()

* reduced sleeps to 0.001s again
davetcoleman pushed a commit that referenced this pull request Aug 23, 2016
* Add capability to execute trajectory as a ROS action

* cleanup instantiation of MoveGroupInterface

- unified names of execute_trajectory service and action capabilities
- consider overall (wall) timeout for waiting for action servers
- output deprecation warning once during instantiation

* addressed Dave's comments

- use MOVEIT_DEPRECATED instead of ROS_DEPRECATED
- use separated function waitForExecuteActionOrService()

* reduced sleeps to 0.001s again
@wkentaro wkentaro deleted the execute-path-action branch August 25, 2016 08:59
JafarAbdi pushed a commit to JafarAbdi/moveit that referenced this pull request Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants