-
Notifications
You must be signed in to change notification settings - Fork 25
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
Interface to AIMNet2 #63
Comments
Agreed. Can I put you in touch with the authors? Do we intend for this to be useful in ML/MM hybrid simulations? Would we envision the ML model to only be used for intra-ML-region interactions, or would we want some part of the model (eg charges) to be used in ML-MM interactions? |
We are already using it for MM/MM and it's good.
…On Tue, Oct 24, 2023 at 10:02 PM John Chodera ***@***.***> wrote:
Agreed. Can I put you in touch with the authors?
Do we intend for this to be useful in ML/MM hybrid simulations? Would we
envision the ML model to only be used for intra-ML-region interactions, or
would we want some part of the model (eg charges) to be used in ML-MM
interactions?
—
Reply to this email directly, view it on GitHub
<#63 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB3KUOTWSIIDSGV75UOXYELYBANFFAVCNFSM6AAAAAA6OEKQXWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZXHE2TANRQGQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
In principle it should be possible to use the charges it computes to compute interactions between the ML and MM regions. But the first step is to use fixed charges the same way we do with ANI. |
That would be great. |
AFAICT they only provide TorchScript compiled files https://github.com/isayevlab/AIMNet2/tree/main/models |
AIMNet2 looks like a good candidate for another model to include. It supports 14 elements and charged molecules. It includes an explicit Coulomb term, which should help with accuracy on large molecules. The current implementation has O(N^2) time and memory in the number of atoms. They say a linear scaling version is coming soon.
The text was updated successfully, but these errors were encountered: