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

Decay rate for single state hysteresis models #3375

Closed
tommaull opened this issue Sep 28, 2023 · 2 comments · Fixed by #3593
Closed

Decay rate for single state hysteresis models #3375

tommaull opened this issue Sep 28, 2023 · 2 comments · Fixed by #3593
Labels

Comments

@tommaull
Copy link
Contributor

Description

Single state hysteresis models typically feature a decay rate. So far, this has not been implemented in pybamm as far as I can see.

Previous hysteresis models implemented in pybamm:
#2256 and #3194

Motivation

Voltage due to hysteresis can have a long timescale, which is left unmodeled in pybamm. This prevents non-instantaneous hysteresis dynamics from being modeled in pybamm.

Possible Implementation

Decay parameter, dependent on change in lithiation. Personally, I also think it would make more sense to have a single OCP with a hysteresis term than switch between two OCPs.
Latz_Plett_hysteresis (1).pdf

Additional context

No response

@ejfdickinson
Copy link

Is the recommendation that this applies to SPMe / DFN models with reference to the open-circuit potential and electrode reaction overpotential, for each electrode independently, and locally in the x-dimension?

The Latz and co-workers approach appears to be speaking of the full cell in terms of OCV and SOC. As a DFN / SPMe submodel alternative to current sigmoid, this would presumably need to work with OCP and lithiation extent respectively.

@tommaull
Copy link
Contributor Author

@ejfdickinson - yes. I hadn't thought about the x-dim as I usually work with SPMes, but that makes sense. I've opened a discussion here, as there's a number of improvements to OCP that could be made. #3376

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants