Skip to content

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

@esoteric-ephemera
Copy link
Collaborator

@JaGeo and @naik-aakash going to resolve the merge conflicts and approve after I merge in #1036

@esoteric-ephemera
Copy link
Collaborator

Hey @JaGeo and @naik-aakash I like the idea of this, but in keeping with MP's general goal of stricter document schemas that define single data types, I might want to split out these changes to a separate document model (AseMultiDoc or something like that). Will think about how best to do that

@JaGeo
Copy link
Member

JaGeo commented Aug 29, 2025

Fine with me!

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