Skip to content

Commit

Permalink
fix ec reaction bug (#4774)
Browse files Browse the repository at this point in the history
fix bugs
  • Loading branch information
aabills authored Jan 17, 2025
1 parent c9cb2bb commit ef8571c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
1 change: 0 additions & 1 deletion src/pybamm/models/submodels/interface/sei/sei_growth.py
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,6 @@ def get_coupled_variables(self, variables):
# j = -F * c_0* k_exp() / (1 + L * k_exp() / D)
# c_ec = c_0 - L * k_exp() / D / (1 + L * k_exp() / D)
# = c_0 / (1 + L * k_exp() / D)
eta_SEI = delta_phi
k_exp = phase_param.k_sei * pybamm.exp(-alpha_SEI * F_RT * eta_SEI)
L_over_D = L_sei / phase_param.D_ec
c_0 = phase_param.c_ec_0
Expand Down
3 changes: 2 additions & 1 deletion src/pybamm/simulation.py
Original file line number Diff line number Diff line change
Expand Up @@ -895,7 +895,6 @@ def solve(
logs["summary variables"] = cycle_sum_vars

# Calculate capacity_start using the first cycle
capacity_stop = None
if cycle_num == 1:
# Note capacity_start could be defined as
# self._parameter_values["Nominal cell capacity [A.h]"] instead
Expand All @@ -907,6 +906,8 @@ def solve(
capacity_stop = value
elif typ == "%":
capacity_stop = value / 100 * capacity_start
else:
capacity_stop = None
logs["stopping conditions"]["capacity"] = capacity_stop

logs["elapsed time"] = timer.time()
Expand Down

0 comments on commit ef8571c

Please sign in to comment.