Replies: 1 comment 1 reply
-
I believe it's from the non-dimensionalization, but we could definitely do with double-checking and writing down the full non-dimensionalization somewhere. Also, parameter values between the different SEI models need to be unified (the same parameter is called different things in different submodels) |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
In pybamm/models/submodels/interface/sei/ec_reaction_limited.py, Line 73, we have # j_sei = - C_sei_ec * c_ec * exp(). The expression of C_sei_ec is in pybamm/parameters/lithium_ion_parameters.py, which contains self.U_n_ref. And this U_n_ref is defined in Line 528 in lithium_ion_parameters.py as the Reference OCP based on initial concentration at current collector/electrode interface. I wonder this term is not defined Equation-10 in Yang’s paper (X. G. Yang, J. Power Sources, 360, 28–40 (2017) http://dx.doi.org/10.1016/j.jpowsour.2017.05.110.)
Beta Was this translation helpful? Give feedback.
All reactions