You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There appears to be an inconsistency in handling the ALGO parameter between the Incar and Vasprun classes. Specifically, when setting the ALGO parameter to "GW0" using the Incar class, the value is automatically reformatted to "Gw0" (with only the first letter capitalized).
ALGO = Gw0
...
However, the Vasprun class expects "GW0" with all letters in uppercase when checking the parameter value:
Python version
Python 3.9
Pymatgen version
Version: 2024.4.13
Operating system version
Ubuntu 20.04 LTS
Current behavior
There appears to be an inconsistency in handling the
ALGO
parameter between theIncar
andVasprun
classes. Specifically, when setting theALGO
parameter to "GW0" using theIncar
class, the value is automatically reformatted to "Gw0" (with only the first letter capitalized).However, the
Vasprun
class expects "GW0" with all letters in uppercase when checking the parameter value:pymatgen/pymatgen/io/vasp/outputs.py
Line 1167 in 442d740
In fact, if I manually change "Gw0" in the vasprun.xml to "GW0", the problem is solved. So, it seems to me that we only need to modify the line to
Expected Behavior
The Vasprun should parse the vasprun.xml generated with
ALGO=Gw0
without any error.Minimal example
Relevant files to reproduce this bug
The minimal example above taks too long to run, but you may try to parse the following vasprun with
Vasprun
class.vasprun.xml.zip
The text was updated successfully, but these errors were encountered: