Skip to content

DiffusionAnalyzer will fail if there is no framework ions #342

@zhuyizhou

Description

@zhuyizhou

https://github.com/materialsvirtuallab/pymatgen-analysis-diffusion/blob/5923fb59ec47698462c8b6092339768a542738fd/pymatgen/analysis/diffusion/analyzer.py#L229-L243

DiffusionAnalyzer will fail if there is no framework ions (namely if all ions are mobile)

Basically if framework_indices is an empty list. The first dimension of framework_disp would be 0. Then drift would be a matrix filled with NAN, which leads to failure in the downstream.

This should a minor bug that is easy to fix.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions