FIX: emit properties cleanup

doc/changelog.d/6835.fixed.md

@@ -0,0 +1 @@
+Emit properties cleanup

ignore_words.txt

@@ -32,4 +32,5 @@ aline
 COM
 gRPC
 Toolkits
-Extensions
+Extensions
+Synopsys

src/ansys/aedt/core/emit_core/emit_schematic.py

@@ -182,7 +182,8 @@ def create_radio_antenna(
             new_antenna = self.create_component("Antenna", antenna_name, "Antennas")
             if new_radio and new_antenna:
                 self.connect_components(new_antenna.name, new_radio.name)  # Connect antenna to radio
-            return new_radio, new_antenna
+                return new_radio, new_antenna
+            raise RuntimeError(f"Failed to create radio of type '{radio_type}' or antenna.")
         except Exception as e:
             self.emit_instance.logger.error(f"Failed to create radio of type '{radio_type}' or antenna: {e}")
             raise RuntimeError(f"Failed to create radio of type '{radio_type}' or antenna: {e}")

src/ansys/aedt/core/emit_core/nodes/emit_node.py

@@ -91,6 +91,28 @@ def name(self) -> str:
         """
         return self._get_property("Name", True)
 
+    @name.setter
+    def name(self, requested_name: str):
+        """Renames the node/component.
+
+        Parameters
+        ----------
+        requested_name : str
+            New name for the node/component.
+
+        Raises
+        ------
+        ValueError
+            If the node is read-only or cannot be renamed.
+        """
+        if self._result_id > 0:
+            raise ValueError("This node is read-only for kept results.")
+
+        if self.get_is_component():
+            self._emit_obj.oeditor.RenameComponent(self.name, requested_name)
+        else:
+            _ = self._oRevisionData.RenameEmitNode(self._result_id, self._node_id, requested_name)
+
     @property
     def _node_type(self) -> str:
         """Type of the node.
@@ -276,9 +298,9 @@ def _get_property(self, prop, skipChecks=False, isTable=False) -> Union[str, Lis
         except Exception:
             raise self._emit_obj.logger.aedt_messages.error_level[-1]
 
-    def _set_property(self, prop, value):
+    def _set_property(self, prop, value, skipChecks=False):
         try:
-            self._oRevisionData.SetEmitNodeProperties(self._result_id, self._node_id, [f"{prop}={value}"], True)
+            self._oRevisionData.SetEmitNodeProperties(self._result_id, self._node_id, [f"{prop}={value}"], skipChecks)
         except Exception:
             error_text = None
             if len(self._emit_obj.logger.messages.error_level) > 0:
@@ -408,6 +430,9 @@ def _delete(self):
     def _rename(self, requested_name: str) -> str:
         """Renames the node/component.
 
+        .. deprecated: 0.21.3
+            Use name property instead
+
         Parameters
         ----------
         requested_name : str
@@ -423,15 +448,10 @@ def _rename(self, requested_name: str) -> str:
         ValueError
             If the node is read-only and cannot be renamed.
         """
-        if self.get_is_component():
-            if self._result_id > 0:
-                raise ValueError("This node is read-only for kept results.")
-            self._emit_obj.oeditor.RenameComponent(self.name, requested_name)
-            new_name = requested_name
-        else:
-            new_name = self._oRevisionData.RenameEmitNode(self._result_id, self._node_id, requested_name)
+        warnings.warn("This property is deprecated in 0.21.3. Use the name property instead.", DeprecationWarning)
+        self.name = requested_name
 
-        return new_name
+        return self.name
 
     def _duplicate(self, new_name):
         raise NotImplementedError("This method is not implemented yet.")
@@ -563,7 +583,7 @@ def _set_table_data(self, table):
                 # with ';' separating rows and '|' separating columns
                 table_key = self._get_property("TableKey", True)
                 data = ";".join("|".join(map(str, row)) for row in table)
-                self._set_property(table_key, data)
+                self._set_property(table_key, data, True)
         except Exception as e:
             print(f"Failed to set table data for node {self.name}. Error: {e}")
 

src/ansys/aedt/core/emit_core/nodes/emitter_node.py

@@ -51,8 +51,8 @@ class EmitterNode(EmitNode):
     """
 
     def __init__(self, emit_obj, result_id, node_id):
-        self._is_component = True
         EmitNode.__init__(self, emit_obj, result_id, node_id)
+        self._is_component = True
         self._radio_node = RadioNode(emit_obj, result_id, node_id)
 
         # create_component code provides the radio_id, but we also

src/ansys/aedt/core/emit_core/nodes/generated/amplifier.py

@@ -23,14 +23,13 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 
-
 from ansys.aedt.core.emit_core.nodes.emit_node import EmitNode
 
 
 class Amplifier(EmitNode):
     def __init__(self, emit_obj, result_id, node_id):
-        self._is_component = True
         EmitNode.__init__(self, emit_obj, result_id, node_id)
+        self._is_component = True
 
     @property
     def node_type(self) -> str:
@@ -64,19 +63,6 @@ def table_data(self):
     def table_data(self, value):
         self._set_table_data(value)
 
-    @property
-    def filename(self) -> str:
-        """Name of file defining the outboard component.
-
-        Value should be a full file path.
-        """
-        val = self._get_property("Filename")
-        return val
-
-    @filename.setter
-    def filename(self, value: str):
-        self._set_property("Filename", f"{value}")
-
     @property
     def noise_temperature(self) -> float:
         """System Noise temperature (K) of the component.

src/ansys/aedt/core/emit_core/nodes/generated/antenna_node.py

@@ -30,8 +30,8 @@
 
 class AntennaNode(EmitNode):
     def __init__(self, emit_obj, result_id, node_id):
-        self._is_component = False
         EmitNode.__init__(self, emit_obj, result_id, node_id)
+        self._is_component = True
 
     @property
     def parent(self):

src/ansys/aedt/core/emit_core/nodes/generated/antenna_passband.py

@@ -28,8 +28,8 @@
 
 class AntennaPassband(EmitNode):
     def __init__(self, emit_obj, result_id, node_id):
-        self._is_component = False
         EmitNode.__init__(self, emit_obj, result_id, node_id)
+        self._is_component = False
 
     @property
     def parent(self):

src/ansys/aedt/core/emit_core/nodes/generated/band.py

@@ -30,8 +30,8 @@
 
 class Band(EmitNode):
     def __init__(self, emit_obj, result_id, node_id):
-        self._is_component = False
         EmitNode.__init__(self, emit_obj, result_id, node_id)
+        self._is_component = False
 
     @property
     def parent(self):
@@ -43,6 +43,26 @@ def node_type(self) -> str:
         """The type of this emit node."""
         return self._node_type
 
+    def duplicate(self, new_name: str):
+        """Duplicate this node"""
+        return self._duplicate(new_name)
+
+    def delete(self):
+        """Delete this node"""
+        self._delete()
+
+    def rename(self, new_name: str):
+        """Rename this node"""
+        self._rename(new_name)
+
+    def import_rx_measurement(self, file_name):
+        """Import a Measurement from a File..."""
+        return self._import(file_name, "RxMeasurement")
+
+    def import_tx_measurement(self, file_name):
+        """Import a Measurement from a File..."""
+        return self._import(file_name, "TxMeasurement")
+
     @property
     def enabled(self) -> bool:
         """Enabled state for this node."""

src/ansys/aedt/core/emit_core/nodes/generated/band_folder.py

@@ -28,8 +28,8 @@
 
 class BandFolder(EmitNode):
     def __init__(self, emit_obj, result_id, node_id):
-        self._is_component = False
         EmitNode.__init__(self, emit_obj, result_id, node_id)
+        self._is_component = False
 
     @property
     def parent(self):

src/ansys/aedt/core/emit_core/nodes/generated/cable.py

@@ -30,8 +30,8 @@
 
 class Cable(EmitNode):
     def __init__(self, emit_obj, result_id, node_id):
-        self._is_component = True
         EmitNode.__init__(self, emit_obj, result_id, node_id)
+        self._is_component = True
 
     @property
     def node_type(self) -> str: