Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented code for atom-typing to support the Alexandria force field. #2664

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dspoel
Copy link
Contributor

@dspoel dspoel commented Jan 16, 2024

The Alexandria Chemistry Toolkit for building force fields will be released shortly. It relies on the OpenBabel software to support it for generating atom types. In this patch the forcefieldalexandria is introduced as a child class to forcefieldgaff in an effort to change as little code as possible.

There are many changes to bondtyp.txt which admittably will be hard to check. These will affect any force field typing in OpenBabel.

The Alexandria Chemistry Toolkit for building force fields
will be released shortly. It relies on the OpenBabel software to
support it for generating atom types. In this patch the
forcefieldalexandria is introduced as a child class to forcefieldgaff
in an effort to change as little code as possible.

There are many changes to bondtyp.txt which admittably will be hard to
check. These will affect any force field typing in OpenBabel.
@dspoel
Copy link
Contributor Author

dspoel commented Jan 16, 2024

Hm, the test failure seems to be related to code I haven't touched. Any suggestion on how to debug this?

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.

None yet

1 participant