-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add our MoveIt 2 roadmap, as discussed at MoveIt Workshop #371
Conversation
Milestone 1: replace "ament" with "colcon"? Determinism: why single out BIT*? I'd perhaps rename this whole category to "Improved Path Quality." While OMPL may not be the best fit for some common settings, it is still useful as a fallback for hard problems, where it tends to be more efficient than alternatives. Also, completeness/correctness are sometimes hard to guarantee with other planning approaches. OMPL can do Cartesian planning, but is not deterministic. One of the suggestions from the workshop was to allow multiple planner plugins in the same move group. If this could be combined with running multiple planners from different plugins in parallel, then I think you could significantly improve path quality on most common problems:
Maybe change the Cartesian Planner bullet and subbullet to: Under Machine Learning, I'd not say nn-based motion planning. It just isn't practical on its own. However, it may be very useful to learn local motion primitives, so maybe focus on that:
|
For completeness, here are some notes from discussions at the roadmapping session of the MoveIt workshop (thanks in part to @mikeferguson and @marip8): Core Planning
API / High-level interface
Applications / Industry / Specialties
@mlautman may have some additional notes from the roadmapping session. |
Thank you so much @mamoll @mikeferguson @marip8 for the additional notes! As I sadly did not attend the workshop, here is some feedback from my side, too. I highlighted everything directly related to this pull-request in bold. Milestone 2The following two things already exist and work in current MoveIt. It does not make sense to me to put them on the roadmap without more specific things we want to improve or rewrite there:
Lifecycle Management of MoveIt nodesSupporting a simple Replace pluginlib with componentsIn the last maintainer meeting where we discussed this point, it was very fuzzy what this could look like. Industry Priority 1
This is not just a matter of MoveIt, but the general question on how this is should be implemented across ROS2. Still, MoveIt could pioneer a new "standard" approach for this (and extend other packages to support it).
This is already possible right now by having a big urdf with all robots and many people did it in the past. What exactly is new/requested here?
Industry Priority 2 - Broadly described as "more moveit task constructor features"Of course, it is quite frustrating to me to see this item here without having been around. More sensors supported in planning scene, instead of just pointcloudsWhat does this mean? Sensors are not at all supported in the PlanningScene, geometric measurements are. What alternative measurements are relevant to people?
This would make for a great "Milestone 2/3" entry, @davetcoleman ! It's an often-discussed shortcoming of MoveIt as of today and it's clearly achievable.
@davetcoleman This, too, would make a wonderful MoveIt2 milestone entry. This request still entails a way to describe meshes as convex, which is not currently available in robot descriptions.
As I'm probably one of the very few people who used this in recent years, here's my experience with the current state:
This boils down to implementing a new joint type |
I would like to add my vote to adding support for scene graphs. This would it make it much easier to integrate MoveIt with task planners or combined task & motion planners. It would be pretty cool if complex tasks could be expressed via, say, PDDL and a feasible motion plan could automatically be generated, Also, as a side note, I get the impression that some people think convex collision checking (similar to what is used in TrajOpt) is the same as continuous collision detection. It is not. It is a heuristic way to approximate swept volume. If I have 2 configurations of a 1-dof revolute link rotated 180 degrees, then the convex hull of the two configurations is a very bad approximation of the swept volume. So convex collision checking is not eliminating the need to choose collision checking resolution appropriately. |
I'm a bit annoyed by the long winded responses from me trying to share a simple roadmap. And I'm afraid I don't have time to spend hours debating the subtleties of mostly un-funded, theoretical work we may or may not do in the future. The goal of this post was to quickly provide a vision for other potential funding sources. The content is based on:
I'll try to incorporate some feedback, but really just need to get something shared online for potential contributors:
Both are used in ROS 2 - Dirk made it very confusing. Ament is the "Build Type"
Its just a rough approximate goal that is very very tangible / understandable. The global planner doesn't have to be realtime. Not providing a concrete target number makes the milestone way more ambiguous. This is meant to inspire us.
It was a passing suggestion of @henningkayser
Added run in parallel
We need more than an interface though...
We do have many working implementations with Descartes and have it somewhat integrated into MoveIt, but in the wrong way. Reworded to "Similar to Descartes but fully integrated as MoveIt planner"
Discussions like these are outside the scope of a high level roadmap, especially for milestones that we have no idea when we'll be able to get to them.
Its the last milestone because its the lowest priority, but its an important buzzword to acknowledge its something we're thinking about and that it will likely, someday, be the dominate approach IMHO.
Added.
What does this have to do with ML? |
@mamoll here is the source spreadsheet with the roadmap. Please leave any further comments directly in there for easier collaboration. Again though, we need this published for MoveIt 2 funding I'm very actively working on right now post-ROSCon discussions. Please be quick. @v4hn I also added you as a review of the spreadsheet: @mamoll When you are satisfied, could you please re-add screenshots of that document into this PR, and add your consolidated / summarized notes from the MoveIt Workshop to the bottom of this new webpage? I appreciate you taking the lead on this PR. |
Thank you for pointing out this overlap, it actually made me aware of a different one. |
@mamoll I added comments to the spreadsheet, please have a look. |
Add our MoveIt 2 roadmap, as discussed at MoveIt Workshop Former-commit-id: 53339bd
I keep getting questions about this, so it needs to be on website