gsim 0.0.2__py3-none-any.whl → 0.0.3__py3-none-any.whl

gsim/palace/driven.py

@@ -8,11 +8,8 @@ from __future__ import annotations
 
 import logging
 import tempfile
-import warnings
 from pathlib import Path
-
-logger = logging.getLogger(__name__)
-from typing import TYPE_CHECKING, Any, Literal
+from typing import Any, Literal
 
 from pydantic import BaseModel, ConfigDict, Field, PrivateAttr
 
@@ -29,8 +26,7 @@ from gsim.palace.models import (
     ValidationResult,
 )
 
-if TYPE_CHECKING:
-    from gdsfactory.component import Component
+logger = logging.getLogger(__name__)
 
 
 class DrivenSim(PalaceSimMixin, BaseModel):
@@ -95,91 +91,6 @@ class DrivenSim(PalaceSimMixin, BaseModel):
     _last_mesh_result: Any = PrivateAttr(default=None)
     _last_ports: list = PrivateAttr(default_factory=list)
 
-    # -------------------------------------------------------------------------
-    # Output directory
-    # -------------------------------------------------------------------------
-
-    def set_output_dir(self, path: str | Path) -> None:
-        """Set the output directory for mesh and config files.
-
-        Args:
-            path: Directory path for output files
-
-        Example:
-            >>> sim.set_output_dir("./palace-sim")
-        """
-        self._output_dir = Path(path)
-        self._output_dir.mkdir(parents=True, exist_ok=True)
-
-    @property
-    def output_dir(self) -> Path | None:
-        """Get the current output directory."""
-        return self._output_dir
-
-    # -------------------------------------------------------------------------
-    # Geometry methods
-    # -------------------------------------------------------------------------
-
-    def set_geometry(self, component: Component) -> None:
-        """Set the gdsfactory component for simulation.
-
-        Args:
-            component: gdsfactory Component to simulate
-
-        Example:
-            >>> sim.set_geometry(my_component)
-        """
-        self.geometry = Geometry(component=component)
-
-    @property
-    def component(self) -> Component | None:
-        """Get the current component (for backward compatibility)."""
-        return self.geometry.component if self.geometry else None
-
-    # Backward compatibility alias
-    @property
-    def _component(self) -> Component | None:
-        """Internal component access (backward compatibility)."""
-        return self.component
-
-    # -------------------------------------------------------------------------
-    # Stack methods
-    # -------------------------------------------------------------------------
-
-    def set_stack(
-        self,
-        *,
-        yaml_path: str | Path | None = None,
-        air_above: float = 200.0,
-        substrate_thickness: float = 2.0,
-        include_substrate: bool = False,
-        **kwargs,
-    ) -> None:
-        """Configure the layer stack.
-
-        If yaml_path is provided, loads stack from YAML file.
-        Otherwise, extracts from active PDK with given parameters.
-
-        Args:
-            yaml_path: Path to custom YAML stack file
-            air_above: Air box height above top metal in um
-            substrate_thickness: Thickness below z=0 in um
-            include_substrate: Include lossy silicon substrate
-            **kwargs: Additional args passed to extract_layer_stack
-
-        Example:
-            >>> sim.set_stack(air_above=300.0, substrate_thickness=2.0)
-        """
-        self._stack_kwargs = {
-            "yaml_path": yaml_path,
-            "air_above": air_above,
-            "substrate_thickness": substrate_thickness,
-            "include_substrate": include_substrate,
-            **kwargs,
-        }
-        # Stack will be resolved lazily during mesh() or simulate()
-        self.stack = None
-
     # -------------------------------------------------------------------------
     # Port methods
     # -------------------------------------------------------------------------
@@ -210,7 +121,9 @@ class DrivenSim(PalaceSimMixin, BaseModel):
 
         Example:
             >>> sim.add_port("o1", layer="topmetal2", length=5.0)
-            >>> sim.add_port("feed", from_layer="metal1", to_layer="topmetal2", geometry="via")
+            >>> sim.add_port(
+            ...     "feed", from_layer="metal1", to_layer="topmetal2", geometry="via"
+            ... )
         """
         # Remove existing config for this port if any
         self.ports = [p for p in self.ports if p.name != name]
@@ -318,88 +231,11 @@ class DrivenSim(PalaceSimMixin, BaseModel):
             excitation_port=excitation_port,
         )
 
-    # -------------------------------------------------------------------------
-    # Material methods
-    # -------------------------------------------------------------------------
-
-    def set_material(
-        self,
-        name: str,
-        *,
-        type: Literal["conductor", "dielectric", "semiconductor"] | None = None,
-        conductivity: float | None = None,
-        permittivity: float | None = None,
-        loss_tangent: float | None = None,
-    ) -> None:
-        """Override or add material properties.
-
-        Args:
-            name: Material name
-            type: Material type (conductor, dielectric, semiconductor)
-            conductivity: Conductivity in S/m (for conductors)
-            permittivity: Relative permittivity (for dielectrics)
-            loss_tangent: Dielectric loss tangent
-
-        Example:
-            >>> sim.set_material("aluminum", type="conductor", conductivity=3.8e7)
-            >>> sim.set_material("sio2", type="dielectric", permittivity=3.9)
-        """
-        # Determine type if not provided
-        if type is None:
-            if conductivity is not None and conductivity > 1e4:
-                type = "conductor"
-            elif permittivity is not None:
-                type = "dielectric"
-            else:
-                type = "dielectric"
-
-        self.materials[name] = MaterialConfig(
-            type=type,
-            conductivity=conductivity,
-            permittivity=permittivity,
-            loss_tangent=loss_tangent,
-        )
-
-    def set_numerical(
-        self,
-        *,
-        order: int = 2,
-        tolerance: float = 1e-6,
-        max_iterations: int = 400,
-        solver_type: Literal["Default", "SuperLU", "STRUMPACK", "MUMPS"] = "Default",
-        preconditioner: Literal["Default", "AMS", "BoomerAMG"] = "Default",
-        device: Literal["CPU", "GPU"] = "CPU",
-        num_processors: int | None = None,
-    ) -> None:
-        """Configure numerical solver parameters.
-
-        Args:
-            order: Finite element order (1-4)
-            tolerance: Linear solver tolerance
-            max_iterations: Maximum solver iterations
-            solver_type: Linear solver type
-            preconditioner: Preconditioner type
-            device: Compute device (CPU or GPU)
-            num_processors: Number of processors (None = auto)
-
-        Example:
-            >>> sim.set_numerical(order=3, tolerance=1e-8)
-        """
-        self.numerical = NumericalConfig(
-            order=order,
-            tolerance=tolerance,
-            max_iterations=max_iterations,
-            solver_type=solver_type,
-            preconditioner=preconditioner,
-            device=device,
-            num_processors=num_processors,
-        )
-
     # -------------------------------------------------------------------------
     # Validation
     # -------------------------------------------------------------------------
 
-    def validate(self) -> ValidationResult:
+    def validate_config(self) -> ValidationResult:
         """Validate the simulation configuration.
 
         Returns:
@@ -428,22 +264,21 @@ class DrivenSim(PalaceSimMixin, BaseModel):
             # Validate port configurations
             for port in self.ports:
                 if port.geometry == "inplane" and port.layer is None:
+                    errors.append(f"Port '{port.name}': inplane ports require 'layer'")
+                if port.geometry == "via" and (
+                    port.from_layer is None or port.to_layer is None
+                ):
                     errors.append(
-                        f"Port '{port.name}': inplane ports require 'layer'"
+                        f"Port '{port.name}': via ports require "
+                        "'from_layer' and 'to_layer'"
                     )
-                if port.geometry == "via":
-                    if port.from_layer is None or port.to_layer is None:
-                        errors.append(
-                            f"Port '{port.name}': via ports require "
-                            "'from_layer' and 'to_layer'"
-                        )
 
             # Validate CPW ports
-            for cpw in self.cpw_ports:
-                if not cpw.layer:
-                    errors.append(
-                        f"CPW port ({cpw.upper}, {cpw.lower}): 'layer' is required"
-                    )
+            errors.extend(
+                f"CPW port ({cpw.upper}, {cpw.lower}): 'layer' is required"
+                for cpw in self.cpw_ports
+                if not cpw.layer
+            )
 
         # Validate excitation port if specified
         if self.driven.excitation_port is not None:
@@ -463,30 +298,7 @@ class DrivenSim(PalaceSimMixin, BaseModel):
     # Internal helpers
     # -------------------------------------------------------------------------
 
-    def _resolve_stack(self) -> LayerStack:
-        """Resolve the layer stack from PDK or YAML.
-
-        Returns:
-            Legacy LayerStack object for mesh generation
-        """
-        from gsim.common.stack import get_stack
-
-        yaml_path = self._stack_kwargs.pop("yaml_path", None)
-        legacy_stack = get_stack(yaml_path=yaml_path, **self._stack_kwargs)
-
-        # Restore yaml_path for potential re-resolution
-        self._stack_kwargs["yaml_path"] = yaml_path
-
-        # Apply material overrides
-        for name, props in self.materials.items():
-            legacy_stack.materials[name] = props.to_dict()
-
-        # Store the LayerStack
-        self.stack = legacy_stack
-
-        return legacy_stack
-
-    def _configure_ports_on_component(self, stack: LayerStack) -> None:
+    def _configure_ports_on_component(self, stack: LayerStack) -> None:  # noqa: ARG002
         """Configure ports on the component using legacy functions."""
         from gsim.palace.ports import (
             configure_cpw_port,
@@ -516,7 +328,7 @@ class DrivenSim(PalaceSimMixin, BaseModel):
                     f"Available ports: {[p.name for p in component.ports]}"
                 )
 
-            if port_config.geometry == "inplane":
+            if port_config.geometry == "inplane" and port_config.layer is not None:
                 configure_inplane_port(
                     gf_port,
                     layer=port_config.layer,
@@ -524,7 +336,9 @@ class DrivenSim(PalaceSimMixin, BaseModel):
                     impedance=port_config.impedance,
                     excited=port_config.excited,
                 )
-            elif port_config.geometry == "via":
+            elif port_config.geometry == "via" and (
+                port_config.from_layer is not None and port_config.to_layer is not None
+            ):
                 configure_via_port(
                     gf_port,
                     from_layer=port_config.from_layer,
@@ -571,60 +385,6 @@ class DrivenSim(PalaceSimMixin, BaseModel):
 
         self._configured_ports = True
 
-    def _build_mesh_config(
-        self,
-        preset: Literal["coarse", "default", "fine"] | None,
-        refined_mesh_size: float | None,
-        max_mesh_size: float | None,
-        margin: float | None,
-        air_above: float | None,
-        fmax: float | None,
-        show_gui: bool,
-    ) -> MeshConfig:
-        """Build mesh config from preset with optional overrides."""
-        # Build mesh config from preset
-        if preset == "coarse":
-            mesh_config = MeshConfig.coarse()
-        elif preset == "fine":
-            mesh_config = MeshConfig.fine()
-        else:
-            mesh_config = MeshConfig.default()
-
-        # Track overrides for warning
-        overrides = []
-        if preset is not None:
-            if refined_mesh_size is not None:
-                overrides.append(f"refined_mesh_size={refined_mesh_size}")
-            if max_mesh_size is not None:
-                overrides.append(f"max_mesh_size={max_mesh_size}")
-            if margin is not None:
-                overrides.append(f"margin={margin}")
-            if air_above is not None:
-                overrides.append(f"air_above={air_above}")
-            if fmax is not None:
-                overrides.append(f"fmax={fmax}")
-
-            if overrides:
-                warnings.warn(
-                    f"Preset '{preset}' values overridden by: {', '.join(overrides)}",
-                    stacklevel=4,
-                )
-
-        # Apply overrides
-        if refined_mesh_size is not None:
-            mesh_config.refined_mesh_size = refined_mesh_size
-        if max_mesh_size is not None:
-            mesh_config.max_mesh_size = max_mesh_size
-        if margin is not None:
-            mesh_config.margin = margin
-        if air_above is not None:
-            mesh_config.air_above = air_above
-        if fmax is not None:
-            mesh_config.fmax = fmax
-        mesh_config.show_gui = show_gui
-
-        return mesh_config
-
     def _generate_mesh_internal(
         self,
         output_dir: Path,
@@ -731,11 +491,9 @@ class DrivenSim(PalaceSimMixin, BaseModel):
         component = self.geometry.component if self.geometry else None
 
         # Validate configuration
-        validation = self.validate()
+        validation = self.validate_config()
         if not validation.valid:
-            raise ValueError(
-                f"Invalid configuration:\n" + "\n".join(validation.errors)
-            )
+            raise ValueError("Invalid configuration:\n" + "\n".join(validation.errors))
 
         # Build mesh config
         mesh_config = self._build_mesh_config(
@@ -776,38 +534,6 @@ class DrivenSim(PalaceSimMixin, BaseModel):
                 config=legacy_mesh_config,
             )
 
-    # -------------------------------------------------------------------------
-    # Convenience methods
-    # -------------------------------------------------------------------------
-
-    def show_stack(self) -> None:
-        """Print the layer stack table.
-
-        Example:
-            >>> sim.show_stack()
-        """
-        from gsim.common.stack import print_stack_table
-
-        if self.stack is None:
-            self._resolve_stack()
-
-        if self.stack is not None:
-            print_stack_table(self.stack)
-
-    def plot_stack(self) -> None:
-        """Plot the layer stack visualization.
-
-        Example:
-            >>> sim.plot_stack()
-        """
-        from gsim.common.stack import plot_stack
-
-        if self.stack is None:
-            self._resolve_stack()
-
-        if self.stack is not None:
-            plot_stack(self.stack)
-
     # -------------------------------------------------------------------------
     # Mesh generation
     # -------------------------------------------------------------------------
@@ -873,11 +599,9 @@ class DrivenSim(PalaceSimMixin, BaseModel):
         )
 
         # Validate configuration
-        validation = self.validate()
+        validation = self.validate_config()
         if not validation.valid:
-            raise ValueError(
-                f"Invalid configuration:\n" + "\n".join(validation.errors)
-            )
+            raise ValueError("Invalid configuration:\n" + "\n".join(validation.errors))
 
         output_dir = self._output_dir