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

Adapt to pymatgen 2023.07.20 update. #393

Merged
merged 13 commits into from
Aug 14, 2023
Merged

Conversation

qchempku2017
Copy link
Collaborator

@qchempku2017 qchempku2017 commented Jul 23, 2023

Summary

  • Add support to pymatgen 2023.07.20
  • Upgrade numpy to 1.25.1
  • [issue] test_orbits/test_fixed_values now fails because licabr_orbit's symmetry operation is now 24 instead of 12. Need to figure this out!

Additional dependencies introduced (if any)

TODO (if any)

Checklist

@lbluque
Copy link
Collaborator

lbluque commented Jul 23, 2023

Perhaps the issue with the number of symmetry operations is related to this PR made some months ago materialsproject/pymatgen#2727

@lbluque
Copy link
Collaborator

lbluque commented Jul 23, 2023

Perhaps the issue with the number of symmetry operations is related to this PR made some months ago materialsproject/pymatgen#2727

Submitted a bug fix, see materialsproject/pymatgen#3179

@qchempku2017
Copy link
Collaborator Author

Perhaps the issue with the number of symmetry operations is related to this PR made some months ago materialsproject/pymatgen#2727

Wonderful! Thank you so much for taking care of this!

requirements.txt Outdated Show resolved Hide resolved
tests/test_cofe/test_orbit.py Outdated Show resolved Hide resolved
@lbluque
Copy link
Collaborator

lbluque commented Aug 14, 2023

Thanks a lot @qchempku2017 !

@lbluque lbluque merged commit 5975fb3 into CederGroupHub:main Aug 14, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants