-
Notifications
You must be signed in to change notification settings - Fork 9
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
non-urgent todos regarding the MLIPs in autoplex #77
Comments
Is this a pure enhancement, or are there questions about how we want to separate the datasets? (I'm asking so we can assign the right "label" to this issue) |
I think @QuantumChemist is referring to separation into single displaced supercells and random ones. This is a feature that might be useful for the phonon part |
I mean this here. For now, one can only do the fit separately for single-atom-displaced, rattled supercells and the combined dataset for GAP: All the other MLIP can only use the combined dataset. |
Ah! Yes it seems to me to make sense to let the user combine datasets, and to choose their MLIP framework, and to do both of these things independently from one another. (But not needed urgently I think) |
just want to note that the following changes should be done to improve the user experience with autoplex
The text was updated successfully, but these errors were encountered: