Merge pull request #2664 from pybamm-team/pre-commit-ci-update-config

chore: update pre-commit hooks

Author: valentinsulzer

.pre-commit-config.yaml

@@ -10,12 +10,12 @@ repos:
         exclude: assets/js/webapp\.js
 
   - repo: https://github.com/psf/black
-    rev: 22.12.0
+    rev: 23.1.0
     hooks:
       - id: black
 
   - repo: https://github.com/charliermarsh/ruff-pre-commit
-    rev: "v0.0.237"
+    rev: "v0.0.242"
     hooks:
       - id: ruff
         args: [--ignore=E741, --exclude=__init__.py]

benchmarks/different_model_options.py

@@ -83,7 +83,6 @@ class TimeBuildModelLossActiveMaterial:
     )
 
     def time_setup_model(self, model, params):
-
         build_model("Ai2020", model, "loss of active material", params)
 
 

benchmarks/work_precision_sets/time_vs_abstols.py

@@ -35,9 +35,7 @@
     itertools.product(solvers.values(), models.values()),
     itertools.product(solvers, models),
 ):
-
     for params in parameters:
-
         time_points = []
         solver = i[0]
 
@@ -70,13 +68,11 @@
         disc.process_model(model)
 
         for tol in abstols:
-
             solver.atol = tol
             solver.solve(model, t_eval=t_eval)
             time = 0
             runs = 20
             for k in range(0, runs):
-
                 solution = solver.solve(model, t_eval=t_eval)
                 time += solution.solve_time.value
             time = time / runs

benchmarks/work_precision_sets/time_vs_dt_max.py

@@ -39,9 +39,7 @@
     models.values(),
     models,
 ):
-
     for params in parameters:
-
         time_points = []
         # solver = pybamm.CasadiSolver()
 
@@ -74,14 +72,12 @@
         disc.process_model(model)
 
         for t in dt_max:
-
             solver = pybamm.CasadiSolver(dt_max=t)
 
             solver.solve(model, t_eval=t_eval)
             time = 0
             runs = 20
             for k in range(0, runs):
-
                 solution = solver.solve(model, t_eval=t_eval)
                 time += solution.solve_time.value
             time = time / runs

benchmarks/work_precision_sets/time_vs_mesh_size.py

@@ -29,7 +29,6 @@
     itertools.product(solvers.values(), models.values()),
     itertools.product(solvers, models),
 ):
-
     for params in parameters:
         time_points = []
         solver = i[0]
@@ -42,7 +41,6 @@
         i = list(i)
 
         for N in npts:
-
             var_pts = {
                 "x_n": N,  # negative electrode
                 "x_s": N,  # separator
@@ -57,7 +55,6 @@
             time = 0
             runs = 20
             for k in range(0, runs):
-
                 solution = sim.solve([0, 3500])
                 time += solution.solve_time.value
             time = time / runs

benchmarks/work_precision_sets/time_vs_no_of_states.py

@@ -28,7 +28,6 @@
     itertools.product(solvers.values(), models.values()),
     itertools.product(solvers, models),
 ):
-
     for params in parameters:
         time_points = []
         ns = []
@@ -42,7 +41,6 @@
         i = list(i)
 
         for N in npts:
-
             var_pts = {
                 "x_n": N,  # negative electrode
                 "x_s": N,  # separator
@@ -57,7 +55,6 @@
             time = 0
             runs = 20
             for k in range(0, runs):
-
                 solution = sim.solve([0, 3500])
                 time += solution.solve_time.value
             time = time / runs

benchmarks/work_precision_sets/time_vs_reltols.py

@@ -41,9 +41,7 @@
     itertools.product(solvers.values(), models.values()),
     itertools.product(solvers, models),
 ):
-
     for params in parameters:
-
         time_points = []
         solver = i[0]
 
@@ -76,13 +74,11 @@
         disc.process_model(model)
 
         for tol in reltols:
-
             solver.rtol = tol
             solver.solve(model, t_eval=t_eval)
             time = 0
             runs = 20
             for k in range(0, runs):
-
                 solution = solver.solve(model, t_eval=t_eval)
                 time += solution.solve_time.value
             time = time / runs

examples/scripts/compare_interface_utilisation.py

@@ -36,7 +36,6 @@
 t_eval = np.linspace(0, 3600, 100)
 
 for model in models:
-
     sim = pybamm.Simulation(model, parameter_values=param)
     solution = sim.solve(t_eval)
     solutions.append(solution)

pybamm/discretisations/discretisation.py

@@ -9,7 +9,6 @@
 
 
 def has_bc_of_form(symbol, side, bcs, form):
-
     if symbol in bcs:
         if bcs[symbol][side][1] == form:
             return True
@@ -346,7 +345,6 @@ def set_internal_boundary_conditions(self, model):
         """
 
         def boundary_gradient(left_symbol, right_symbol):
-
             pybamm.logger.debug(
                 "Calculate boundary gradient ({} and {})".format(
                     left_symbol, right_symbol
@@ -1001,7 +999,6 @@ def check_model(self, model):
         self.check_variables(model)
 
     def check_initial_conditions(self, model):
-
         # Check initial conditions are a numpy array
         # Individual
         for var, eqn in model.initial_conditions.items():

pybamm/expression_tree/interpolant.py

@@ -251,7 +251,6 @@ def _function_evaluate(self, evaluated_children):
         if self.dimension == 1:
             return self.function(*children_eval_flat).flatten()[:, np.newaxis]
         elif self.dimension in [2, 3]:
-
             # If the children are scalars, we need to add a dimension
             shapes = []
             for child in evaluated_children:
@@ -273,7 +272,6 @@ def _function_evaluate(self, evaluated_children):
                 shape = shapes.pop()
             new_evaluated_children = []
             for child in evaluated_children:
-
                 if hasattr(child, "shape") and child.shape == shape:
                     new_evaluated_children.append(child.flatten())
                 else: