Skip to content

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

Open
wants to merge 43 commits into
base: main
Choose a base branch
from

Conversation

naik-aakash
Copy link
Contributor

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.

@naik-aakash naik-aakash marked this pull request as draft May 6, 2025 06:50
@JaGeo JaGeo changed the title [WIP] Forcefield job update [WIP] Faster Phonon Forcefield Workflow May 6, 2025
@JaGeo
Copy link
Member

JaGeo commented May 7, 2025

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

  • Fix tests
  • Fix linting
  • Find a test case where only one displacement (or different from 1) in the phonon workflow is created
  • Fix overflow in bson document

@JaGeo JaGeo changed the title [WIP] Faster Phonon Forcefield Workflow Faster Phonon Forcefield Workflow By Introducing Batch ASEMakers May 8, 2025
@JaGeo JaGeo marked this pull request as ready for review May 8, 2025 06:25
@JaGeo JaGeo changed the title Faster Phonon Forcefield Workflow By Introducing Batch ASEMakers Faster Phonon Forcefield Workflow By Introducing Batch ASEMakers (especially relevant for remote execution) May 8, 2025
@JaGeo
Copy link
Member

JaGeo commented May 8, 2025

The code snowballstemmer pushed a new release after 4!!! years. I am pinning the version for now.

@esoteric-ephemera
Copy link
Collaborator

@JaGeo will take a look soon!

@JaGeo
Copy link
Member

JaGeo commented May 9, 2025

@esoteric-ephemera It's mainly to make sure I am not breaking anything by accident.

@JaGeo
Copy link
Member

JaGeo commented Jun 14, 2025

@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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants