gplugins 1.1.2__tar.gz → 1.1.4__tar.gz

{gplugins-1.1.2 → gplugins-1.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.1
+Metadata-Version: 2.3
 Name: gplugins
-Version: 1.1.2
+Version: 1.1.4
 Summary: gdsfactory plugins
 Keywords: python
 Author-email: gdsfactory <contact@gdsfactory.com>
@@ -14,6 +14,7 @@ Requires-Dist: gdsfactory>=8.5.3
 Requires-Dist: pint
 Requires-Dist: gdstk
 Requires-Dist: tqdm
+Requires-Dist: numpy<2
 Requires-Dist: dagster ; extra == "dagster"
 Requires-Dist: dagit ; extra == "dagster"
 Requires-Dist: pre-commit ; extra == "dev"
@@ -34,7 +35,7 @@ Requires-Dist: tidy3d>=2.5.2,<2.8 ; extra == "devsim"
 Requires-Dist: jupytext ; extra == "docs"
 Requires-Dist: matplotlib ; extra == "docs"
 Requires-Dist: jupyter-book~=1.0 ; extra == "docs"
-Requires-Dist: pyvista[all,trame]<=0.43.8 ; extra == "docs"
+Requires-Dist: pyvista[all, trame]<=0.43.8 ; extra == "docs"
 Requires-Dist: femwell~=0.1.11 ; extra == "femwell"
 Requires-Dist: meshwell~=1.0.7 ; extra == "femwell"
 Requires-Dist: jinja2 ; extra == "gfviz"
@@ -52,6 +53,8 @@ Requires-Dist: shapely ; extra == "gmsh"
 Requires-Dist: meshwell~=1.0.7 ; extra == "gmsh"
 Requires-Dist: klayout ; extra == "klayout"
 Requires-Dist: pyvis<=0.3.1 ; extra == "klayout"
+Requires-Dist: vlsir~=6.0 ; extra == "klayout"
+Requires-Dist: vlsirtools~=6.0 ; extra == "klayout"
 Requires-Dist: jax>=0.4.26 ; extra == "meow"
 Requires-Dist: jaxlib>=0.4.26 ; extra == "meow"
 Requires-Dist: flax>=0.8.2 ; extra == "meow"
@@ -60,7 +63,7 @@ Requires-Dist: tidy3d>=2.5.2,<2.8 ; extra == "meow"
 Requires-Dist: jax>=0.4.26 ; extra == "sax"
 Requires-Dist: jaxlib>=0.4.26 ; extra == "sax"
 Requires-Dist: flax>=0.8.2 ; extra == "sax"
-Requires-Dist: sax~=0.13.1 ; extra == "sax"
+Requires-Dist: sax>=0.13.1,<0.15.0 ; extra == "sax"
 Requires-Dist: scikit-learn ; extra == "sax"
 Requires-Dist: pyvis<=0.3.1 ; extra == "sax"
 Requires-Dist: bokeh ; extra == "schematic"
@@ -68,6 +71,7 @@ Requires-Dist: ipywidgets ; extra == "schematic"
 Requires-Dist: natsort ; extra == "schematic"
 Requires-Dist: tidy3d>=2.5.2,<2.8 ; extra == "tidy3d"
 Requires-Dist: meshio ; extra == "tidy3d"
+Requires-Dist: numpy<2 ; extra == "tidy3d"
 Requires-Dist: meshwell~=1.0.7 ; extra == "tidy3d"
 Requires-Dist: vlsir~=6.0 ; extra == "vlsir"
 Requires-Dist: vlsirtools~=6.0 ; extra == "vlsir"
@@ -85,7 +89,7 @@ Provides-Extra: schematic
 Provides-Extra: tidy3d
 Provides-Extra: vlsir
 
-# gplugins 1.1.2
+# gplugins 1.1.4
 
 [![docs](https://github.com/gdsfactory/gplugins/actions/workflows/pages.yml/badge.svg)](https://gdsfactory.github.io/gplugins/)
 [![PyPI](https://img.shields.io/pypi/v/gplugins)](https://pypi.org/project/gplugins/)
@@ -109,7 +113,7 @@ gdsfactory plugins:
 - `mpb` for MPB mode solver.
 - `elmer` for electrostatic (capacitive) simulations.
 - `palace` for full-wave driven (S parameter) and electrostatic (capacitive) simulations.
-- `vlsir` for parsing GDS-extracted circuit netlists into Spice, Spectre and Xyce Schematic File formats.
+- `vlsir` for parsing GDS-extracted circuit netlists into Cadence Spectre, NgSpice and Xyce Schematic File formats.
 
 ## Installation
 

{gplugins-1.1.2 → gplugins-1.1.4}/README.md

@@ -1,4 +1,4 @@
-# gplugins 1.1.2
+# gplugins 1.1.4
 
 [![docs](https://github.com/gdsfactory/gplugins/actions/workflows/pages.yml/badge.svg)](https://gdsfactory.github.io/gplugins/)
 [![PyPI](https://img.shields.io/pypi/v/gplugins)](https://pypi.org/project/gplugins/)
@@ -22,7 +22,7 @@ gdsfactory plugins:
 - `mpb` for MPB mode solver.
 - `elmer` for electrostatic (capacitive) simulations.
 - `palace` for full-wave driven (S parameter) and electrostatic (capacitive) simulations.
-- `vlsir` for parsing GDS-extracted circuit netlists into Spice, Spectre and Xyce Schematic File formats.
+- `vlsir` for parsing GDS-extracted circuit netlists into Cadence Spectre, NgSpice and Xyce Schematic File formats.
 
 ## Installation
 

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/__init__.py

@@ -1,6 +1,6 @@
-"""gplugins - gdsfactory plugins"""
+"""gplugins - gdsfactory plugins."""
 
-__version__ = "1.1.2"
+__version__ = "1.1.4"
 
 import pathlib
 

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/common/base_models/component.py

@@ -39,6 +39,7 @@ class LayeredComponentBase(BaseModel):
     slice_stack: tuple[int, int | None] = (0, None)
 
     def __hash__(self):
+        """Returns a hash of the model dump."""
         if not hasattr(self, "_hash"):
             dump = str.encode(self.model_dump_json())
             self._hash = int(md5(dump).hexdigest()[:15], 16)

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/common/utils/async_helpers.py

@@ -48,7 +48,8 @@ async def execute_and_stream_output(
     stream_stderr: io.TextIOWrapper | None = sys.stderr,
     **kwargs,
 ) -> asyncio.subprocess.Process:
-    """Run a command asynchronously and stream *stdout* and *stderr* to given IO and a log file
+    """Run a command asynchronously and stream *stdout* and *stderr* to given IO and a log file.
+
     in ``log_file_dir / log_file_str``. Uses ``shell=True`` as default unlike ``subprocess.Popen``. Returns an asyncio process.
 
     Args:
@@ -59,6 +60,8 @@ async def execute_and_stream_output(
         log_file_str: Log file name. Will be expanded to ``f'{log_file_str}_out.log'`` and ``f'{log_file_str}_err.log'``.
         stream_stdout: Stream to write stdout to. Defaults to ``sys.stdout``.
         stream_stderr: Stream to write stderr to. Defaults to ``sys.stderr``.
+        *args: Additional arguments to pass to :func:`~
+        **kwargs: Additional keyword arguments to pass to :func:`~
 
     ``*args`` and ``**kwargs`` are passed to :func:`~create_subprocess_shell` or :func:`create_subprocess_exec`,
     which in turn passes them to :class:`subprocess.Popen`.
@@ -102,8 +105,9 @@ async def execute_and_stream_output(
 
 
 def run_async_with_event_loop(coroutine: Coroutine[Any, Any, T] | Awaitable[T]) -> T:
-    """Run a coroutine within an asyncio event loop, either by adding it to the
-    existing running event loop or by creating a new event loop. Returns the result.
+    """Run a coroutine within an asyncio event loop.
+
+    Either by adding it to the existing running event loop or by creating a new event loop. Returns the result.
 
     Args:
         coroutine: The coroutine (async function) to be executed.
@@ -120,7 +124,6 @@ def run_async_with_event_loop(coroutine: Coroutine[Any, Any, T] | Awaitable[T])
 
         run_async_with_event_loop(main())
     """
-
     try:
         loop = asyncio.get_running_loop()
         try:

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/common/utils/disable_print.py

@@ -7,7 +7,7 @@ import sys
 
 
 class DisablePrint:
-    def __init__(self):
+    def __init__(self) -> None:
         self.output = sys.stdout
 
     def __enter__(self) -> DisablePrint:

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/common/utils/get_component_with_local_layers.py

@@ -3,8 +3,9 @@ from dataclasses import dataclass
 
 import gdsfactory as gf
 import gdstk
+from gdsfactory.component import Component
 from gdsfactory.pdk import get_layer
-from gdsfactory.typings import Component, Layer, List
+from gdsfactory.typings import Layer
 from gdstk import Polygon
 
 
@@ -22,13 +23,13 @@ class LocalMapping:
     new_layer_name: str
     new_layer_number: Layer
     old_layer_name: str
-    domains: List[Polygon]
+    domains: list[Polygon]
 
 
 def get_component_with_local_layers(
     component,
     layer_stack,
-    mappings: List[LocalMapping],
+    mappings: list[LocalMapping],
     precision: float = 1e-4,
 ) -> Component:
     """Returns a component where polygons within "domains" belonging to "old_layer_name" are remapped to "new_layer_name" (with layer details copied from old_layer), and polygons outside the domain are kept on "old_layer_name".

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/common/utils/get_component_with_net_layers.py

@@ -1,7 +1,7 @@
 import copy
 
 import gdsfactory as gf
-import gdstk
+from gdsfactory.technology import LogicalLayer
 from gdsfactory.typings import Component, LayerStack
 
 
@@ -55,10 +55,9 @@ def get_component_with_net_layers(
         new_layers_init: initial layer number for the temporary new layers.
         add_to_layerstack: True by default, but can be set to False to disable parsing of the layerstack.
     """
-
-    return NotImplementedError(
-        "Meshing component with net layers not implemented in gdsfactory8 yet."
-    )
+    # return NotImplementedError(
+    #     "Meshing component with net layers not implemented in gdsfactory8 yet."
+    # )
 
     # Initialize returned component
     net_component = component.copy()
@@ -67,17 +66,23 @@ def get_component_with_net_layers(
     for i, portname in enumerate(port_names):
         port = component.ports[portname]
         # Get original port layer polygons, and modify a new component without that layer
-        polygons = net_component.extract(layers=[port.layer]).get_polygons()
-        net_component = net_component.remove_layers(layers=[port.layer])
+        polygons = (
+            net_component.extract(layers=(port.layer,))
+            .get_polygons()
+            .get(port.layer, [])
+        )
+        net_component = net_component.remove_layers(layers=(port.layer,))
         for polygon in polygons:
             # If polygon belongs to port, create a unique new layer, and add the polygon to it
-
-            if gdstk.inside(
-                [port.center],
-                gdstk.offset(gdstk.Polygon(polygon), gf.get_active_pdk().grid_size),
-            )[0]:
+            if polygon.sized(3 * gf.kcl.dbu).inside(port.center):
+                # if gdstk.inside(
+                #     [port.center],
+                #     gdstk.offset(gdstk.Polygon(polygon), gf.get_active_pdk().grid_size),
+                # )[0]:
                 try:
-                    port_layernames = layer_stack.get_layer_to_layername()[port.layer]
+                    port_layernames = layer_stack.get_layer_to_layername()[
+                        LogicalLayer(layer=port.layer)
+                    ]
                 except KeyError as e:
                     raise KeyError(
                         "Make sure your `layer_stack` contains all layers with ports"
@@ -89,10 +94,13 @@ def get_component_with_net_layers(
                     )
                     if add_to_layerstack:
                         new_layer = copy.deepcopy(layer_stack.layers[old_layername])
-                        new_layer.layer = (
-                            new_layers_init[0] + i,
-                            new_layers_init[1] + j,
+                        new_layer.layer = LogicalLayer(
+                            layer=(
+                                new_layers_init[0] + i,
+                                new_layers_init[1] + j,
+                            )
                         )
+                        new_layer.name = f"{old_layername}{delimiter}{portname}"
                         layer_stack.layers[f"{old_layername}{delimiter}{portname}"] = (
                             new_layer
                         )

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/common/utils/optical_constants.py

@@ -30,7 +30,7 @@ def permittivity_imag_from_index(n, k):
 
 
 def D_conductivity_um(n, k, wavelength):
-    """conductivity as defined by Meep https://meep.readthedocs.io/en/latest/Materials/#conductivity-and-complex
+    """Conductivity as defined by Meep https://meep.readthedocs.io/en/latest/Materials/#conductivity-and-complex.
 
     Assumes natural units, with lengthscale in microns (see https://meep.readthedocs.io/en/latest/Introduction/#units-in-meep)
 

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/common/utils/plot.py

@@ -10,7 +10,7 @@ import matplotlib.pyplot as plt
 import numpy as np
 
 
-def _check_ports(sp: dict[str, np.ndarray], ports: Sequence[str]):
+def _check_ports(sp: dict[str, np.ndarray], ports: Sequence[str]) -> None:
     """Ensure ports exist in Sparameters."""
     for port in ports:
         if port not in sp:

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/common/utils/tests/test_get_component_with_new_port_layers.py

@@ -11,7 +11,7 @@ layernames_before = set(LAYER_STACK.layers.keys())
 original_component = straight_heater_metal()
 
 
-def test_component_with_new_port_layers():
+def test_component_with_new_port_layers() -> None:
     layer_stack = LAYER_STACK.model_copy()
     get_component_with_net_layers(
         component=original_component,
@@ -31,7 +31,7 @@ def test_component_with_new_port_layers():
     # print(original_component.get_layers())
 
 
-def test_remove_empty_layer_stack_layers():
+def test_remove_empty_layer_stack_layers() -> None:
     layer_stack = LAYER_STACK.model_copy()
     new_component = get_component_with_net_layers(
         component=original_component,

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/elmer/get_capacitance.py

@@ -40,7 +40,7 @@ def _generate_sif(
     element_order: int,
     background_tag: str | None = None,
     simulator_params: Mapping[str, Any] | None = None,
-):
+) -> None:
     # pylint: disable=unused-argument
     """Generates a sif file for Elmer simulations using Jinja2."""
     # Have background_tag as first s.t. unaccounted elements use it by default
@@ -61,7 +61,7 @@ def _generate_sif(
         fp.write(output)
 
 
-def _elmergrid(simulation_folder: Path, name: str, n_processes: int = 1):
+def _elmergrid(simulation_folder: Path, name: str, n_processes: int = 1) -> None:
     """Run ElmerGrid for converting gmsh mesh to Elmer format."""
     elmergrid = shutil.which("ElmerGrid")
     if elmergrid is None:
@@ -99,7 +99,7 @@ def _elmergrid(simulation_folder: Path, name: str, n_processes: int = 1):
         )
 
 
-def _elmersolver(simulation_folder: Path, name: str, n_processes: int = 1):
+def _elmersolver(simulation_folder: Path, name: str, n_processes: int = 1) -> None:
     """Run simulations with ElmerFEM."""
     elmersolver_name = (
         "ElmerSolver" if (no_mpi := n_processes == 1) else "ElmerSolver_mpi"
@@ -193,7 +193,6 @@ def run_capacitive_simulation_elmer(
 
     .. _Elmer: https://github.com/ElmerCSC/elmerfem
     """
-
     if layer_stack is None:
         layer_stack = LayerStack(
             layers={

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/elmer/tests/test_elmer.py

@@ -83,7 +83,7 @@ def get_reasonable_mesh_parameters(c: Component):
 
 @pytest.fixture(scope="session")
 def elmer_capacitance_simulation_basic_results(geometry) -> ElectrostaticResults:
-    """Run a Elmer capacitance simulation and cache the results"""
+    """Run a Elmer capacitance simulation and cache the results."""
     c = geometry
     return run_capacitive_simulation_elmer(
         c,
@@ -100,7 +100,7 @@ def test_elmer_capacitance_simulation_runs(
 
 
 @pytest.mark.parametrize("n_processes", [(1), (2)])
-def test_elmer_capacitance_simulation_n_processes(geometry, n_processes):
+def test_elmer_capacitance_simulation_n_processes(geometry, n_processes) -> None:
     c = geometry
     run_capacitive_simulation_elmer(
         c,

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/fdtdz/get_epsilon_fdtdz.py

@@ -123,7 +123,8 @@ def component_to_epsilon_pjz(
 
 def component_to_epsilon_femwell():
     """TODO: Uses gdsfactory meshing + femwell physical tagging + export to a cartesian\
-            grid to define the (3, xx, yy, zz) array of permittivity values"""
+            grid to define the (3, xx, yy, zz) array of permittivity values.
+    """
     return NotImplementedError
 
 
@@ -154,7 +155,6 @@ def plot_epsilon(
         nm_per_pixel: (int) resolution (default to 1000 for pixel index).
         figsize: figure size.
     """
-
     # Checks
     if (x and y) or (y and z) or (x and z):
         raise ValueError("Only one of x, y or z must be numeric!")
@@ -207,7 +207,7 @@ def plot_epsilon(
     return fig
 
 
-def add_plot_labels(arg0, arg1, arg2, arg3):
+def add_plot_labels(arg0, arg1, arg2, arg3) -> None:
     plt.xlabel(arg0)
     plt.ylabel(arg1)
     plt.title(f"{arg2}{arg3}")

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/fdtdz/get_ports_fdtdz.py

@@ -17,8 +17,7 @@ def get_epsilon_port(
     port_extent_xy: float = 1,
     port_offset: int = 0,
 ):
-    """
-    This function extracts a xz or yz slice of the epsilon distribution at the location of the port to mode solve.
+    """This function extracts a xz or yz slice of the epsilon distribution at the location of the port to mode solve.
 
     Parameters:
         port (Port): The port object from the component.
@@ -32,7 +31,6 @@ def get_epsilon_port(
     Returns:
         port_slice (array): The slice of the epsilon distribution at the location of the port.
     """
-
     xarray, yarray, zarray = create_physical_grid(
         xmin, ymin, zmin, epsilon, nm_per_pixel
     )
@@ -119,7 +117,6 @@ def plot_mode(
         nm_per_pixel: (int) resolution (default to 1000 for pixel index)
         figsize: figure size.
     """
-
     # Create physical grid
     xarray, yarray, zarray = create_physical_grid(
         xmin, ymin, zmin, epsilon_port, nm_per_pixel

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/fdtdz/get_sparameters_fdtdz.py

@@ -91,7 +91,6 @@ def get_sparameters_fdtdz(
         sp = gz.get_sparameters(c, run=False)
 
     """
-
     # Checks from gmeep
     component_ref = component.ref()
     component_ref.dx = 0

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gfviz/netlist.py

@@ -293,7 +293,7 @@ def wrap_component_in_netlist(name):
     }
 
 
-def _bad_instance(_):
+def _bad_instance(_) -> bool:
     return False
 
 

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmeep/get_meep_geometry.py

@@ -44,12 +44,15 @@ def get_meep_geometry_from_component(
     layer_to_polygons = component_with_booleans.get_polygons_points()
 
     ordered_layer_stack_keys = order_layer_stack(layer_stack)[::-1]
+
     for layername in ordered_layer_stack_keys:
         layer = layer_stack.layers[layername].layer
 
         if layer not in layer_to_polygons:
             continue
         polygons = layer_to_polygons[layer]
+        print(f"layer: {layer}, polygons: {polygons}")
+
         if layer in layer_to_thickness and layer in layer_to_material:
             height = layer_to_thickness[layer] if is_3d else mp.inf
             zmin_um = layer_to_zmin[layer] if is_3d else 0

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmeep/get_simulation.py

@@ -270,7 +270,7 @@ def get_simulation(
 
     # Add port monitors dict
     monitors = {}
-    for port in component_ref:
+    for port in component_ref.ports:
         port_name = port.name
         angle_rad = np.radians(port.orientation)
         width = port.width + 2 * port_margin

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmeep/meep_adjoint_optimization.py

@@ -137,7 +137,7 @@ def get_meep_adjoint_optimizer(
         for monitor in monitors.values()
     ]
 
-    c = component.copy()
+    c = component.dup()
     for design_region, design_variable in zip(design_regions, design_variables):
         sim.geometry.append(
             Block(design_region.size, design_region.center, material=design_variable)

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmeep/test_eigenmode.py

@@ -1,5 +1,6 @@
 """Compares the modes of a gdsfactory + MEEP waveguide cross-section vs a
-direct MPB calculation."""
+direct MPB calculation.
+"""
 
 from __future__ import annotations
 
@@ -21,7 +22,7 @@ def lumerical_parser(E_1D, H_1D, y_1D, z_1D, res=50, z_offset=0.11 * 1e-6):
 
     Lumerical data is in 1D arrays, and over a nonregular mesh
 
-    Args
+    Args:
         E_1D: E array from Lumerical.
         H_1D: H array from Lumerical.
         y_1D: y array from Lumerical.
@@ -114,9 +115,8 @@ def MPB_eigenmode_toDisk() -> None:
 
 
 def compare_mpb_lumerical(plot=False) -> None:
-    """
-    WARNING: Segmentation fault occurs if both ms object above and sim object exist in memory at the same time
-    Instead load results from separate MPB run
+    """WARNING: Segmentation fault occurs if both ms object above and sim object exist in memory at the same time
+    Instead load results from separate MPB run.
 
     Same namespace run does not work
     # MPB mode

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmeep/test_materials.py

@@ -6,7 +6,6 @@ from gplugins.gmeep.write_sparameters_meep import write_sparameters_meep
 
 def test_materials_override() -> None:
     """Checks that materials are properly overridden if index is provided."""
-
     c = gf.components.straight(length=2)
 
     # Default (materials strings)
@@ -34,7 +33,6 @@ def test_materials_override() -> None:
 
 def test_materials_override_complex() -> None:
     """Checks that materials are properly overridden if complex index is provided."""
-
     c = gf.components.straight(length=2)
 
     # Default (materials strings)

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmeep/write_sparameters_meep.py

@@ -251,7 +251,7 @@ def write_sparameters_meep(
         plot_args: if animate or not run, customization keyword arguments passed to
           `plot2D()` (i.e. `labels`, `eps_parameters`, `boundary_parameters`, `field_parameters`, etc.)
 
-    keyword Args:
+    Keyword Args:
         extend_ports_length: to extend ports beyond the PML (um).
         zmargin_top: thickness for cladding above core (um).
         zmargin_bot: thickness for cladding below core (um).

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmeep/write_sparameters_meep_batch.py

@@ -50,7 +50,7 @@ def write_sparameters_meep_batch(
     If there are more simulations than cores each batch runs sequentially.
 
 
-    Args
+    Args:
         jobs: list of Dicts containing the simulation settings for each job.
             for write_sparameters_meep.
         cores_per_run: number of processors to assign to each component simulation.
@@ -61,7 +61,7 @@ def write_sparameters_meep_batch(
         layer_stack: contains layer to thickness, zmin and material.
             Defaults to active pdk.layer_stack.
 
-    keyword Args:
+    Keyword Args:
         resolution: in pixels/um (30: for coarse, 100: for fine).
         port_symmetries: Dict to specify port symmetries, to save number of simulations.
         dirpath: directory to store Sparameters.

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmeep/write_sparameters_meep_mpi.py

@@ -123,7 +123,7 @@ def write_sparameters_meep_mpi(
         filepath for sparameters CSV (wavelengths, s11a, o1@0,o2@0, ...)
             where `a` is the angle in radians and `m` the module.
 
-    TODO:
+    Todo:
         write stdout to file, maybe simulation logs too.
 
     """

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmsh/get_mesh.py

@@ -55,7 +55,6 @@ def get_mesh(
 
     TODO! make compatible with new unified plugins interface
     """
-
     # Add WAFER layer:
     padded_component = Component()
     component = gf.get_component(component)

{gplugins-1.1.2 → gplugins-1.1.4}/gplugins/gmsh/parse_gds.py

@@ -24,7 +24,6 @@ def round_coordinates(geom, ndigits=4):
 
 def fuse_polygons(component, layer, round_tol=4, simplify_tol=1e-4, offset_tol=None):
     """Take all polygons from a layer, and returns a single (Multi)Polygon shapely object."""
-
     layer_region = layer.get_shapes(component)
 
     # Convert polygons to shapely
@@ -37,7 +36,7 @@ def fuse_polygons(component, layer, round_tol=4, simplify_tol=1e-4, offset_tol=N
             exterior_points.append((point.x / 1000, point.y / 1000))
         for hole_index in range(klayout_polygon.holes()):
             holes_points = []
-            for point in layer_region.each_polygon_hole(hole_index):
+            for point in klayout_polygon.each_point_hole(hole_index):
                 holes_points.append((point.x / 1000, point.y / 1000))
             interior_points.append(holes_points)