Skip to content

[Bug]: get_pourbaix_entries does not support solid_compat=None #1060

@lkkmpn

Description

@lkkmpn

Code snippet

with MPRester() as mpr:
    mpr.get_pourbaix_entries('Mn', solid_compat=None)  # ValueError

What happened?

The docstring for get_pourbaix_entries suggests that solid_compat can be set to None:

If None, solid DFT energies are used as-is."

However, the implementation only allows solid_compat to be either one of two supported strings or an instance of a Compatibility class.

Version

0.46.0

Which OS?

  • MacOS
  • Windows
  • Linux

Log output

ValueError: Solid compatibility can only be 'MaterialsProjectCompatibility', 'MaterialsProject2020Compatibility', or an instance of a Compatibility class

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions