Skip to content

MoveIt Task Constructor [MTC] - #371

Merged
rhaschke merged 10 commits into
moveit:masterfrom
PickNikRobotics:moveit_task_constructor
Oct 10, 2019
Merged

MoveIt Task Constructor [MTC]#371
rhaschke merged 10 commits into
moveit:masterfrom
PickNikRobotics:moveit_task_constructor

Conversation

@simonGoldstein

@simonGoldstein simonGoldstein commented Aug 1, 2019

Copy link
Copy Markdown
Contributor

Description

With MTC gaining such popularity it is important to create an official tutorial for it. This is a basic tutorial modeled after the MoveIt Grasps tutorial. This links to the MTC repo to use the to be added demo package to show the panda in a simple pick and place scenario. The PR to add the demo can he found here

Selection_013

This shows how the different stage types and ideas are described.

Selection_014

The reason the build fails is because the Edit this page link destination doesn't exist as this is a new tutorial

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

@rhaschke rhaschke left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this first contribution. I pushed some improvements to the wording.
As suggested in moveit/moveit_task_constructor#104 (review), I suggest to put the corresponding code next to the tutorial.
For the future, I think we should split the MTC tutorial(s) into multiple pages, discussing

  • the overall concept and usage (as provided by this PR)
  • task pipeline configuration
  • property inheritance
  • and programmatic extension (programming new stages)

in separate sub tutorials. Would be great, if you could prepare the infrastructure for this.

Comment thread doc/moveit_task_constructor/moveit_task_constructor_tutorial.rst Outdated
@rhaschke
rhaschke merged commit 9289ed4 into moveit:master Oct 10, 2019
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.

4 participants