Skip to content
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

MatPESStaticSet restore GGA tag removal if xc_functional.upper() == "R2SCAN" #3288

Merged
merged 9 commits into from Sep 1, 2023

Conversation

janosh
Copy link
Member

@janosh janosh commented Aug 31, 2023

13883af use str.split for INHERITED_INCAR_PARAMS
9d6ee83 add MatPESStaticSet init type hints
a89090a move ValueError Unrecognized {xc_functional=} to top
aeebf00 refactor MatPESStaticSet def incar(self) -> Incar:
bfd9476 leave user-defined ALGO if set, always apply LADU if xc_functional.upper().endswith("+U") and restore removing GGA tag if xc_functional.upper() == "R2SCAN"
5b7ab3f fix test_functionals

@janosh janosh added io Input/output functionality fix Bug fix PRs vasp Vienna Ab initio Simulation Package labels Aug 31, 2023
Signed-off-by: Janosh Riebesell <janosh.riebesell@gmail.com>
@janosh janosh enabled auto-merge (squash) August 31, 2023 15:55
@janosh janosh disabled auto-merge August 31, 2023 16:10
Signed-off-by: Janosh Riebesell <janosh.riebesell@gmail.com>
revert setdefault r2SCAN algo
@janosh janosh merged commit bc9039a into master Sep 1, 2023
21 of 22 checks passed
@janosh janosh deleted the fix-MatPESStaticSet branch September 1, 2023 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Bug fix PRs io Input/output functionality vasp Vienna Ab initio Simulation Package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant