Skip to content

Commit fa4595a

Browse files
authored
Fix the order of planning_adapters (moveit#511)
* Update chomp_planner_tutorial.rst Fix the order of planning_adapters. Add TimeParameterization should be on the top of the list. See moveit/moveit#2053 * Update planning_adapters_tutorial.rst
1 parent 2ebb32e commit fa4595a

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

doc/chomp_planner/chomp_planner_tutorial.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,12 +117,12 @@ To achieve this, follow the steps:
117117
<!-- load OMPL planning pipeline, but add the CHOMP planning adapter. -->
118118
<include file="$(find panda_moveit_config)/launch/ompl_planning_pipeline.launch.xml">
119119
<arg name="planning_adapters" value="
120+
default_planner_request_adapters/AddTimeParameterization
120121
default_planner_request_adapters/FixWorkspaceBounds
121122
default_planner_request_adapters/FixStartStateBounds
122123
default_planner_request_adapters/FixStartStateCollision
123124
default_planner_request_adapters/FixStartStatePathConstraints
124-
chomp/OptimizerAdapter
125-
default_planner_request_adapters/AddTimeParameterization"
125+
chomp/OptimizerAdapter"
126126
/>
127127
</include>
128128
<!-- load chomp config -->

doc/planning_adapters/planning_adapters_tutorial.rst

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,12 +41,12 @@ To achieve this, follow the steps:
4141
#. Open the ``ompl_planning_pipeline.launch`` file in the ``<robot_moveit_config>/launch`` folder of your robot. For the Panda robot it is this `file <https://github.com/ros-planning/panda_moveit_config/blob/melodic-devel/launch/ompl_planning_pipeline.launch.xml>`_. Edit this launch file, find the lines where ``<arg name="planning_adapters">`` is mentioned and change it to: ::
4242

4343
<arg name="planning_adapters"
44-
value="default_planner_request_adapters/FixWorkspaceBounds
44+
value="default_planner_request_adapters/AddTimeParameterization
45+
default_planner_request_adapters/FixWorkspaceBounds
4546
default_planner_request_adapters/FixStartStateBounds
4647
default_planner_request_adapters/FixStartStateCollision
4748
default_planner_request_adapters/FixStartStatePathConstraints
48-
chomp/OptimizerAdapter
49-
default_planner_request_adapters/AddTimeParameterization" />
49+
chomp/OptimizerAdapter" />
5050

5151
#. The values of the ``planning_adapters`` is the order in which the mentioned adapters are called / invoked. Order here matters. Inside the CHOMP adapter, a :moveit_codedir:`call <moveit_planners/chomp/chomp_optimizer_adapter/src/chomp_optimizer_adapter.cpp#L169>` to OMPL is made before invoking the CHOMP optimization solver, so CHOMP takes the initial path computed by OMPL as the starting point to further optimize it.
5252

0 commit comments

Comments
 (0)