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

On NHC adjoint #1

Open
wwang2 opened this issue Mar 17, 2023 · 1 comment
Open

On NHC adjoint #1

wwang2 opened this issue Mar 17, 2023 · 1 comment

Comments

@wwang2
Copy link

wwang2 commented Mar 17, 2023

Hi

I wanted to inform you that a similar implementation of NHC and Verlet adjoint equations can be found at this GitHub repository: https://github.com/torchmd/mdgrad/blob/cea2332e0ae2c90e2d8caa656c3c51d3b03c59e2/torchmd/sovlers.py#L106. This implementation is licensed under MIT.

Please note that the adjoint equation used in this implementation may induce small gradient errors. Although I am not sure if you are already aware of this issue, it would be nice if someone could fix it at some point. Alternatively, you could use the PyTorch autograd engine for the O(T) backpropagation.

Thank you.

@jakublala
Copy link
Owner

Hi Wujie,

Thanks for commenting on here. This is essentially your implementation copied - sorry for not mentioning it anywhere and not attaching the MIT license. That is all my fault and I apologise.

I didn't know that this implementation may induce small gradient errors! Thank you for notifying me - I will have a look at it once I have the time.

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