Skip to content

Commit

Permalink
#3357 remove x-lumped as it is already encompassed by lumped (#3367)
Browse files Browse the repository at this point in the history
  • Loading branch information
brosaplanella authored Sep 25, 2023
1 parent 3c48029 commit acad5fa
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions benchmarks/different_model_options.py
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ class TimeBuildModelThermal:
param_names = ["model", "model option"]
params = (
[pybamm.lithium_ion.SPM, pybamm.lithium_ion.DFN],
["isothermal", "lumped", "x-lumped", "x-full"],
["isothermal", "lumped", "x-full"],
)

def time_setup_model(self, model, params):
Expand All @@ -226,7 +226,7 @@ class TimeSolveThermal:
param_names = ["model", "model option", "solver class"]
params = (
[pybamm.lithium_ion.SPM, pybamm.lithium_ion.DFN],
["isothermal", "lumped", "x-lumped", "x-full"],
["isothermal", "lumped", "x-full"],
[pybamm.CasadiSolver, pybamm.IDAKLUSolver],
)

Expand Down

0 comments on commit acad5fa

Please sign in to comment.