Skip to content

Commit d9a9d40

Browse files
committed
Switched Ranked planner to use list of tuples instead of map.
1 parent 7204d24 commit d9a9d40

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/prpy/planning/base.py

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

310310
def plan(self, method, args, kw_args):
311311
all_planners = self._planners
312-
futures = dict()
312+
futures = []
313313
results = [None] * len(self._planners)
314314

315315
# Helper function to call a planner and return its result.
@@ -327,12 +327,12 @@ def call_planner(planner):
327327
.format(planner, method))
328328
continue
329329
else:
330-
futures[index] = defer(call_planner, args=(planner,))
330+
futures.append((index, defer(call_planner, args=(planner,))))
331331

332332
# Each time a planner completes, check if we have a valid result
333333
# (a planner found a solution and all higher-ranked planners had
334334
# already failed).
335-
for index, future in futures.viewitems():
335+
for index, future in futures:
336336
try:
337337
return future.result()
338338
except MetaPlanningError as e:

0 commit comments

Comments
 (0)