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

SMIRNOFF: LJPME not allowed #11

Open
mattwthompson opened this issue Jun 16, 2021 · 5 comments
Open

SMIRNOFF: LJPME not allowed #11

mattwthompson opened this issue Jun 16, 2021 · 5 comments
Assignees
Labels

Comments

@mattwthompson
Copy link
Member

It probably should be; it's supported in the biggest simulation engines, has valid scientific basis, and sees use in force fields elsewhere.

@mattwthompson
Copy link
Member Author

Blocked by #15

@davidlmobley
Copy link

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").

@mrshirts
Copy link

Yes, P3M from LAMMPS is essentially equivalent to PME LJ.

@mattwthompson
Copy link
Member Author

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.

@davidlmobley
Copy link

Yes! Looks resolved, but fully support adding the characters "LJPME" somewhere to make it searchable, maybe just as a parenthetical?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants