-
Notifications
You must be signed in to change notification settings - Fork 123
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Inconsistent runtime error when using a nequip model to preform Langevin Dynamics in ASE馃悰 [BUG] #232
Comments
Hi @NicholasHattrup ,
This error means that for the given system state and cutoff, every single atom has no neighbors. The vast majority of the time this is a sign that something is wrong (such as trying to set the cutoff in the wrong distance units), and so we made it an error. In MD, it could also be a sign that your simulation is exploding. What kind of system are you simulating, and have you checked the trajectories that throw this error? Do they look physically plausible / is having a system state with such low density reasonable for your application? If it is, I can make this a configurable option to suppress the error. Note that in the case where all atoms have no neighbors, the predicted forces on all atoms will be zero, since the total energy will consist only of constant per-atom and global shifts (depending on your configuration). |
I ran into the same ValueError and noticed that it disappeared when I increased r_max. It would be great if the message was more helpful (f"Every single atom has no neighbors within the cutoff r_min: {r_min}") |
Thanks, although now that I think more about it I don't think this should really be an error. It would be great to use nequip to train/predict the H2 or O2 energy as a function of distance, for example (dissociation curve). Then it would be awkward to only be able to predict points inside r_max. |
Quoting the above,
But like I said if people need it I'm happy to make an option to suppress this error. |
Describe the bug
When running multiple dynamic runs using the nequip calculator for ASE I sometimes have trajectories crashing and giving the error below:
To Reproduce
Most recent nequip version with ASE and if needed I am happy to supply the deployed nequip model I am using. Besides that below is the code I am using to generate the Langevin class to run dynamics with:
Expected behavior
Should just preform Dynamics with no issues and print the associated trajectory number and path where data is being written, i.e.:
Environment (please complete the following information):
Additional Context
For the Trajectories that do not fail, they look perfectly reasonable
The text was updated successfully, but these errors were encountered: