Skip to content

Commit

Permalink
Merge pull request #2248 from pybamm-team/composite-release-parameters
Browse files Browse the repository at this point in the history
Composite release parameters
  • Loading branch information
valentinsulzer authored Aug 24, 2022
2 parents 1472d97 + 0c4b972 commit 4a8a221
Show file tree
Hide file tree
Showing 105 changed files with 4,776 additions and 3,900 deletions.
10 changes: 8 additions & 2 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

- Added new cumulative variables `Throughput capacity [A.h]` and `Throughput energy [W.h]` to standard variables and summary variables, to assist with degradation studies. Throughput variables are only calculated if `calculate discharge energy` is set to `true`. `Time [s]` and `Time [h]` also added to summary variables. ([#2249](https://github.com/pybamm-team/PyBaMM/pull/2249))
- Added `lipf6_OKane2022` electrolyte to `OKane2022` parameter set ([#2249](https://github.com/pybamm-team/PyBaMM/pull/2249))
- Reformated submodel structure to allow composite electrodes. Composite positive electrode is now also possible. With current implementation, electrodes can have at most two phases. ([#2248](https://github.com/pybamm-team/PyBaMM/pull/2248))

## Bug fixes

Expand All @@ -17,6 +18,10 @@
- Simplified scaling for the exchange-current density. The dimensionless parameter `C_r` is kept, but no longer used anywhere ([#2238](https://github.com/pybamm-team/PyBaMM/pull/2238))
- Added limits for variables in some functions to avoid division by zero, sqrt(negative number), etc ([#2213](https://github.com/pybamm-team/PyBaMM/pull/2213))

## Breaking changes

- Parameters specific to a (primary/secondary) phase in a domain are doubly nested. e.g. `param.c_n_max` is now `param.n.prim.c_max` ([#2248](https://github.com/pybamm-team/PyBaMM/pull/2248))

# [v22.7](https://github.com/pybamm-team/PyBaMM/tree/v22.7) - 2022-07-31

## Features
Expand Down Expand Up @@ -51,7 +56,7 @@
## Breaking changes

- Exchange-current density functions (and some other functions) now take an additional argument, the maximum particle concentration for that phase ([#2134](https://github.com/pybamm-team/PyBaMM/pull/2134))
- Loss of lithium to SEI on cracks is now a degradation variable, so setting a particle mechanics submodel is now compulsory (NoMechanics will suffice)
- Loss of lithium to SEI on cracks is now a degradation variable, so setting a particle mechanics submodel is now compulsory (NoMechanics will suffice) ([#2104](https://github.com/pybamm-team/PyBaMM/pull/2104))

# [v22.6](https://github.com/pybamm-team/PyBaMM/tree/v22.6) - 2022-06-30

Expand All @@ -73,6 +78,7 @@

## Features

- Added a casadi version of the IDKLU solver, which is used for `model.convert_to_format = "casadi"` ([#2002](https://github.com/pybamm-team/PyBaMM/pull/2002))
- Added functionality to generate Julia expressions from a model. See [PyBaMM.jl](https://github.com/tinosulzer/PyBaMM.jl) for how to use these ([#1942](https://github.com/pybamm-team/PyBaMM/pull/1942)))
- Added basic callbacks to the Simulation class, and a LoggingCallback ([#1880](https://github.com/pybamm-team/PyBaMM/pull/1880)))

Expand All @@ -82,7 +88,7 @@

## Breaking changes

- Changed domain-specific parameter names to a nested attribute, e.g. `param.c_n_max` is now `param.n.c_max` ([#2063](https://github.com/pybamm-team/PyBaMM/pull/2063))
- Changed domain-specific parameter names to a nested attribute. `param.n.l_n` is now `param.n.l` ([#2063](https://github.com/pybamm-team/PyBaMM/pull/2063))

# [v22.4](https://github.com/pybamm-team/PyBaMM/tree/v22.4) - 2022-04-30

Expand Down
35 changes: 21 additions & 14 deletions examples/notebooks/models/compare-ecker-data.ipynb

Large diffs are not rendered by default.

10 changes: 5 additions & 5 deletions examples/notebooks/models/electrode-state-of-health.ipynb
Original file line number Diff line number Diff line change
Expand Up @@ -51,12 +51,12 @@
{
"data": {
"application/vnd.jupyter.widget-view+json": {
"model_id": "97443d8234c241ca8c2fcaad3e21e416",
"model_id": "34d757993a5247958345757ac4f7cbef",
"version_major": 2,
"version_minor": 0
},
"text/plain": [
"interactive(children=(FloatSlider(value=0.0, description='t', max=2.3248422253249537, step=0.02324842225324953"
"interactive(children=(FloatSlider(value=0.0, description='t', max=2.324842225325321, step=0.023248422253253208"
]
},
"metadata": {},
Expand All @@ -65,7 +65,7 @@
{
"data": {
"text/plain": [
"<pybamm.plotting.quick_plot.QuickPlot at 0x7fe0614e9bb0>"
"<pybamm.plotting.quick_plot.QuickPlot at 0x7fac51ea9fd0>"
]
},
"execution_count": 2,
Expand Down Expand Up @@ -146,8 +146,8 @@
"Cp = parameter_values.evaluate(param.p.cap_init)\n",
"n_Li = parameter_values.evaluate(param.n_Li_particles_init)\n",
"\n",
"Un = param.n.U_dimensional\n",
"Up = param.p.U_dimensional\n",
"Un = param.n.prim.U_dimensional\n",
"Up = param.p.prim.U_dimensional\n",
"T_ref = param.T_ref"
]
},
Expand Down
Loading

0 comments on commit 4a8a221

Please sign in to comment.