diff --git a/CHANGELOG.md b/CHANGELOG.md index 1ab711cf21..b2f8637442 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,7 @@ # [Unreleased](https://github.com/pybamm-team/PyBaMM/) +- Replaced rounded faraday constant with its exact value in `bpx.py` for better comparison between different tools + ## Features - Added additional user-configurable options to the (`IDAKLUSolver`) and adjusted the default values to improve performance. ([#4282](https://github.com/pybamm-team/PyBaMM/pull/4282)) diff --git a/pybamm/parameters/bpx.py b/pybamm/parameters/bpx.py index 680477a74b..2f259a8f52 100644 --- a/pybamm/parameters/bpx.py +++ b/pybamm/parameters/bpx.py @@ -240,7 +240,7 @@ def _entropic_change(sto, c_s_max, dUdT, constant=False): # (cs-cs_max)) in BPX exchange current is defined j0 = F * k_norm * sqrt((ce/ce0) * # (cs/cs_max) * (1-cs/cs_max)) c_e = pybamm_dict["Initial concentration in electrolyte [mol.m-3]"] - F = 96485 + F = pybamm.constants.F.value def _exchange_current_density(c_e, c_s_surf, c_s_max, T, k_ref, Ea): return (