diff --git a/gdsfactory/circuitviz.py b/gdsfactory/circuitviz.py index 62a9cc74849..376aa9f7a1a 100644 --- a/gdsfactory/circuitviz.py +++ b/gdsfactory/circuitviz.py @@ -401,10 +401,10 @@ def viz_instance( } for layer, polys in polys_by_layer.items(): - xs = [[p.points[:, 0]] for p in polys] - ys = [[p.points[:, 1]] for p in polys] color_info = colors_by_ldt.get(layer) if color_info: + xs = [[p.points[:, 0]] for p in polys] + ys = [[p.points[:, 1]] for p in polys] lp = LayerPolygons( tag=instance_name, xs=xs, diff --git a/gdsfactory/read/from_yaml.py b/gdsfactory/read/from_yaml.py index d6f9ea488c3..223d987ca83 100644 --- a/gdsfactory/read/from_yaml.py +++ b/gdsfactory/read/from_yaml.py @@ -599,11 +599,7 @@ def from_yaml( settings = conf.get("settings", {}) - if "mode" in kwargs: - mode = kwargs.pop("mode") - else: - mode = "layout" - + mode = kwargs.pop("mode") if "mode" in kwargs else "layout" for key, value in kwargs.items(): if key not in settings: raise ValueError(f"{key!r} not in {settings.keys()}") diff --git a/gdsfactory/simulation/eme/meow_eme.py b/gdsfactory/simulation/eme/meow_eme.py index da2fc49defe..1b0353bc6a3 100644 --- a/gdsfactory/simulation/eme/meow_eme.py +++ b/gdsfactory/simulation/eme/meow_eme.py @@ -336,7 +336,7 @@ def compute_sparameters(self) -> Dict[str, np.ndarray]: if not self.overwrite: logger.info(f"Simulation loaded from {self.filepath!r}") return dict(np.load(self.filepath)) - elif self.overwrite: + else: self.filepath.unlink() start = time.time() @@ -374,8 +374,6 @@ def compute_sparameters(self) -> Dict[str, np.ndarray]: return sp - return sp - if __name__ == "__main__": c = gf.components.taper(length=10, width2=2) diff --git a/gdsfactory/simulation/fem/mode_solver.py b/gdsfactory/simulation/fem/mode_solver.py index acc41994cf6..a7922d730b7 100644 --- a/gdsfactory/simulation/fem/mode_solver.py +++ b/gdsfactory/simulation/fem/mode_solver.py @@ -68,7 +68,7 @@ def compute_cross_section_modes( epsilon[basis0.get_dofs(elements=layername)] = ( _ACTIVE_PDK.materials_index[layer.material](wl) ** 2 ) - if "background_tag" in kwargs.keys(): + if "background_tag" in kwargs: epsilon[basis0.get_dofs(elements=kwargs["background_tag"])] = ( _ACTIVE_PDK.materials_index[kwargs["background_tag"]](wl) ** 2 ) @@ -86,9 +86,6 @@ def compute_cross_section_modes( return lams, basis, xs - def plot_cross_section_modes(): - return True - if __name__ == "__main__": filtered_layerstack = LayerStack( @@ -105,12 +102,12 @@ def plot_cross_section_modes(): filtered_layerstack.layers["core"].thickness = 0.2 - resolutions = {} - resolutions["core"] = {"resolution": 0.02, "distance": 2} - resolutions["clad"] = {"resolution": 0.2, "distance": 1} - resolutions["box"] = {"resolution": 0.2, "distance": 1} - resolutions["slab90"] = {"resolution": 0.05, "distance": 1} - + resolutions = { + "core": {"resolution": 0.02, "distance": 2}, + "clad": {"resolution": 0.2, "distance": 1}, + "box": {"resolution": 0.2, "distance": 1}, + "slab90": {"resolution": 0.05, "distance": 1}, + } compute_cross_section_modes( cross_section="rib", layerstack=filtered_layerstack, diff --git a/gdsfactory/simulation/fem/test_mode_solver.py b/gdsfactory/simulation/fem/test_mode_solver.py index 565eec4c72c..686bfe3296b 100644 --- a/gdsfactory/simulation/fem/test_mode_solver.py +++ b/gdsfactory/simulation/fem/test_mode_solver.py @@ -21,12 +21,12 @@ def test_compute_cross_section_mode(): filtered_layerstack.layers["core"].thickness = 0.2 - resolutions = {} - resolutions["core"] = {"resolution": 0.02, "distance": 2} - resolutions["clad"] = {"resolution": 0.2, "distance": 1} - resolutions["box"] = {"resolution": 0.2, "distance": 1} - resolutions["slab90"] = {"resolution": 0.05, "distance": 1} - + resolutions = { + "core": {"resolution": 0.02, "distance": 2}, + "clad": {"resolution": 0.2, "distance": 1}, + "box": {"resolution": 0.2, "distance": 1}, + "slab90": {"resolution": 0.05, "distance": 1}, + } lams, basis, xs = compute_cross_section_modes( cross_section="rib", layerstack=filtered_layerstack, diff --git a/gdsfactory/simulation/gmsh/tests/test_meshing.py b/gdsfactory/simulation/gmsh/tests/test_meshing.py index eb9b3c9da1d..08157f71e35 100644 --- a/gdsfactory/simulation/gmsh/tests/test_meshing.py +++ b/gdsfactory/simulation/gmsh/tests/test_meshing.py @@ -23,11 +23,11 @@ def test_gmsh_uz_xsection_mesh(): } ) - resolutions = {} - resolutions["core"] = {"resolution": 0.05, "distance": 2} - resolutions["slab90"] = {"resolution": 0.03, "distance": 1} - resolutions["via_contact"] = {"resolution": 0.1, "distance": 1} - + resolutions = { + "core": {"resolution": 0.05, "distance": 2}, + "slab90": {"resolution": 0.03, "distance": 1}, + "via_contact": {"resolution": 0.1, "distance": 1}, + } uz_xsection_mesh( waveguide, [(4, -15), (4, 15)], @@ -35,7 +35,6 @@ def test_gmsh_uz_xsection_mesh(): resolutions=resolutions, background_tag="Oxide", ) - assert True def test_gmsh_xy_xsection_mesh(): @@ -57,10 +56,10 @@ def test_gmsh_xy_xsection_mesh(): } ) - resolutions = {} - resolutions["core"] = {"resolution": 0.05, "distance": 0.1} - resolutions["via_contact"] = {"resolution": 0.1, "distance": 0} - + resolutions = { + "core": {"resolution": 0.05, "distance": 0.1}, + "via_contact": {"resolution": 0.1, "distance": 0}, + } xy_xsection_mesh( component=waveguide, z=0.09, @@ -68,4 +67,3 @@ def test_gmsh_xy_xsection_mesh(): resolutions=resolutions, background_tag="Oxide", ) - assert True diff --git a/gdsfactory/technology/layer_views.py b/gdsfactory/technology/layer_views.py index aaef32e324e..10803f45955 100644 --- a/gdsfactory/technology/layer_views.py +++ b/gdsfactory/technology/layer_views.py @@ -216,9 +216,9 @@ def _build_xml_element(self, tag: str, name: str) -> ET.Element: "marked": str(self.marked).lower(), "xfill": str(self.xfill).lower(), "animation": self.animation, - "name": name - if not self.layer_in_name - else f"{name} {self.layer[0]}/{self.layer[1]}", + "name": f"{name} {self.layer[0]}/{self.layer[1]}" + if self.layer_in_name + else name, "source": f"{self.layer[0]}/{self.layer[1]}@1" if self.layer is not None else "*/*@*", @@ -359,7 +359,7 @@ def __init__(self, filepath: Optional[PathLike] = None, **data): logger.info( f"Importing LayerViews from KLayout layer properties file: {filepath}." ) - elif (filepath.suffix == ".yaml") or (filepath.suffix == ".yml"): + elif filepath.suffix in [".yaml", ".yml"]: lvs = LayerViews.from_yaml(layer_file=filepath) logger.info(f"Importing LayerViews from YAML file: {filepath}.") else: diff --git a/gdsfactory/tests/test_klayout/test_klayout_tech.py b/gdsfactory/tests/test_klayout/test_klayout_tech.py index 1c5f5819aa3..aa76ad6ba54 100644 --- a/gdsfactory/tests/test_klayout/test_klayout_tech.py +++ b/gdsfactory/tests/test_klayout/test_klayout_tech.py @@ -14,9 +14,9 @@ def test_klayout_tech_create( data_regression: DataRegressionFixture, check: bool = True ) -> None: - lyp = LayerDisplayProperties.from_lyp(str(PATH.klayout_lyp)) - if check: + lyp = LayerDisplayProperties.from_lyp(str(PATH.klayout_lyp)) + data_regression.check(lyp.dict())