Robert Timms
Robert Timms
Closing this as it’s been hanging around a while and not sure it’s needed.
Assuming that each layer (electrode pair) behaves identically we can just use the parameter "Number of electrodes connected in parallel to make a cell" to modify the thermal response. The...
The x-full model is only valid for a single-layer pouch, so we should raise a warning when processing the x-full model with "Number of electrodes connected in parallel to make...
Closing as this has been hanging around a while. Can reopen if needed.
At the moment we register citations (e.g. [here](https://github.com/pybamm-team/PyBaMM/blob/develop/pybamm/models/full_battery_models/lithium_ion/dfn.py#L37)) in various parts of the code, which then get logged using the `Citations` class, see [here](https://github.com/pybamm-team/PyBaMM/blob/develop/pybamm/citations.py). You can then print out which...
@agriyakhetarpal sounds great, thanks!
Closing this. New issues can be opened for specific models.
FYI we (Ionworks) have a FUSE internship open for applications that is related to this, so we can support this over the summer See https://www.linkedin.com/jobs/view/3851436140/
Is this just with the `CasadiSolver`? And does it help to set the max step size? ` solver = pybamm.CasadiSolver(extra_options_setup={"max_step_size": dt})` (will slow it down).
Most of the new variables were added so that copying from one model to the next in experiments worked (you need to `Variable` objects in the `variables` dict). > make...