#1547 changelog

valentinsulzer · valentinsulzer · commit c42c309cd402 · 2021-07-14T16:21:07.000-04:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -27,6 +27,7 @@
 
 ## Bug fixes
 
+-   Fixed ElectrodeSOH model for multi-dimensional simulations ([#1548](https://github.com/pybamm-team/PyBaMM/pull/1548))
 -   Removed the overly-restrictive check "each variable in the algebraic eqn keys must appear in the eqn" ([#1510](https://github.com/pybamm-team/PyBaMM/pull/1510))
 -   Made parameters importable through pybamm ([#1475](https://github.com/pybamm-team/PyBaMM/pull/1475))
 
diff --git a/pybamm/solvers/solution.py b/pybamm/solvers/solution.py
@@ -746,7 +746,7 @@ def get_cycle_summary_variables(cycle_solution, esoh_sim):
                 },
                 solver=solver,
             )
-        except pybamm.SolverError:
+        except pybamm.SolverError:  # pragma: no cover
             raise pybamm.SolverError(
                 "Could not solve for summary variables, run "
                 "`sim.solve(calc_esoh=False)` to skip this step"

Original file line number	Diff line number	Diff line change
`@@ -746,7 +746,7 @@ def get_cycle_summary_variables(cycle_solution, esoh_sim):`
`746`	`746`	`},`
`747`	`747`	`solver=solver,`
`748`	`748`	`)`
`749`		`- except pybamm.SolverError:`
	`749`	`+ except pybamm.SolverError: # pragma: no cover`
`750`	`750`	`raise pybamm.SolverError(`
`751`	`751`	`"Could not solve for summary variables, run "`
`752`	`752`	"`sim.solve(calc_esoh=False)` to skip this step"