FIX: Miscellaneous R25.2 fixes (#502)

Author: drewm102

doc/source/api/definition.rst

@@ -29,7 +29,6 @@ Classes
    definition.package_def.PackageDef
    definition.padstack_def.PadstackDef
    definition.padstack_def_data.PadstackDefData
-   definition.material_def.ThermalModifier
    definition.solder_ball_property.SolderBallProperty
    definition.component_property.ComponentProperty
    definition.ic_component_property.ICComponentProperty

src/ansys/edb/core/definition/material_def.py

@@ -5,6 +5,9 @@
 import ansys.api.edb.v1.material_def_pb2 as pb
 
 from ansys.edb.core.definition.dielectric_material_model import DielectricMaterialModel
+from ansys.edb.core.definition.material_property_thermal_modifier import (
+    MaterialPropertyThermalModifier,
+)
 from ansys.edb.core.edb_defs import DefinitionObjType
 from ansys.edb.core.inner import ObjBase, messages
 from ansys.edb.core.session import MaterialDefServiceStub, StubAccessor, StubType
@@ -28,10 +31,6 @@ class MaterialProperty(Enum):
     INVALID_PROPERTY = pb.INVALID_PROPERTY
 
 
-class ThermalModifier(ObjBase):
-    """Represents a thermal modifier model."""
-
-
 class MaterialDef(ObjBase):
     """Represents a material definition."""
 
@@ -236,10 +235,10 @@ def get_thermal_modifier(self, material_property_id):
 
         Returns
         -------
-        ThermalModifier
+        MaterialPropertyThermalModifier
             Thermal modifier of the material definition.
         """
-        return ThermalModifier(
+        return MaterialPropertyThermalModifier(
             self.__stub.GetThermalModifier(
                 MaterialDef._property_message(self, material_property_id)
             )
@@ -253,7 +252,7 @@ def set_thermal_modifier(self, material_property_id, thermal_modifier):
         material_property_id : \
         :class:`MaterialProperty`
             Property ID.
-        thermal_modifier : ThermalModifier
+        thermal_modifier : MaterialPropertyThermalModifier
             Thermal modifier to set to the material definition.
         """
         self.__stub.SetThermalModifier(
@@ -277,10 +276,10 @@ def get_anisotropic_thermal_modifier(self, material_property_id, component_id):
 
         Returns
         -------
-        :class:`.ThermalModifier`
+        :class:`.MaterialPropertyThermalModifier`
             Anisotropic thermal modifier of the material definition.
         """
-        return ThermalModifier(
+        return MaterialPropertyThermalModifier(
             self.__stub.GetAnisotropicThermalModifier(
                 pb.MaterialDefPropertyComponentMessage(
                     materialDef=messages.edb_obj_message(self),
@@ -302,7 +301,7 @@ def set_anisotropic_thermal_modifier(
             Property ID.
         component_id : int
             Component ID.
-        thermal_modifier : :class:`.ThermalModifier`
+        thermal_modifier : :class:`.MaterialPropertyThermalModifier`
             Anisotropic thermal modifier to set to the material definition.
         """
         self.__stub.SetAnisotropicThermalModifier(

src/ansys/edb/core/definition/padstack_def_data.py

@@ -118,7 +118,7 @@ def layer_names(self):
         This property is read-only.
         """
         layer_names_msg = self.__stub.GetLayerNames(self.msg).names
-        return layer_names_msg
+        return list(layer_names_msg)
 
     @property
     def layer_ids(self):
@@ -127,7 +127,7 @@ def layer_ids(self):
         This property is read-only.
         """
         layer_ids_msg = self.__stub.GetLayerIds(self.msg)
-        return layer_ids_msg.ids
+        return list(layer_ids_msg.ids)
 
     def add_layers(self, names):
         """

src/ansys/edb/core/inner/base.py

@@ -23,6 +23,8 @@ def is_null(self):
 
         This property is read-only.
         """
+        if (buffer := get_buffer()) is not None:
+            buffer.flush()
         return self.id == 0
 
     @property
@@ -46,7 +48,7 @@ def msg(self):
         if io_mgr.is_enabled:
             if self._is_future:
                 msg.is_future = True
-            get_io_manager().active_request_edb_obj_msg_mgr.add_active_request_edb_obj_msg(msg)
+            io_mgr.active_request_edb_obj_msg_mgr.add_active_request_edb_obj_msg(msg)
         return msg
 
     @msg.setter

src/ansys/edb/core/layout/mcad_model.py

@@ -54,31 +54,31 @@ def is_mcad(cls, connectable):
 
         This method makes a call directly on a :term:`Connectable`.
         """
-        return cls.__stub.IsMcad(messages.edb_obj_message(connectable))
+        return cls.__stub.IsMcad(messages.edb_obj_message(connectable)).value
 
     @classmethod
     def is_mcad_stride(cls, connectable):
         """Determine if a connectable object is a Stride model.
 
         This method makes a call directly on a :term:`Connectable`.
         """
-        return cls.__stub.IsMcadStride(messages.edb_obj_message(connectable))
+        return cls.__stub.IsMcadStride(messages.edb_obj_message(connectable)).value
 
     @classmethod
     def is_mcad_hfss(cls, connectable):
         """Determine if a connectable object is an HFSS model.
 
         This method makes a call directly on a :term:`Connectable`.
         """
-        return cls.__stub.IsMcadHfss(messages.edb_obj_message(connectable))
+        return cls.__stub.IsMcadHfss(messages.edb_obj_message(connectable)).value
 
     @classmethod
     def is_mcad_3d_comp(cls, connectable):
         """Determine if a connectable object is a 3D component model.
 
         This method makes a call directly on a :term:`Connectable`.
         """
-        return cls.__stub.IsMcad3dComp(messages.edb_obj_message(connectable))
+        return cls.__stub.IsMcad3dComp(messages.edb_obj_message(connectable)).value
 
     @property
     def cell_instance(self):
@@ -108,6 +108,7 @@ def design_name(self):
         return self.__stub.GetDesignName(messages.edb_obj_message(self)).value
 
     @property
+    @parser.to_point3d_data
     def origin(self):
         """:class:`.Point3DData`: \
         Origin 3D point of the MCAD model."""

src/ansys/edb/core/net/net.py

@@ -58,7 +58,7 @@ def find_by_name(cls, layout, name):
     @property
     def name(self):
         """:class:`str`: Name of the net."""
-        return self.__stub.GetName(self.msg).value
+        return self.no_net_name if self.is_no_net else self.__stub.GetName(self.msg).value
 
     @name.setter
     def name(self, value):
@@ -119,3 +119,8 @@ def extended_net(self):
         """
         en = self._layout_objs(LayoutObjType.NET_CLASS)[0]
         return None if en.is_null else en
+
+    @property
+    def is_no_net(self):
+        """:obj:`bool` Flag indicating if this the "no net" (empty) net."""
+        return self.is_null