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

Variable problem of lithium_ionModel #4355

Open
Santiagopeacely opened this issue Aug 19, 2024 · 3 comments
Open

Variable problem of lithium_ionModel #4355

Santiagopeacely opened this issue Aug 19, 2024 · 3 comments

Comments

@Santiagopeacely
Copy link

Santiagopeacely commented Aug 19, 2024

pybamm.lithium_ion.DFN(). Is there a problem with the expression of the two variables related to the heating of positive and negative current collectors? Their heating power is only on the order of 1E-5.
image

@brosaplanella
Copy link
Member

Can you provide a minimum working example about this bug?

@Santiagopeacely
Copy link
Author

I'm not sure if this detail is a bug, but I think the amount of heat generated by the current collector is a bit small.

@kratman kratman added the needs-reply Needs further information from the author and may be closed if no response is received label Nov 21, 2024
@MarcinK00
Copy link

The calculations are correct. For example, for a 10 um thick copper collector at a current density of 3 mA/cm2 (one side coat), the ohmic heating is 1.55E-5 W/m3. At the same time, the value completely disagrees with reality. It should be several orders of magnitude higher. This is due to a simple reason. The calculated ohmic heating is for conduction through the (collector) plane; in practice, the heating is related to conduction in the (collector) plane. Getting the correct value would require solving an additional PDE in 2D. I believe that as it stands, ohmic heating for current collectors is a bit misleading, but I guess it is required.

@github-actions github-actions bot removed the needs-reply Needs further information from the author and may be closed if no response is received label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants