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
Fix Lobsterenv Bug #3637
Fix Lobsterenv Bug #3637
Conversation
Thanks! Can you add a test to catch future regressions? |
Yes, just added one now |
Hi @janosh , can a new release of Pymatgen be made please? As this is bug breaks our LobsterPy package JaGeo/LobsterPy#232 Sorry for the inconvenience. |
we just had one today so i'd prefer to accumulate a few more changes before the next release |
@janosh , this currently prevents our on-going JOSS review for LobsterPy. We could use the latest master branch from pymatgen for the release but it's not exactly best practice. |
i see. let's also get in #3593 (awaiting OP) and then we can release |
@janosh Thanks! |
v2024.2.23 should be on PyPI in ~1h. |
Thanks 😄 |
Bug
The charges read from file are not assigned
Fix
Fix the erroneous if else condition introduced during #3624
Todo