Skip to content

Commit 8fd8b43

Browse files
committed
Merge branch 'master' of https://github.com/personalrobotics/prpy into feature/more_planningmethods
2 parents 02c7ee3 + 14d5db5 commit 8fd8b43

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

src/prpy/base/robot.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -370,7 +370,6 @@ def PostProcessPath(self, path,
370370

371371
# Copy the trajectory into the output environment.
372372
output_traj = CopyTrajectory(traj, env=self.GetEnv())
373-
output_traj.SetDescription(path.GetDescription())
374373
return output_traj
375374

376375
def ExecutePath(self, path, **kwargs):

src/prpy/planning/base.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -330,7 +330,7 @@ def get_planners(self, method_name):
330330

331331
def plan(self, method, args, kw_args):
332332
all_planners = self._planners
333-
futures = dict()
333+
futures = []
334334
results = [None] * len(self._planners)
335335

336336
# Helper function to call a planner and return its result.
@@ -348,12 +348,12 @@ def call_planner(planner):
348348
.format(planner, method))
349349
continue
350350
else:
351-
futures[index] = defer(call_planner, args=(planner,))
351+
futures.append((index, defer(call_planner, args=(planner,))))
352352

353353
# Each time a planner completes, check if we have a valid result
354354
# (a planner found a solution and all higher-ranked planners had
355355
# already failed).
356-
for index, future in futures.viewitems():
356+
for index, future in futures:
357357
try:
358358
return future.result()
359359
except MetaPlanningError as e:

0 commit comments

Comments
 (0)