moveit_task_constructor
moveit_task_constructor copied to clipboard
Combine MTC with core MoveIt packages?
I think it would make sense to merge the MoveIt Task Constructor packages in this repo with the core MoveIt packages in http://github.com/ros-planning/moveit and http://github.com/ros-planning/moveit2, so I wanted to check with other MTC developers and users (especially @rhaschke and @v4hn ) to see what other folks thought about this.
Based on the structure of the C++ namespaces and the inclusion of some MTC-specific tutorials in the core MoveIt docs, I think this consolidation of packages was the original intent of the MTC authors.
Other reasons why I think this is a good idea:
- Lets us support MTC as a first-class feature of MoveIt instead of a 3rd-party add-on.
- Improves visibility for MTC issues and PRs.
- Makes it easier to keep MTC up to date with new MoveIt features.
- Conversely, makes it easier to keep MoveIt up to date with features added to improve MTC.