Skip to content

Conversation

Thomas3R
Copy link
Contributor

@Thomas3R Thomas3R commented Dec 12, 2023

Periodic boundary conditions are now available in xtb when using either the --gfnff or --mcgfnff command line argument. Here, GFN-FF refers to the original parameterization, and mcGFN-FF to a version with scaled non-covalent interactions for molecular crystals. As with other methods, the lattice parameters have to be present in the input file in order for xtb to recognize a periodic system.

Thomas3R added 30 commits March 24, 2023 16:13
@Thomas3R Thomas3R requested review from MtoLStoN and Albkat December 12, 2023 12:41
Copy link
Contributor

@MtoLStoN MtoLStoN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have added some comments, but this looks generally good to me. Maybe take a look at your error handling.

@Thomas3R Thomas3R merged commit 6c2a62a into grimme-lab:main Feb 8, 2024
gorges97 pushed a commit to gorges97/xtb that referenced this pull request Mar 27, 2024
* Implementation of periodic boundary conditions for GFN-FF "--gfnff" and scaled version "--mcgfnff" for molecular crystals

Co-authored-by: Marcel Stahn <70513124+MtoLStoN@users.noreply.github.com>
Signed-off-by: Johannes Gorges <58849467+gorges97@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants