fix(rf): add warning if port mesh refinement is incompatible with GridSpec

Author: dmarek-flex

CHANGELOG.md

@@ -36,6 +36,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `tidy3d.plugins.design.DesignSpace.run(..., fn_post=...)` now accepts a `priority` keyword to propagate vGPU queue priority to all automatically batched simulations.
 - Introduced `BroadbandPulse` for exciting simulations across a wide frequency spectrum.
 - Added `interp_spec` in `ModeSpec` to allow downsampling and interpolation of waveguide modes in frequency.
+- Added warning if port mesh refinement is incompatible with the `GridSpec` in the `TerminalComponentModeler`.
 
 ### Breaking Changes
 - Edge singularity correction at PEC and lossy metal edges defaults to `True`.

tests/test_plugins/smatrix/terminal_component_modeler_def.py

@@ -267,8 +267,10 @@ def make_coaxial_component_modeler(
     def make_port(center, direction, type, name) -> Union[CoaxialLumpedPort, WavePort]:
         if type is CoaxialLumpedPort:
             port_cells = None
+            enable_snapping_points = False
             if port_refinement:
                 port_cells = 21
+                enable_snapping_points = True
             port = CoaxialLumpedPort(
                 center=center,
                 outer_diameter=2 * Router,
@@ -278,6 +280,7 @@ def make_port(center, direction, type, name) -> Union[CoaxialLumpedPort, WavePor
                 name="coax" + name,
                 num_grid_cells=port_cells,
                 impedance=reference_impedance,
+                enable_snapping_points=enable_snapping_points,
             )
         else:
             mean_radius = (Router + Rinner) / 2

tests/test_plugins/smatrix/test_component_modeler_autograd.py

@@ -197,13 +197,17 @@ def build_terminal_modeler(scale: float) -> TerminalComponentModeler:
         voltage_axis=1,
         name="lp1",
         impedance=50.0,
+        num_grid_cells=None,
+        enable_snapping_points=False,
     )
     p2 = RectLumpedPort(
         center=(0.0, 0.0, 1.5),
         size=port_size,
         voltage_axis=1,
         name="lp2",
         impedance=50.0,
+        num_grid_cells=None,
+        enable_snapping_points=False,
     )
 
     freqs = [2.0e14]

tests/test_plugins/smatrix/test_terminal_component_modeler.py

@@ -29,7 +29,7 @@
 from tidy3d.plugins.smatrix.ports.base_lumped import AbstractLumpedPort
 from tidy3d.plugins.smatrix.utils import s_to_z, validate_square_matrix
 
-from ...utils import run_emulated
+from ...utils import AssertLogLevel, run_emulated
 from .terminal_component_modeler_def import (
     make_basic_filter_terminals,
     make_coaxial_component_modeler,
@@ -521,14 +521,16 @@ def test_ab_to_s_component_modeler():
     assert np.isclose(S_matrix, b_matrix).all()
 
 
-def test_port_snapping(tmp_path):
+def test_port_snapping():
     """Make sure that the snapping behavior of the load resistor is mirrored
     by all other components in the modeler simulations with rectangular ports.
     """
     y_z_grid = td.UniformGrid(dl=0.1 * 1e3)
     x_grid = td.UniformGrid(dl=11 * 1e3)
     grid_spec = td.GridSpec(grid_x=x_grid, grid_y=y_z_grid, grid_z=y_z_grid)
-    modeler = make_component_modeler(planar_pec=True, port_refinement=False, grid_spec=grid_spec)
+    modeler = make_component_modeler(
+        planar_pec=True, port_refinement=False, port_snapping=False, grid_spec=grid_spec
+    )
     check_lumped_port_components_snapped_correctly(modeler=modeler)
 
 
@@ -548,7 +550,7 @@ def test_coaxial_port_source_size(axis):
     assert np.isclose(source.size[axis], 0)
 
 
-def test_coarse_grid_at_port(monkeypatch, tmp_path):
+def test_coarse_grid_at_port(monkeypatch):
     modeler = make_component_modeler(planar_pec=True, port_refinement=False, port_snapping=False)
     # Without port refinement the grid is much too coarse for these port sizes
     with pytest.raises(SetupError):
@@ -819,8 +821,8 @@ def compute_current_patch(self, sim_data):
 
 def test_make_coaxial_component_modeler_with_wave_ports(tmp_path):
     """Checks that the terminal component modeler is created successfully with wave ports."""
-    z_grid = td.UniformGrid(dl=1 * 1e3)
-    xy_grid = td.UniformGrid(dl=0.1 * 1e3)
+    z_grid = td.QuasiUniformGrid(dl=1 * 1e3)
+    xy_grid = td.QuasiUniformGrid(dl=0.1 * 1e3)
     grid_spec = td.GridSpec(grid_x=xy_grid, grid_y=xy_grid, grid_z=z_grid)
     _ = make_coaxial_component_modeler(
         port_types=(WavePort, WavePort),
@@ -834,8 +836,8 @@ def test_run_coaxial_component_modeler_with_wave_ports(
     monkeypatch, tmp_path, voltage_enabled, current_enabled
 ):
     """Checks that the terminal component modeler runs with wave ports."""
-    z_grid = td.UniformGrid(dl=1 * 1e3)
-    xy_grid = td.UniformGrid(dl=0.1 * 1e3)
+    z_grid = td.QuasiUniformGrid(dl=1 * 1e3)
+    xy_grid = td.QuasiUniformGrid(dl=0.1 * 1e3)
     grid_spec = td.GridSpec(grid_x=xy_grid, grid_y=xy_grid, grid_z=z_grid)
     if not (voltage_enabled or current_enabled):
         with pytest.raises(pd.ValidationError):
@@ -891,8 +893,8 @@ def test_run_coaxial_component_modeler_with_wave_ports(
 
 def test_run_mixed_component_modeler_with_wave_ports(monkeypatch, tmp_path):
     """Checks the terminal component modeler will allow mixed ports."""
-    z_grid = td.UniformGrid(dl=1 * 1e3)
-    xy_grid = td.UniformGrid(dl=0.1 * 1e3)
+    z_grid = td.QuasiUniformGrid(dl=1 * 1e3)
+    xy_grid = td.QuasiUniformGrid(dl=0.1 * 1e3)
     grid_spec = td.GridSpec(grid_x=xy_grid, grid_y=xy_grid, grid_z=z_grid)
     modeler = make_coaxial_component_modeler(
         port_types=(CoaxialLumpedPort, WavePort), grid_spec=grid_spec
@@ -1359,8 +1361,8 @@ def test_get_combined_antenna_parameters_data(monkeypatch, tmp_path):
 
 def test_run_only_and_element_mappings(monkeypatch, tmp_path):
     """Checks the terminal component modeler works when running with a subset of excitations."""
-    z_grid = td.UniformGrid(dl=1 * 1e3)
-    xy_grid = td.UniformGrid(dl=0.1 * 1e3)
+    z_grid = td.QuasiUniformGrid(dl=1 * 1e3)
+    xy_grid = td.QuasiUniformGrid(dl=0.1 * 1e3)
     grid_spec = td.GridSpec(grid_x=xy_grid, grid_y=xy_grid, grid_z=z_grid)
     modeler = make_coaxial_component_modeler(
         port_types=(CoaxialLumpedPort, CoaxialLumpedPort), grid_spec=grid_spec
@@ -1674,8 +1676,8 @@ def test_low_freq_smoothing_spec_sim_dict():
 def test_S_parameter_deembedding(monkeypatch, tmp_path):
     """Test S-parameter de-embedding."""
 
-    z_grid = td.UniformGrid(dl=1 * 1e3)
-    xy_grid = td.UniformGrid(dl=0.1 * 1e3)
+    z_grid = td.QuasiUniformGrid(dl=1 * 1e3)
+    xy_grid = td.QuasiUniformGrid(dl=0.1 * 1e3)
     grid_spec = td.GridSpec(grid_x=xy_grid, grid_y=xy_grid, grid_z=z_grid)
     modeler = make_coaxial_component_modeler(port_types=(WavePort, WavePort), grid_spec=grid_spec)
 
@@ -1974,8 +1976,8 @@ def test_validate_run_only_membership():
 
 def test_validate_run_only_with_wave_ports():
     """Test run_only validation with WavePorts in TerminalComponentModeler."""
-    z_grid = td.UniformGrid(dl=1 * 1e3)
-    xy_grid = td.UniformGrid(dl=0.1 * 1e3)
+    z_grid = td.QuasiUniformGrid(dl=1 * 1e3)
+    xy_grid = td.QuasiUniformGrid(dl=0.1 * 1e3)
     grid_spec = td.GridSpec(grid_x=xy_grid, grid_y=xy_grid, grid_z=z_grid)
     modeler = make_coaxial_component_modeler(port_types=(WavePort, WavePort), grid_spec=grid_spec)
 
@@ -2197,8 +2199,8 @@ def test_wave_port_mode_index_validation():
 
 def test_wave_port_mode_index_with_modeler():
     """Test that WavePort.mode_index works correctly with TerminalComponentModeler."""
-    z_grid = td.UniformGrid(dl=1 * 1e3)
-    xy_grid = td.UniformGrid(dl=0.1 * 1e3)
+    z_grid = td.QuasiUniformGrid(dl=1 * 1e3)
+    xy_grid = td.QuasiUniformGrid(dl=0.1 * 1e3)
     grid_spec = td.GridSpec(grid_x=xy_grid, grid_y=xy_grid, grid_z=z_grid)
 
     # Create mode_spec with 4 modes - use defaults
@@ -2306,3 +2308,12 @@ def test_get_task_name():
     # Test with wave port (no mode_index) - should work
     task_name = TerminalComponentModeler.get_task_name(port=wave_port)
     assert task_name == "wave@1"
+
+
+def test_validate_port_refinement_with_uniform_grid():
+    """Test that port refinement options raise error with uniform grid."""
+    uniform_grid = td.GridSpec.uniform(dl=0.5 * mm)
+    with AssertLogLevel("WARNING", contains_str="mesh refinement options enabled"):
+        make_coaxial_component_modeler(
+            port_types=(CoaxialLumpedPort, WavePort), grid_spec=uniform_grid
+        )

tidy3d/plugins/smatrix/component_modelers/terminal.py

@@ -742,6 +742,31 @@ def _validate_3d_simulation(cls, val):
             )
         return val
 
+    @pd.validator("ports")
+    @skip_if_fields_missing(["simulation"])
+    def _validate_port_refinement_usage(cls, val, values):
+        """Warn if port refinement options are enabled, but the supplied simulation
+        does not contain a grid type that will make use of them."""
+
+        sim: Simulation = values.get("simulation")
+        # If grid spec is using AutoGrid
+        # then set up is acceptable
+        if sim.grid_spec.auto_grid_used:
+            return val
+
+        for port in val:
+            if port._is_using_mesh_refinement:
+                log.warning(
+                    f"A port with name '{port.name}' has mesh refinement options enabled, but the "
+                    "'Simulation' passed to the 'TerminalComponentModeler' was setup with a 'GridSpec' which "
+                    "does not support mesh refinement. For accurate simulations, please setup the "
+                    "'Simulation' to use an 'AutoGrid'. To suppress this warning, please explicitly disable "
+                    "mesh refinement options in the port, which are by default enabled. For example, set "
+                    "the 'enable_snapping_points=False' and 'num_grid_cells=None' for lumped ports."
+                )
+
+        return val
+
     @pd.validator("radiation_monitors")
     @skip_if_fields_missing(["freqs"])
     def _validate_radiation_monitors(cls, val, values):

tidy3d/plugins/smatrix/ports/base_lumped.py

@@ -94,3 +94,11 @@ def to_monitors(
     @abstractmethod
     def _check_grid_size(self, yee_grid: YeeGrid) -> None:
         """Raises :class:`SetupError` if the grid is too coarse at port locations."""
+
+    @property
+    def _is_using_mesh_refinement(self) -> bool:
+        """Check if this lumped port is using any mesh refinement options.
+
+        Returns ``True`` if snapping points are enabled or custom grid cell count is specified.
+        """
+        return self.enable_snapping_points or self.num_grid_cells is not None

tidy3d/plugins/smatrix/ports/wave.py

@@ -365,3 +365,11 @@ def _validate_mode_index(cls, val, values):
                 f"'mode_spec.num_modes' ({num_modes}). Valid range is 0 to {num_modes - 1}."
             )
         return val
+
+    @property
+    def _is_using_mesh_refinement(self) -> bool:
+        """Check if this wave port is using mesh refinement options.
+
+        Returns ``True`` if a custom grid cell count is specified.
+        """
+        return self.num_grid_cells is not None