Skip to content

Add updated MP workflows + cleanup #1139

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

Merged
merged 14 commits into from
Mar 12, 2025

Conversation

esoteric-ephemera
Copy link
Collaborator

@esoteric-ephemera esoteric-ephemera commented Mar 12, 2025

Additions / removals:

  • Add MP workflows currently being used in production with updated / rebenchmarked r2SCAN input sets (pymatgen discussion and foundation discussion)
  • Remove deprecated input sets (MatPES and legacy MP) that were marked for deprecation on 1 January, 2025

Dev:

copy_vasp_kwargs={"additional_vasp_files": ("WAVECAR", "CHGCAR")}
)
)
clean_files: Sequence[str] | None = ("WAVECAR",)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One last point: we also delete wavecars in the lobster workflow. Might be worth to make this consistent in the future.

@JaGeo
Copy link
Member

JaGeo commented Mar 12, 2025

This is great, @esoteric-ephemera . I will merge this and open an issue about the potential update for the Lobster workflow

@JaGeo JaGeo merged commit e6c9476 into materialsproject:main Mar 12, 2025
21 checks passed
@esoteric-ephemera esoteric-ephemera mentioned this pull request Mar 12, 2025
@esoteric-ephemera esoteric-ephemera deleted the updated_mp branch March 14, 2025 18:45
esoteric-ephemera added a commit to esoteric-ephemera/atomate2 that referenced this pull request Apr 3, 2025
* Update pyproject.toml

* Update pyproject.toml

* test a new pymatgen version

* Update pyproject.toml

* Update test_utils.py

* redraft mp flows

* split off openff tests

* remove deprecated flows (completely forgot at the start of the year

* fix pin in strict openff wf

* correctly partition tests?

* revert openff_md test ref change

* openff test over py310-312

---------

Co-authored-by: J. George <[email protected]>
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.

2 participants