Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ICOHPLIST.lobster format changed in lobster 5.1.1 #336

Closed
yumaoDFT opened this issue Oct 2, 2024 · 5 comments · Fixed by #342
Closed

ICOHPLIST.lobster format changed in lobster 5.1.1 #336

yumaoDFT opened this issue Oct 2, 2024 · 5 comments · Fixed by #342

Comments

@yumaoDFT
Copy link

yumaoDFT commented Oct 2, 2024

ICOHPLIST.lobster format changed in lobster 5.1.1, which pops out a ValueError when parsing with Analysis.
Snipaste_2024-10-02_13-53-39

I checked the ICOHPLIST.lobster; it now has spin 1 and spin 2 values on the same row. Previously, spin 1 and spin 2 are written in different rows.
Snipaste_2024-10-02_13-54-51

@JaGeo
Copy link
Owner

JaGeo commented Oct 2, 2024

Thank you for reporting this. We are working on this. We currently have a PR open in pymatgen and hope to close it in the next days.

See materialsproject/pymatgen#4065

@naik-aakash
Copy link
Collaborator

Hi @yumaoDFT, the changes necessary in the code to make it work with new versions of LOBSTER are already in place.

Maybe a new release of Pymatgen can take a while. In case you need to be able to use it urgently, uninstall the Pymatgen version you have and then please install the specific branch using the following command:

pip install git+https://github.com/naik-aakash/pymatgen.git@lobster_parsers_update

@yumaoDFT
Copy link
Author

yumaoDFT commented Oct 2, 2024

Thank you very much!

@JaGeo
Copy link
Owner

JaGeo commented Oct 17, 2024

To close this, we should update as soon as a new pymatgen version is out

@JaGeo
Copy link
Owner

JaGeo commented Oct 25, 2024

@yumaoDFT there is a new release now with the fixes thanks to @naik-aakash

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

Successfully merging a pull request may close this issue.

3 participants