Skip to content

Drop deprecated SubstrateAnalyzer + ZSLGenerator reexports#2981

Merged
janosh merged 7 commits intomasterfrom
drop-deprecated-substrate-analyzer-reexport
May 11, 2023
Merged

Drop deprecated SubstrateAnalyzer + ZSLGenerator reexports#2981
janosh merged 7 commits intomasterfrom
drop-deprecated-substrate-analyzer-reexport

Conversation

@janosh
Copy link
Member

@janosh janosh commented May 9, 2023

f0f6087 breaking: remove deprecated re-export of SubstrateAnalyzer, ZSLGenerator from pymatgen/analysis/substrate_analyzer.py
16f6da3 drop custom ArrayLike fallback type, make numpy.typing.ArrayLike required

janosh added 2 commits May 8, 2023 21:15
…tor from pymatgen/analysis/substrate_analyzer.py

now available from pymatgen.analysis.interfaces
@janosh janosh added the breaking Breaking change label May 9, 2023
janosh added 5 commits May 8, 2023 21:27
assert {e.name for e in new.stable_entries} == {
            "ZnO(s)",
            "Zn[2+]",
            "ZnHO2[-]",
            "ZnO2[2-]",
            "Zn(s)",
            "ZnO2(s)",
            "ZnH(s)",
        }, "List of stable entries for unfiltered pbx does not match"
E       AssertionError: List of stable entries for unfiltered pbx does not match
E       assert {'Zn(s)', 'Zn...-]', 'Zn[2+]'} == {'Zn(s)', 'Zn...nO2[2-]', ...}
E         Extra items in the right set:
E         'ZnH(s)'
E         'ZnO2(s)'
E         Use -v to get more diff
@janosh janosh enabled auto-merge (squash) May 11, 2023 17:23
@janosh janosh merged commit 3dd5300 into master May 11, 2023
@janosh janosh deleted the drop-deprecated-substrate-analyzer-reexport branch May 11, 2023 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant