-
Notifications
You must be signed in to change notification settings - Fork 3
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
SMIRNOFF: LJPME not allowed #11
Comments
Blocked by #15 |
Re LJ-PME, we definitely want it, as there is a very strong argument for using it e.g. in binding free energy calculations. We probably want this to be called something like "lattice-sum LJ" or similar, again not specifying the exact algorithm (so avoiding the term "PME"). |
Yes, P3M from LAMMPS is essentially equivalent to PME LJ. |
This is probably resolved, but it'd be nice for the prose to be updated to literally include "LJPME" somewhere. When I open up the page and search for just that keyword, there are no hits. |
Yes! Looks resolved, but fully support adding the characters "LJPME" somewhere to make it searchable, maybe just as a parenthetical? |
It probably should be; it's supported in the biggest simulation engines, has valid scientific basis, and sees use in force fields elsewhere.
The text was updated successfully, but these errors were encountered: