PyBaMM icon indicating copy to clipboard operation
PyBaMM copied to clipboard

Extend `pybamm.ParameterValues.create_from_bpx()` to automate initial concentrations for blended electrodes

Open ejfdickinson opened this issue 1 year ago • 4 comments

Description

Calling pybamm.ParameterValues.create_from_bpx() with a BPX file containing a blended electrode returns a warning as follows:

Initial concentrations cannot be set using stoichiometry limits for blend electrodes. Please set the initial concentrations manually.

The method could automate this calculation by solving the equations specified in the BPX standard.

Motivation

No response

Possible Implementation

No response

Additional context

No response

ejfdickinson avatar Aug 13 '24 08:08 ejfdickinson

Tagging https://github.com/pybamm-team/PyBaMM/issues/2682 as related. Currently the electrode state of health models in pybamm don't support blended electrodes.

rtimms avatar Aug 13 '24 20:08 rtimms

@rtimms Off-topic I know, but for this reason I find it odd that calc_esoh=True is the default in pybamm.Simulation.solve().

ejfdickinson avatar Aug 14 '24 10:08 ejfdickinson

Yes, this is simply because calc_esoh predates blended electrodes. I’m open to changing this.

rtimms avatar Aug 15 '24 13:08 rtimms

The proper fix, which we are planning to do, is to do "just in time" calculations of esoh values only when they are requested

valentinsulzer avatar Aug 16 '24 03:08 valentinsulzer