Skip to content
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

[FEATURE] How to add a new planner tutorial #280

Open
ChamzasKonstantinos opened this issue Mar 28, 2022 · 1 comment
Open

[FEATURE] How to add a new planner tutorial #280

ChamzasKonstantinos opened this issue Mar 28, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@ChamzasKonstantinos
Copy link
Collaborator

I noticed that several new OMPL planners are still not available in MoveIt here, especially from AO planners only three exist
I think we should
A) make a PR to MoveIt adding new planners such as AIT* and BIT* and
B) Add some scripts and documentation to show examples of adding a new OMPL (or not) planners in robowflex. For example using the access to simple setup.

@ChamzasKonstantinos ChamzasKonstantinos added the enhancement New feature or request label Mar 28, 2022
@mamoll
Copy link
Collaborator

mamoll commented Mar 28, 2022

Re. A), this is likely several PRs, one for each ROS release you care about (which is typically tied to one OMPL <major>.<minor> release). I did this for melodic here in my own moveit clone for the HyperPlan work (the current version of this code in MoveIt is not as ugly, BTW). It'd be cool if MoveIt could just auto-discover all Planner-derived classes in the namespace ompl::geometric, instead of having to hard-code each planner type. That's probably not that easy; for instance, you'd also have to check whether the constructors have the right signature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants