Skip to content

Commit

Permalink
Merge pull request pybamm-team#1093 from pybamm-team/issue-1092-entro…
Browse files Browse the repository at this point in the history
…pic-fix

Only update entropic variables if reaction_name is empty
  • Loading branch information
valentinsulzer authored Jul 1, 2020
2 parents 60d0d5a + baedfdf commit 33e695e
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions pybamm/models/submodels/interface/base_interface.py
Original file line number Diff line number Diff line change
Expand Up @@ -599,8 +599,15 @@ def _get_standard_ocp_variables(self, ocp, dUdT):
+ " electrode"
+ self.reaction_name
+ " open circuit potential [V]": ocp_av_dim,
self.domain + " electrode entropic change": dUdT,
"X-averaged " + self.domain.lower() + " electrode entropic change": dUdT_av,
}
if self.reaction_name == "":
variables.update(
{
self.domain + " electrode entropic change": dUdT,
"X-averaged "
+ self.domain.lower()
+ " electrode entropic change": dUdT_av,
}
)

return variables

0 comments on commit 33e695e

Please sign in to comment.