Skip to content
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

Temperature dependence in SEI models #1695

Closed
valentinsulzer opened this issue Sep 28, 2021 · 6 comments · Fixed by #2104
Closed

Temperature dependence in SEI models #1695

valentinsulzer opened this issue Sep 28, 2021 · 6 comments · Fixed by #2104
Assignees
Labels
difficulty: easy A good issue for someone new. Can be done in a few hours

Comments

@valentinsulzer
Copy link
Member

valentinsulzer commented Sep 28, 2021

  • For the EC reaction limited, reaction limited, and interstitial diffusion limited, multiply the exponent by the following factor (dimensionless 1/T):
T = variables["Negative electrode temperature"]
prefactor = 1 / (1 + self.param.Theta * T)
  • Add arrhenius dependency to diffusivity in ec reaction limited and solvent diffusion limited (I don't know what the arrhenius factor should be)
  • Arrhenius dependence for the conductivity of the SEI in electron migration limited
@valentinsulzer valentinsulzer added the difficulty: easy A good issue for someone new. Can be done in a few hours label Oct 8, 2021
@soma2000-lang
Copy link
Contributor

Working on this issue@tinosulzer

@soma2000-lang
Copy link
Contributor

@tinosulzer I will be submitting a proposal for the idea "Documentation" ,for GSOC22 under pybamm.
I have a doubt about how to know what the arrhenius dependency should be.
It would be great to be mentored by you and other mentors

@valentinsulzer
Copy link
Member Author

Don't worry about arrhenius for now, it's a little bit more complicated. If you can just open a PR for the first bullet point that would be great

@soma2000-lang
Copy link
Contributor

@tinosulzer Can you please tell me where SEI model file is, I am having difficulty in finding them.

@soma2000-lang
Copy link
Contributor

@tinosulzer thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty: easy A good issue for someone new. Can be done in a few hours
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants