* Refactor pipeline planner
Make code readable
Re-order plan functions
Make usable with parallel planning
Enable configuring multiple pipelines
Add callbacks
Cleanup and documentation
Add API to set parallel planning callbacks and deprecate functions
Pass pipeline map by reference
Small clang-tidy fix
Update core/src/solvers/pipeline_planner.cpp
Co-authored-by: Sebastian Castro <4603398+sea-bass@users.noreply.github.com>
Update core/src/solvers/pipeline_planner.cpp
Format
Refactor to avoid calling .at(0) twice
Use no default stopping criteria
Update fallbacks_move demo
* Cleanup + address deprecation warnings
* Enabling optionally using a property defined pipeline planner map
* Address review
* Disable humble CI for ros2 branch
* Add pipeline planner unittests + some checks
* Add short comment
To provide accurate coverage information.
It was originally RelWithDebInfo because MoveIt uses it
and Travis's timeout would kick in with the old setup.
According to Robert the timeout should not be a problem anymore with
the GHA ci setup. Actually I don't think the timeout was ever a problem for MTC,
but the config was just copied.
And add warnings for gcc AND clang builds (as they can differ slightly).
Clang does not support -Wused-but-set-parameter yet, though
support is [on the way](https://reviews.llvm.org/D100581).
So this gets rid of this warning we see in CI:
> error: unknown warning option '-Wno-unused-but-set-parameter'; did you mean '-Wno-unused-parameter'?