Merge pull request #358 from gdsfactory/pre-commit-ci-update-config

[pre-commit.ci] pre-commit autoupdate

.pre-commit-config.yaml

@@ -15,7 +15,7 @@ repos:
       - id: trailing-whitespace
 
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: "v0.2.2"
+    rev: "v0.3.3"
     hooks:
       - id: ruff
       - id: ruff-format
@@ -26,7 +26,7 @@ repos:
   #     - id: shellcheck
 
   - repo: https://github.com/PyCQA/bandit
-    rev: 1.7.7
+    rev: 1.7.8
     hooks:
       - id: bandit
         args: [--exit-zero]

gplugins/common/utils/add_simulation_markers.py

@@ -1,4 +1,5 @@
 """Returns component with simulation markers."""
+
 from __future__ import annotations
 
 import warnings

gplugins/common/utils/async_helpers.py

@@ -18,9 +18,11 @@ async def handle_return(
     append: bool = False,
 ) -> None:
     """Reads through a :class:`StreamReader` and tees content to ``out_stream`` and ``log_file``."""
-    with open(
-        log_file, "a" if append else "w", encoding="utf-8", buffering=1
-    ) if log_file else nullcontext(None) as f:
+    with (
+        open(log_file, "a" if append else "w", encoding="utf-8", buffering=1)
+        if log_file
+        else nullcontext(None) as f
+    ):
         while True:
             # Without this sleep, the program won't exit
             await asyncio.sleep(0)

gplugins/common/utils/disable_print.py

@@ -1,4 +1,5 @@
 """Disable print statements for overly verbose simulators."""
+
 from __future__ import annotations
 
 import os

gplugins/common/utils/get_component_with_net_layers.py

@@ -88,9 +88,9 @@ def get_component_with_net_layers(
                             new_layers_init[0] + i,
                             new_layers_init[1] + j,
                         )
-                        layer_stack.layers[
-                            f"{old_layername}{delimiter}{portname}"
-                        ] = new_layer
+                        layer_stack.layers[f"{old_layername}{delimiter}{portname}"] = (
+                            new_layer
+                        )
                     net_component.add_polygon(polygon, layer=new_layer_number)
             # Otherwise put the polygon back on the same layer
             else:

gplugins/common/utils/optical_constants.py

@@ -1,4 +1,5 @@
 """Useful optical constant manipulations."""
+
 import numpy as np
 
 

gplugins/common/utils/parse_layer_stack.py

@@ -1,4 +1,5 @@
 """Preprocessing involving only the LayerStack."""
+
 from __future__ import annotations
 
 import numpy as np

gplugins/devsim/get_simulation_xsection.py

@@ -8,7 +8,6 @@
 
 """
 
-
 from __future__ import annotations
 
 import warnings
@@ -57,8 +56,8 @@ def dn_carriers(wavelength: float, dN: float, dP: float) -> float:
     else:
         wavelength *= 1e-6
         return (
-            -3.64 * 1e-10 * wavelength**2 * dN
-            - 3.51 * 1e-6 * wavelength**2 * np.power(dP, 0.8)
+            -3.64 * 1e-10 * wavelength** 2 * dN
+            - 3.51 * 1e-6 * wavelength** 2 * np.power(dP, 0.8)
         )
 
 

gplugins/femwell/solve_thermal.py

@@ -45,16 +45,16 @@ def get_thermal_conductivities(basis):
         # adding the layer to the thermal_conductivities dict. Check for that
         # case
         if domain in thermal_conductivities:
-            thermal_conductivity[
-                basis.get_dofs(elements=domain)
-            ] = thermal_conductivities[domain]
+            thermal_conductivity[basis.get_dofs(elements=domain)] = (
+                thermal_conductivities[domain]
+            )
         else:
             for material, labels in materials_dict.items():
                 if domain in labels:
                     # Assign the right values
-                    thermal_conductivity[
-                        basis.get_dofs(elements=domain)
-                    ] = thermal_conductivities[material]
+                    thermal_conductivity[basis.get_dofs(elements=domain)] = (
+                        thermal_conductivities[material]
+                    )
                     break
 
     thermal_conductivity *= 1e-12  # 1e-12 -> conversion from 1/m^2 -> 1/um^2

gplugins/gmeep/get_simulation.py

@@ -1,4 +1,5 @@
 """Returns simulation from component."""
+
 from __future__ import annotations
 
 import inspect

gplugins/gmeep/get_simulation_grating_farfield.py

@@ -7,6 +7,7 @@
 - add tests
 
 """
+
 from __future__ import annotations
 
 from typing import Any

gplugins/gmeep/write_sparameters_grating.py

@@ -6,6 +6,7 @@
 - 9.2 for Oband
 
 """
+
 from __future__ import annotations
 
 import hashlib

gplugins/gmsh/parse_component.py

@@ -1,4 +1,5 @@
 """Preprocessing involving both the GDS and the LayerStack, or the resulting simulation polygons."""
+
 from __future__ import annotations
 
 import numpy as np
@@ -75,32 +76,32 @@ def process_buffers(layer_polygons_dict: dict, layer_stack: LayerStack):
                         layer_stack.layers[layername].thickness * zs[z_ind + 1]
                         - layer_stack.layers[layername].thickness * z
                     )
-                    extended_layer_stack_layers[
-                        f"{layername}_{poly_ind}_{z}"
-                    ] = LayerLevel(
-                        layer=layer_stack.layers[layername].layer,
-                        thickness=new_thickness,
-                        zmin=new_zmin,
-                        material=layer_stack.layers[layername].material,
-                        info=layer_stack.layers[layername].info,
-                        mesh_order=layer_stack.layers[layername].mesh_order,
+                    extended_layer_stack_layers[f"{layername}_{poly_ind}_{z}"] = (
+                        LayerLevel(
+                            layer=layer_stack.layers[layername].layer,
+                            thickness=new_thickness,
+                            zmin=new_zmin,
+                            material=layer_stack.layers[layername].material,
+                            info=layer_stack.layers[layername].info,
+                            mesh_order=layer_stack.layers[layername].mesh_order,
+                        )
                     )
                     extended_layer_polygons_dict[f"{layername}_{poly_ind}_{z}"] = (
                         f"{layername}",
                         f"{layername}_{poly_ind}_{zs[z_ind+1]}",
                         polygon.buffer(width_buffer),
                         polygon.buffer(width_buffers[z_ind + 1]),
                     )
-                extended_layer_stack_layers[
-                    f"{layername}_{poly_ind}_{zs[-1]}"
-                ] = LayerLevel(
-                    layer=layer_stack.layers[layername].layer,
-                    thickness=0,
-                    zmin=layer_stack.layers[layername].zmin
-                    + layer_stack.layers[layername].thickness,
-                    material=layer_stack.layers[layername].material,
-                    info=layer_stack.layers[layername].info,
-                    mesh_order=layer_stack.layers[layername].mesh_order,
+                extended_layer_stack_layers[f"{layername}_{poly_ind}_{zs[-1]}"] = (
+                    LayerLevel(
+                        layer=layer_stack.layers[layername].layer,
+                        thickness=0,
+                        zmin=layer_stack.layers[layername].zmin
+                        + layer_stack.layers[layername].thickness,
+                        material=layer_stack.layers[layername].material,
+                        info=layer_stack.layers[layername].info,
+                        mesh_order=layer_stack.layers[layername].mesh_order,
+                    )
                 )
     return extended_layer_polygons_dict, LayerStack(layers=extended_layer_stack_layers)
 

gplugins/gmsh/parse_gds.py

@@ -1,4 +1,5 @@
 """Preprocessing involving mostly the GDS polygons."""
+
 from __future__ import annotations
 
 import gdsfactory as gf

gplugins/klayout/drc/samples/drc_errors.py

@@ -1,4 +1,5 @@
 """Write GDS with sample errors."""
+
 from __future__ import annotations
 
 import gdsfactory as gf

gplugins/lumerical/write_sparameters_lumerical.py

@@ -1,4 +1,5 @@
 """Write Sparameters with Lumerical FDTD."""
+
 from __future__ import annotations
 
 import shutil

gplugins/lumerical/write_sparameters_lumerical_components.py

@@ -1,4 +1,5 @@
 """Write Sparameters with for different components."""
+
 from __future__ import annotations
 
 import gdsfactory as gf

gplugins/modes/find_modes.py

@@ -10,6 +10,7 @@
 1/1.5 = 0.6667.
 
 """
+
 import json
 import pathlib
 import pickle

gplugins/modes/find_modes_cross_section.py

@@ -10,6 +10,7 @@
 1/1.5 = 0.6667.
 
 """
+
 from __future__ import annotations
 
 import pathlib

gplugins/modes/neff_convergence_test.py

@@ -6,6 +6,7 @@
 hyperparameters that allowed convergence.
 
 """
+
 from __future__ import annotations
 
 import meep as mp

gplugins/process/implant_tables.py

@@ -1,6 +1,7 @@
 """
 Reference: Selberherr, S. (1984). Process Modeling. In: Analysis and Simulation of Semiconductor Devices. Springer, Vienna. https://doi.org/10.1007/978-3-7091-8752-4_3
 """
+
 from pathlib import Path
 
 import numpy as np

gplugins/process/pysrim.py

@@ -50,9 +50,7 @@ def run_fragmented_calculation(
 
     for i, num_ions in enumerate(fragment(step, number_ions)):
         print(
-            "total ions completed: {:06d}\tion: {}\tions in step: {:06d}".format(
-                i * step, ion.symbol, num_ions
-            )
+            f"total ions completed: {i * step:06d}\tion: {ion.symbol}\tions in step: {num_ions:06d}"
         )
         trim_settings = trim_settings or {"calculation": 2}
         trim = TRIM(target, ion, number_ions=num_ions, **trim_settings)

gplugins/process/silicon.py

@@ -4,6 +4,7 @@
     Material class
     Pint for units
 """
+
 import numpy as np
 
 kB = 1.380649e-23  # m2 kg s-2 K-1, boltzmann constant

gplugins/sax/read.py

@@ -1,4 +1,5 @@
 """read Sparameters from CSV file and returns sax model."""
+
 from __future__ import annotations
 
 import pathlib

gplugins/sax/tests/test_mzi.py

@@ -1,4 +1,5 @@
 """Demo of non-hierarchical circuit simulations."""
+
 from __future__ import annotations
 
 import gdsfactory as gf

gplugins/sax/tests/test_mzi_lattice.py

@@ -1,4 +1,5 @@
 """Test hierarchical circuit simulations."""
+
 from __future__ import annotations
 
 import gdsfactory as gf

gplugins/tidy3d/component.py

@@ -611,14 +611,14 @@ def write_sparameters(
             for port_out in s.port_out.values:
                 for mode_index_in in s.mode_index_in.values:
                     for mode_index_out in s.mode_index_out.values:
-                        sp[
-                            f"{port_in}@{mode_index_in},{port_out}@{mode_index_out}"
-                        ] = s.sel(
-                            port_in=port_in,
-                            port_out=port_out,
-                            mode_index_in=mode_index_in,
-                            mode_index_out=mode_index_out,
-                        ).values
+                        sp[f"{port_in}@{mode_index_in},{port_out}@{mode_index_out}"] = (
+                            s.sel(
+                                port_in=port_in,
+                                port_out=port_out,
+                                mode_index_in=mode_index_in,
+                                mode_index_out=mode_index_out,
+                            ).values
+                        )
 
         frequency = s.f.values
         sp["wavelengths"] = td.constants.C_0 / frequency

gplugins/tidy3d/get_simulation_grating_coupler.py

@@ -1,4 +1,5 @@
 """Returns tidy3d simulation from gdsfactory Component."""
+
 from __future__ import annotations
 
 import warnings