fix: rf license warnings and docs

Author: daquinteroflex

docs/api/microwave/index.rst

@@ -6,7 +6,7 @@ Overview
 
 .. warning::
 
-   RF simulations will be subject to new license requirements in the future.
+   RF simulations and functionality will require new license requirements in an upcoming release. All RF-specific classes are now available within the sub-package 'tidy3d.rf'.
 
 .. warning::
 

docs/api/plugins/microwave.rst

@@ -5,7 +5,7 @@ Microwave
 
 .. warning::
 
-    RF simulations are subject to new license requirements in the future. These components are within the RF scope.
+    RF simulations and functionality will require new license requirements in an upcoming release. All RF-specific classes are now available within the sub-package 'tidy3d.rf'. These classes are included.
 
 
 .. autosummary::

docs/api/plugins/smatrix.rst

@@ -14,7 +14,7 @@ S-Matrix Component Modelers Plugin
 
 .. seealso::
 
-   For classes related to microwave/RF modeling, please refer to the main `Microwave and RF <../microwave/index.html>`_ page. 
+   For classes related to microwave/RF modeling, please refer to the main `Microwave and RF <../microwave/index.html>`_ page and `tidy3d.rf` sub-package. 
 
 
 .. warning::

tidy3d/components/microwave/base.py

@@ -2,36 +2,15 @@
 
 from __future__ import annotations
 
-import pydantic.v1 as pd
-
 from tidy3d.components.base import Tidy3dBaseModel
 from tidy3d.config import config
-from tidy3d.log import log
 
 
 class MicrowaveBaseModel(Tidy3dBaseModel):
     """Base model that all RF and microwave specific components inherit from."""
 
-    @pd.root_validator(pre=False)
-    def _warn_rf_license(cls, values):
-        from tidy3d.config import config
-
-        # Skip warning when globally suppressed via config
-        if not config.microwave.suppress_rf_license_warning:
-            log.warning(
-                "ℹ️ ⚠️ RF simulations are subject to new license requirements in the future. "
-                "You have instantiated at least one RF-specific component.",
-                log_once=True,
-            )
-        return values
-
     @classmethod
     def _default_without_license_warning(cls) -> MicrowaveBaseModel:
         """Internal helper factory function for classes inheriting from ``MicrowaveBaseModel``."""
         if config.microwave.suppress_rf_license_warning is True:
             return cls()
-        else:
-            config.microwave.suppress_rf_license_warning = True
-            default_constructed = cls()
-            config.microwave.suppress_rf_license_warning = False
-            return default_constructed

tidy3d/components/simulation.py

@@ -4282,7 +4282,7 @@ def _warn_rf_license(self) -> None:
 
         # issue warning
         if rf_component_breakdown_msg != "":
-            msg = " ℹ️ ⚠️ RF simulations are subject to new license requirements in the future. You are using RF-specific components in this simulation."
+            msg = "RF simulations and functionality will require new license requirements in an upcoming release. All RF-specific classes are now available within the sub-package 'tidy3d.rf'."
             msg += rf_component_breakdown_msg
             log.warning(msg, log_once=True)
 

tidy3d/log.py

@@ -251,6 +251,13 @@ def _log(
     ) -> None:
         """Distribute log messages to all handlers"""
 
+        # Check global cache if requested (before composing/capturing to avoid duplicates)
+        if log_once:
+            # Use the message body before composition as key
+            if message in self._static_cache:
+                return
+            self._static_cache.add(message)
+
         # Compose message
         if len(args) > 0:
             try:
@@ -267,13 +274,6 @@ def _log(
                 custom_loc = []
             self._stack[-1]["messages"].append((level_name, composed_message, custom_loc))
 
-        # Check global cache if requested
-        if log_once:
-            # Use the message body before composition as key
-            if message in self._static_cache:
-                return
-            self._static_cache.add(message)
-
         # Context-local logger emits a single message and consolidates the rest
         if self._counts is not None:
             if len(self._counts) > 0:

tidy3d/plugins/smatrix/__init__.py

@@ -33,7 +33,7 @@
 # Instantiate on plugin import till we unite with toplevel
 warnings.filterwarnings(
     "once",
-    message="ℹ️ ⚠️ RF simulations are subject to new license requirements in the future. You have instantiated at least one RF-specific component.",
+    message="RF simulations and functionality will require new license requirements in an upcoming release. All RF-specific classes are now available within the sub-package 'tidy3d.rf'.",
     category=FutureWarning,
 )
 

tidy3d/plugins/smatrix/component_modelers/base.py

@@ -102,6 +102,14 @@ class AbstractComponentModeler(ABC, Tidy3dBaseModel):
         "Otherwise, a default source time will be constructed.",
     )
 
+    @pd.root_validator(pre=False)
+    def _warn_refactor_2_10(cls, values):
+        log.warning(
+            f"'{cls.__name__}' was refactored (tidy3d 'v2.10.0'). Existing functionality is available differently. Please consult the migration documentation: https://docs.flexcompute.com/projects/tidy3d/en/latest/api/microwave/microwave_migration.html",
+            log_once=True,
+        )
+        return values
+
     @pd.validator("simulation", always=True)
     def _sim_has_no_sources(cls, val):
         """Make sure simulation has no sources as they interfere with tool."""

tidy3d/plugins/smatrix/component_modelers/terminal.py

@@ -210,14 +210,6 @@ class TerminalComponentModeler(AbstractComponentModeler, MicrowaveBaseModel):
         description="The low frequency smoothing parameters for the terminal component simulation.",
     )
 
-    @pd.root_validator(pre=False)
-    def _warn_refactor_2_10(cls, values):
-        log.warning(
-            f"ℹ️ ⚠️ The {cls.__name__} class was refactored in tidy3d version 2.10. Migration documentation will be provided, and existing functionality can be accessed in a different way.",
-            log_once=True,
-        )
-        return values
-
     @property
     def _sim_with_sources(self) -> Simulation:
         """Instance of :class:`.Simulation` with all sources and absorbers added for each port, for plotting."""

tidy3d/plugins/smatrix/data/terminal.py

@@ -14,7 +14,6 @@
 from tidy3d.components.microwave.base import MicrowaveBaseModel
 from tidy3d.components.microwave.data.monitor_data import AntennaMetricsData
 from tidy3d.constants import C_0
-from tidy3d.log import log
 from tidy3d.plugins.smatrix.component_modelers.terminal import TerminalComponentModeler
 from tidy3d.plugins.smatrix.data.base import AbstractComponentModelerData
 from tidy3d.plugins.smatrix.data.data_array import (
@@ -227,14 +226,6 @@ def smatrix_deembedded(self, port_shifts: np.ndarray = None) -> MicrowaveSMatrix
         """Interface function returns  de-embedded S-parameter matrix."""
         return self.change_port_reference_planes(self.smatrix(), port_shifts=port_shifts)
 
-    @pd.root_validator(pre=False)
-    def _warn_rf_license(cls, values):
-        log.warning(
-            "ℹ️ ⚠️ RF simulations are subject to new license requirements in the future. You have instantiated at least one RF-specific component.",
-            log_once=True,
-        )
-        return values
-
     def _monitor_data_at_port_amplitude(
         self,
         port_index: NetworkIndex,