-
Notifications
You must be signed in to change notification settings - Fork 116
Faster Phonon Forcefield Workflow By Introducing Batch ASEMakers (especially relevant for remote execution) #1196
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
base: main
Are you sure you want to change the base?
Faster Phonon Forcefield Workflow By Introducing Batch ASEMakers (especially relevant for remote execution) #1196
Conversation
Similar to the "socket" implementation in FHI AIMs, I will implement an option that any ASEMaker can take more than 1 structure as an input. I will, therefore, adapt all output models and also include the optional "all_forces" entry in the output model. This will significantly speed up the code execution of such workflows for large structures and certain calculators, as the calculator only needs to be set up once. Additionally, it will avoid that one has to run hundreds of very small jobs on a remote cluster. There are still some more tests to perform and potentially fix. I would be happy to hear some additional opinions now. Especially also from you @esoteric-ephemera, as you have implemented many of the ASE-related functionalities. @janosh @utf
|
The code snowballstemmer pushed a new release after 4!!! years. I am pinning the version for now. |
@JaGeo will take a look soon! |
…te2 into phonon_workflow_ff_mod
@esoteric-ephemera It's mainly to make sure I am not breaking anything by accident. |
@esoteric-ephemera could you Take a look and see if this would be fine for you? I think it relates to your other forcefield updates |
Summary
This is an initial experimental code implementation for combining the phonon workflow in a single job when using forcefields. Its not very up to date with changes in over a year. Needs quite bit of modifications to make it work with updated API.
Will be adapted by @JaGeo further.