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

Add LightStructureEnvironments.from_structure_environments() fallback value if ce_and_neighbors is None #3002

Merged
merged 2 commits into from
May 20, 2023

Conversation

janosh
Copy link
Member

@janosh janosh commented May 20, 2023

Closes #2756.

@janosh janosh added fix Bug fix PRs analysis Concerning pymatgen.analysis labels May 20, 2023
@janosh janosh enabled auto-merge (squash) May 20, 2023 15:05
@janosh janosh merged commit 7276923 into master May 20, 2023
@janosh janosh deleted the fix-gh-2756 branch May 20, 2023 15:25
lbluque pushed a commit to lbluque/pymatgen that referenced this pull request May 23, 2023
… value if `ce_and_neighbors` is None (materialsproject#3002)

* LightStructureEnvironments.from_structure_environments() add fallback value if ce_and_neighbors is None (closes materialsproject#2756)

* add StructureEnvironmentsTest.test_from_structure_environments() to prevent future regression
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analysis Concerning pymatgen.analysis fix Bug fix PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TypeError: 'NoneType' object is not subscriptable
1 participant