bug fix

duartegroup · Jul 17, 2024 · c9286b3 · c9286b3
1 parent 076bad6
commit c9286b3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/autode/opt/optimisers/crfo.py b/autode/opt/optimisers/crfo.py
@@ -133,7 +133,7 @@ def _step(self) -> None:
         self._take_step_within_max_move(delta_s)
         return None
 
-    def _take_step_within_max_move(self, delta_s):
+    def _take_step_within_max_move(self, delta_s: np.ndarray):
         """
         Take the step by converting internal coordinates to Cartesian
         coordinates, and scaling back if the maximum movement of an

diff --git a/tests/test_opt/test_crfo.py b/tests/test_opt/test_crfo.py
@@ -424,7 +424,7 @@ def test_composite_bond_constraint():
 def test_trust_radius_limits():
     import autode.opt.optimisers.crfo
 
-    max_lim = autode.opt.optimisers.crfo._max_trust
+    max_lim = autode.opt.optimisers.crfo.MAX_TRUST
     opt = CRFOptimiser(
         maxiter=10, gtol=1e-3, etol=1e-4, init_trust=max_lim + 0.1
     )