Skip to content

Commit c9cb2bb

Browse files
Merge pull request #4763 from pybamm-team/generalize-get-esoh-solver
don't get esoh solver for any non-lithium-ion model
2 parents 1ac422a + 7342d4e commit c9cb2bb

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/pybamm/simulation.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1022,8 +1022,8 @@ def step(
10221022
def _get_esoh_solver(self, calc_esoh):
10231023
if (
10241024
calc_esoh is False
1025-
or isinstance(self._model, pybamm.lead_acid.BaseModel)
1026-
or isinstance(self._model, pybamm.equivalent_circuit.Thevenin)
1025+
or not isinstance(self._model, pybamm.lithium_ion.BaseModel)
1026+
or self._model.options["particle phases"] not in ["1", ("1", "1")]
10271027
or self._model.options["working electrode"] != "both"
10281028
):
10291029
return None

0 commit comments

Comments
 (0)