Skip to content

Commit

Permalink
Temporarily drop down number of trials for level1 and level2
Browse files Browse the repository at this point in the history
In Qiskit#3999 we increased the number of trials, but the global benefit from
doing this isn't clear. With the pending release weighing the increased
run time for more trials against this, this commit temporarily reverts
the number of trials for level 1 and level 2 back to their previous
setting of 20. When we get to the bottom of Qiskit#4094 and have a fix or
change for that we can increase the number of trials again.

Related Qiskit#4094
  • Loading branch information
mtreinish committed Apr 7, 2020
1 parent 9d7c711 commit 0b3b89e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion qiskit/transpiler/preset_passmanagers/level1.py
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ def _swap_condition(property_set):
if routing_method == 'basic':
_swap += [BasicSwap(coupling_map)]
elif routing_method == 'stochastic':
_swap += [StochasticSwap(coupling_map, trials=50, seed=seed_transpiler)]
_swap += [StochasticSwap(coupling_map, trials=20, seed=seed_transpiler)]
elif routing_method == 'lookahead':
_swap += [LookaheadSwap(coupling_map, search_depth=4, search_width=4)]
else:
Expand Down
2 changes: 1 addition & 1 deletion qiskit/transpiler/preset_passmanagers/level2.py
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ def _swap_condition(property_set):
if routing_method == 'basic':
_swap += [BasicSwap(coupling_map)]
elif routing_method == 'stochastic':
_swap += [StochasticSwap(coupling_map, trials=100, seed=seed_transpiler)]
_swap += [StochasticSwap(coupling_map, trials=20, seed=seed_transpiler)]
elif routing_method == 'lookahead':
_swap += [LookaheadSwap(coupling_map, search_depth=5, search_width=5)]
else:
Expand Down

0 comments on commit 0b3b89e

Please sign in to comment.