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

Position-dependent charge for electrostatistics #156

Open
smlee99 opened this issue Oct 5, 2024 · 1 comment
Open

Position-dependent charge for electrostatistics #156

smlee99 opened this issue Oct 5, 2024 · 1 comment

Comments

@smlee99
Copy link

smlee99 commented Oct 5, 2024

Hi OpenMM devs!
I have a very similar question as previously mentioned on here. The one who opened this issue just adopt the charge during the simulation using updateParametersInContext method. As pointed by @jchodera , the position-dependence of the charges should also contribute to the force. Is it possible to account for this using a TorchForce object along with OpenMM PME electrostatistics? There are some articles (for instance, see https://www.tandfonline.com/doi/abs/10.1080/13642819208207645) mentioning the technical aspects of implementing charge transfer process.

image image
@peastman
Copy link
Member

peastman commented Oct 5, 2024

As long as gradients can be computed correctly for your force, this will be handled correctly. Your model takes positions and returns energy. TorchForce calls backward() on it to get the gradient with respect to positions.

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

No branches or pull requests

2 participants