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
Add compute style mliap to MLIAP package #2211
Conversation
Currently the only descriptor style is *sna*, indicating the bispectrum component | ||
descriptors used by the Spectral Neighbor Analysis Potential (SNAP) potentials of | ||
:doc:`pair_style snap <pair_snap>`. | ||
The \'p\' in SNAP is dropped, because keywords that match pair_styles are silently stripped |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@athomps do you know why this is? should we try to do something about it? as you may recall from fixing the pair style hybrid error message, we have now better facilities to do keyword matching (and that uses only one of them).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, my original diagnosis was wrong. The keyword snap is not silently stripped, rather it is matched to a pair style. This causes PairMLIAP::settings() to be called with a truncated argument list. Ironically, this would have worked with the old pair hybrid code, before it was fixed. I don't see a need to fix this new issue and I am okay with removing the comment "The 'p' in SNAP is dropped, because keywords that match pair_styles are silently stripped."
Summary
Added a compute mliap that can be used to train model parameters for the mliap pair_style, analogous to compute snap for the snap pair styles
Related Issues
None
Author(s)
Aidan Thompson, Sandia National Laboratories
Licensing
By submitting this pull request, I agree, that my contribution will be included in LAMMPS and redistributed under either the GNU General Public License version 2 (GPL v2) or the GNU Lesser General Public License version 2.1 (LGPL v2.1).
Backward Compatibility
n/a
Implementation Notes
This compute reproduces output from the compute.snap and compute.quadratic examples in examples/snap. Those examples were also modified slightly to expose more opportunities for failure.
Post Submission Checklist
Further Information, Files, and Links