moveit_task_constructor icon indicating copy to clipboard operation
moveit_task_constructor copied to clipboard

Add to MoveIt Melodic Release

Open davetcoleman opened this issue 7 years ago • 10 comments
trafficstars

Can we add this to the moveit.rosinstall melodic release of MoveIt?

Is it time to deprecate the old pick place functionality in Melodic?

davetcoleman avatar Oct 24 '18 01:10 davetcoleman

We don't yet have a drop-in replacement for the old pick-and-place. That's a nice WMD candidate though.

Can we add this to the moveit.rosinstall melodic release of MoveIt?

Why do you think, this is useful? MTC is a package on its own and MoveIt doesn't depend on it.

rhaschke avatar Oct 24 '18 06:10 rhaschke

Is it time to deprecate the old pick place functionality in Melodic?

Not yet, but we are getting closer.

We don't yet have a drop-in replacement for the old pick-and-place. That's a nice WMD candidate though.

Sadly this is too much work for WMD, unless someone with a strong background in the old pick pipeline and some insight into the Task Constructor project works on Pick (not Place yet, that's more work). This should be implemented as a MoveGroupCapability.

Also we still change API and fix core problems, so I'm not in favor of a MoveIt Melodic release yet.

v4hn avatar Oct 24 '18 11:10 v4hn

Can you at least announce it and add tutorials / documentation for it? The more users the better, right?

btw I finally read your paper on the MTC. Nice paper! I'm curious how the task state machine handles error states but that's a discussion for a different thread.

davetcoleman avatar Oct 24 '18 16:10 davetcoleman

I'm curious how the task state machine handles error states but that's a discussion for a different thread.

Not yet at all. This is an idea for future work...

rhaschke avatar Oct 24 '18 17:10 rhaschke

Can you at least announce it

It's somewhat hard to announce it louder than via a ROSCon presentation.

and add tutorials / documentation for it?

Yes. It's on my radar and gets more and more urgent.

You are welcome to fix bugs in MoveIt in the meanwhile.

v4hn avatar Oct 24 '18 21:10 v4hn

It's somewhat hard to announce it louder than via a ROSCon presentation.

I think there are people in the community that have not yet heard of it actually. There was discussion someone about that exact this week (I forget where). How about a blog post? If you don't want to fully create it certainly @rcc0002 can help. I'm sure we can copy the ROSCon abstract as a good start

You are welcome to fix bugs in MoveIt in the meanwhile.

Touche ;-)

davetcoleman avatar Oct 26 '18 18:10 davetcoleman

2 years ago you said this was not ready to be released, but surely not it is. Can we release it to Melodic and Noetic? We can still call it beta and make breaking changes to it...

davetcoleman avatar Jun 02 '20 20:06 davetcoleman

I'm fine with doing so.

rhaschke avatar Jun 02 '20 21:06 rhaschke

heads up @tylerjw Note also MTC is now on the /moveit README badge table: https://github.com/ros-planning/moveit/blob/master/README.md

davetcoleman avatar Jun 02 '20 22:06 davetcoleman

I agree, we will definitely release for Noetic and at the same time might release the same version to Melodic as well.

I would like to have a few interface changes in there before then (prominently finally moving away from unique_ptr), but let's see once everything else is ready for noetic.

v4hn avatar Jun 03 '20 20:06 v4hn