-
Notifications
You must be signed in to change notification settings - Fork 10
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
Use openMM to redefine the forcefield #393
Comments
This issue is stale because it has been open for 30 days with no activity. |
Following up on this after today's meeting, we decided to give high priority to modifying the package to be able to read in pdb files with hydrogens generated with @joaomcteixeira's code.
Also, @LilySnow asked to use @joaomcteixeira's code to calculate pairwise energies such as electrostatic and van der waals after we updated the forcefield. Reflecting on it now, to me it's not clear why is this needed, since we've already these features. For sure I'd need more context. |
OPLS is old, so it's recommended to replace it |
We need Joao's code because we will not use anymore the param file for vdw, but we will use OpenMM directly for computing pair-wise energies. See this PR DeepRank/pdbprep#12 |
Hi, Yes, all the code I did is in the https://github.com/DeepRank/pdbprep repository. Regarding interface atom-atom pairwise energy calculation using OpenMM it is all in the DeepRank/pdbprep#12, as you mentioned. Let me know if something is not clear in the documentation of the code or the code itself. Best wishes, |
This is done in pdbprep repository, but deeprank2 still uses OPLS forcefield without OpenMM. Integration of this into deeprank2 will be handled by #509 |
This has multiple advantages:
The text was updated successfully, but these errors were encountered: