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

(Structure|Molecule).alphabetical_formula #3478

Merged
merged 9 commits into from
Nov 18, 2023
Merged

Conversation

janosh
Copy link
Member

@janosh janosh commented Nov 17, 2023

fc7d8db fix AssertionError assert structure # for mypy
cbb44a2 fix PhononDosPlotter.get_plot only ax.set_ylim if relevant_y is non-empty
77191b6 PhononDosPlotter.get_plot add legend keyword to customize legend
1cee2f5 Add alphabetical_formula property to SiteCollection class
5f2d7bf add TestIStructure.test_alphabetical_formula

        if temperature and not structure:
            raise ValueError("If using temperature input, you must also include structure")
sync axes labels font size to legend font size
@janosh janosh added enhancement A new feature or improvement to an existing one core Pymatgen core labels Nov 17, 2023
@janosh janosh merged commit 27066d3 into master Nov 18, 2023
20 of 22 checks passed
@janosh janosh deleted the struct-alph-formula branch November 18, 2023 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Pymatgen core enhancement A new feature or improvement to an existing one
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant