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

Consider replacing the use of label_molecules() by create_openmm_system() from OFF's ForceField #25

Open
martimunicoy opened this issue Aug 6, 2020 · 0 comments
Labels
question Further information is requested

Comments

@martimunicoy
Copy link
Owner

Description

create_openmm_system runs the parameter assignment for OpenMM and it might be a safer option than label_molecules(). For example, the latter does not care about aspects such as exclusion lists (see issue #24).

Solution

Check the differences between both methods before performing any substitution. In case, it happens to be a better method to obtain the parameters for PELE, replace it in: toolkits.py

@martimunicoy martimunicoy added the question Further information is requested label Aug 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant