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

added Exp6-AAD, AMBER-AA, TRAPPE-UA, Mie-UA, FF styles tests and XML … #108

Merged
merged 13 commits into from
Aug 4, 2024

Conversation

bc118
Copy link
Collaborator

@bc118 bc118 commented Aug 3, 2024

##Added Exp6-AAD, AMBER-AA, TRAPPE-UA, Mie-UA, FF styles tests and XML FF examples in the test/files directory. Changed the FF form to U_opls... This resolves the issue below for this repo, but a few tests examples, will be created here https://github.com/GOMC-WSU/GOMC_Examples/tree/main/MoSDeF-dihedral-fit for the Exp6, Mie, TRAPPE, and Amber FF examples.

#99

Changed the FF form to U_opls and periodic dihedrals, RB-torsion, This resolves this part of the following issue:

issue = #100

Line 100-103: I would like to suggest for the equations to use physical meaningful symbols as targets quantities (U or V). Examples: V_{dihedral}^{OPLS}(\theta) (instead of OPLS_{Energy}), V_{dihedral}^{RB}(\psi), V_{dihedral}^{Periodic}(\theta)

…FF examples in the test/files directory. Changed the FF form to U_opls...
@bc118 bc118 added the enhancement New feature or request label Aug 3, 2024
@bc118 bc118 requested a review from daico007 August 3, 2024 19:02
Copy link
Collaborator

@daico007 daico007 left a comment

Choose a reason for hiding this comment

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

everything looks fine on quick skim through, only some minor typo

README.md Show resolved Hide resolved
mosdef_dihedral_fit/dihedral_fit/fit_dihedral_with_gomc.py Outdated Show resolved Hide resolved
mosdef_dihedral_fit/dihedral_fit/fit_dihedral_with_gomc.py Outdated Show resolved Hide resolved
mosdef_dihedral_fit/dihedral_fit/fit_dihedral_with_gomc.py Outdated Show resolved Hide resolved
mosdef_dihedral_fit/dihedral_fit/fit_dihedral_with_gomc.py Outdated Show resolved Hide resolved
mosdef_dihedral_fit/dihedral_fit/fit_dihedral_with_gomc.py Outdated Show resolved Hide resolved
mosdef_dihedral_fit/dihedral_fit/fit_dihedral_with_gomc.py Outdated Show resolved Hide resolved
@bc118 bc118 merged commit 0129997 into GOMC-WSU:main Aug 4, 2024
6 checks passed
@bc118 bc118 deleted the add_ff_forms branch August 24, 2024 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants