-
-
Notifications
You must be signed in to change notification settings - Fork 529
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SEI transfer coefficient #2338
Comments
Sure. Let's handle it the same way as symmetric/asymmetric BV. Add new SEI options "reaction limited (asymmetric)" and "EC reaction limited (asymmetric)" which require the transfer coefficient to be defined. |
Shall I leave the option |
Definitely have a deprecation warning if it changes. I'm not sure it needs to be changed though? |
The SEI transfer coefficient for
reaction limited
andEC reaction limited
have the transfer coefficient hardcoded to 0.5. This value is a modelling assumption (see Safari et al. 2009), so I think we should allow to define it in the parameter values so it can be changed.The text was updated successfully, but these errors were encountered: