Skip to content

Commit

Permalink
Change Driver Defaults
Browse files Browse the repository at this point in the history
The default max iteration count for the driver based calls was different than the default behavior for the OO approach. This update brings those two in line.
Corey Ostrove committed Feb 21, 2024
1 parent 1f2a62f commit ff777dd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pygsti/drivers/longsequence.py
Original file line number Diff line number Diff line change
@@ -930,7 +930,7 @@ def _get_optimizer(advanced_options, model_being_optimized):
from pygsti.forwardsims.matrixforwardsim import MatrixForwardSimulator as _MatrixFSim
advanced_options = advanced_options or {}
default_fditer = 1 if isinstance(model_being_optimized.sim, _MatrixFSim) else 0
optimizer = {'maxiter': advanced_options.get('max_iterations', 100000),
optimizer = {'maxiter': advanced_options.get('max_iterations', 100),
'tol': advanced_options.get('tolerance', 1e-6),
'fditer': advanced_options.get('finitediff_iterations', default_fditer)}
optimizer.update(advanced_options.get('extra_lm_opts', {}))

0 comments on commit ff777dd

Please sign in to comment.