Skip to content

Commit

Permalink
revert parts of prfo update
Browse files Browse the repository at this point in the history
  • Loading branch information
shoubhikraj committed Jun 12, 2024
1 parent d39926b commit 4130d48
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
4 changes: 3 additions & 1 deletion autode/opt/optimisers/prfo.py
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,9 @@ def _initialise_run(self) -> None:
"""
assert self._species is not None, "Must have a species to init"

self._build_internal_coordinates()
self._coords = CartesianCoordinates(self._species.coordinates).to(
"dic"
)
self._update_hessian_gradient_and_energy()
return None

Expand Down
5 changes: 3 additions & 2 deletions tests/test_opt/test_prfo.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ def test_diels_alder_ts_opt():
print(xyz_file_string, file=file)

mol = Molecule("init.xyz")
PRFOptimiser.optimise(mol, method=xtb, maxiter=50, init_alpha=0.1)
PRFOptimiser.optimise(mol, method=xtb, maxiter=30, init_alpha=0.05)
mol.print_xyz_file(filename="/Users/shoubhikrajmaiti/ts.xyz")
assert has_single_imag_freq_at_xtb_level(mol)
# print(mol.imaginary_frequencies) # should be ~600 cm-1
print(mol.imaginary_frequencies) # should be ~600 cm-1

0 comments on commit 4130d48

Please sign in to comment.