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

New pymatgen version #3744

Closed
JaGeo opened this issue Apr 10, 2024 · 10 comments · Fixed by #3750
Closed

New pymatgen version #3744

JaGeo opened this issue Apr 10, 2024 · 10 comments · Fixed by #3750
Labels
enhancement A new feature or improvement to an existing one

Comments

@JaGeo
Copy link
Member

JaGeo commented Apr 10, 2024

Feature Requested

I would like to ask for a new pymatgen version after many new implementations in the last month. We currently face a bug in the atomate2 LOBSTER workflow that can only be fixed with a new version. (Cc @naik-aakash )

Proposed Solution

Release of a new version.

I am wondering if the many new type hints and improvements in recent weeks might have an effect on other packages (atomate2, matminer). Do you expect anything like this, @janosh ?

Relevant Information

No response

@JaGeo JaGeo added the enhancement A new feature or improvement to an existing one label Apr 10, 2024
@janosh
Copy link
Member

janosh commented Apr 10, 2024

there was this PR #3624 which you know the downstream implications of better than I.

there also appears to have been an accidental change in #3705

@JaGeo
Copy link
Member Author

JaGeo commented Apr 10, 2024

there was this PR #3624 which you know the downstream implications of better than I.

there also appears to have been an accidental change in #3705

Yes, this one will fix our bug and add some enhancements! Thank you. 😃

@DanielYang59
Copy link
Contributor

DanielYang59 commented Apr 10, 2024

Ah I'm terribly sorry. I don't know why I made such change in #3705 (comment). And it certainly should not be there. I'm looking into the commit history now.

@Andrew-S-Rosen
Copy link
Member

Subscribing because I'm waiting on the next release too :)

@janosh
Copy link
Member

janosh commented Apr 12, 2024

closed by v2024.4.12.

this is a larger-than-usual release with many small and a couple large changes. please report any issues that might come up. e.g. i confirmed that #3624 breaks atomate2 tests

@janosh janosh closed this as completed Apr 12, 2024
@JaGeo
Copy link
Member Author

JaGeo commented Apr 12, 2024

@naik-aakash and I are on it! Thanks!

Side note: We were thinking about a meta package in the foundation meeting but maybe checking automatically in pymatgen before release if it breaks atomate2 would be enough?

@JaGeo
Copy link
Member Author

JaGeo commented Apr 12, 2024

@janosh : This method name got changend in a way that it does not even describe what it does anymore:

def has_antiband_states_below_efermi(self, spin=None, limit=0.01):

Leading to failures in LobsterPy.

Will investigate further but this needs to be changed in any case. It was named "has_antibnd_states_below_efermi" before for Antibonding not antiband.

@JaGeo
Copy link
Member Author

JaGeo commented Apr 12, 2024

Collecting some more issues here: #3705 (comment)

@janosh
Copy link
Member

janosh commented Apr 16, 2024

i think this can be called closed by v2024.4.13?

@janosh janosh closed this as completed Apr 16, 2024
@JaGeo
Copy link
Member Author

JaGeo commented Apr 16, 2024

Yes, thank you! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A new feature or improvement to an existing one
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants