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

Add "Heat transfer coefficient [W.m-2.K-1]" to BPX schema #89

Open
ikorotkin opened this issue Jan 17, 2025 · 3 comments
Open

Add "Heat transfer coefficient [W.m-2.K-1]" to BPX schema #89

ikorotkin opened this issue Jan 17, 2025 · 3 comments

Comments

@ikorotkin
Copy link
Collaborator

It appears in the lumped thermal model (9a) in the BPX doc.

We have "External surface area [m2]", "Volume [m3]", and "Ambient temperature [K]" in the schema, but no "Heat transfer coefficient [W.m-2.K-1]" to make use of the external surface area and the ambient temperature.

@ikorotkin
Copy link
Collaborator Author

I understand it's not a property of the cell itself but it would be convenient to have it as an optional parameter (for the exchange purposes). So, for example, I can fully reproduce my thermal simulation.

@ejfdickinson
Copy link
Collaborator

See also #4. I don't think this belongs as a "Cell" property, as you say, but may be useful to have a shortlist of common environmental parameters to support full simulation reproduction.

I would support schema re-organisation to remove temperature from the cell properties, too.

@ikorotkin
Copy link
Collaborator Author

ikorotkin commented Feb 10, 2025

The thermal parameters look inconsistent in BPX schema in general:

  • "External surface area [m2]", "Volume [m3]", "Density [kg.m-3]", etc. - optional
  • "Ambient temperature [K]" - required (shouldn't be optional?)
  • "Initial temperature [K]" - optional (shouldn't be required? A simple NMC example from examples directory does not start in PyBaMM without this parameter)
  • "Heat transfer coefficient [W.m-2.K-1]" is defined in the BPX document (lumped thermal model) but does not exist in the schema
  • "Thermal conductivity [W.m-1.K-1]" - not used in the lumped thermal model but defined in the schema (related issue Remove thermal conductivity from BPX schema #50)

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

2 participants