Geodesic Interpolated Starting Path #158
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implements the initial path construction specified in: https://pubs.aip.org/aip/jcp/article/150/16/164103/198363/Geodesic-interpolation-for-reaction-pathways.
Code by and large lifted from the repository associated with the paper: https://github.com/virtualzx-nad/geodesic-interpolate/tree/master. However, this implementation honours FixAtoms constraints and periodic boundary conditions in ASE.
Essentially gives a more robust starting guess for the initial path than linear interpolation. Although the full geodesic interpolation algorithm is not implemented, the path should give a better starting guess for IDPP (IDPP uses the linear interpolation as a starting guess and suffers from nasty clashes if two atoms pass through one another).
Test case shows a 120 degree rotation of a CH3 group in ethane, which would otherwise fail spectacularly with the default IDPP interpolation without a better starting guess.
Best use for now would look like: