moveit / moveit_tutorials

A sphinx-based centralized documentation repo for MoveIt
https://moveit.github.io/moveit_tutorials/
BSD 3-Clause "New" or "Revised" License
483 stars 695 forks source link

Improvements to ompl benchmarking tutorial (melodic) #478

Open simonschmeisser opened 4 years ago

simonschmeisser commented 4 years ago

Description

I tried following the benchmarking tutorial and ran into some issues, collecting them here first and hope to address them in PRs

[ INFO] [1586249659.474563951]: Planner configuration 'panda_arm[BKPIECEkConfigDefault]' will use planner 'geometric::BKPIECE'. Additional configuration parameters will be set when the planner is constructed.
[ INFO] [1586249659.474780724]: panda_arm[BKPIECEkConfigDefault]: Attempting to use default projection.
[ERROR] [1586249659.474879369]: No default projection is set. Perhaps setup() needs to be called
[ERROR] [1586249659.478963309]: OMPL encountered an error: No projection evaluator specified
================================================================================REQUIRED process [moveit_run_benchmark-3] has died!

Your environment

welcome[bot] commented 4 years ago

Thanks for reporting an issue. Because we're a volunteer community, providing a pull request with suggested changes is always welcomed.

simonschmeisser commented 4 years ago

default projection can be fixed by adding:

projection_evaluator: joints(panda_joint1,panda_joint2) to ompl_planning.yaml

simonschmeisser commented 4 years ago

@rhaschke could yout tag this as WorldMoveItDay candidate so whoever is interested can find it?

mamoll commented 4 years ago

The benchmarking functionality is actually not OMPL-specific. Other planning pipelines (CHOMP, STOMP, ...) can also be used. This tutorial has lots of issues as @simonschmeisser has already pointed out. I took me a while to get things working, too.

felixvd commented 3 years ago

In addition to the thread @simonschmeisser just linked from, I noticed that the Stored Scenes and Stored States tabs are never mentioned or used outside of the Benchmarking tutorial page. The only use "in the wild" I have seen is in this PR, for which there are good reasons not to merge imo (although the functionality has a use).

I would consider what the intended workflow for setting up a benchmarking problem is, and then try to combine the Stored Scenes/Stored States tab into one if possible. I actually also think the warehouse connection on the connect tab should be in there, because it affects nothing except those two tabs.

Note that the scripts in this package allow saving scenes/states to files, which is more intuitive and much easier to share/add to a repo.