pyedb 0.55.0__py3-none-any.whl → 0.57.0__py3-none-any.whl

pyedb/grpc/database/source_excitations.py

@@ -25,8 +25,7 @@ from typing import Any, Dict, List, Optional, Set, Tuple, Union
 from ansys.edb.core.database import ProductIdType as GrpcProductIdType
 from ansys.edb.core.geometry.point_data import PointData as GrpcPointData
 from ansys.edb.core.geometry.polygon_data import PolygonData as GrpcPolygonData
-from ansys.edb.core.terminal.edge_terminal import EdgeTerminal as GrpcEdgeTerminal
-from ansys.edb.core.terminal.edge_terminal import PrimitiveEdge as GrpcPrimitiveEdge
+from ansys.edb.core.terminal.edge_terminal import EdgeTerminal as GrpcEdgeTerminal, PrimitiveEdge as GrpcPrimitiveEdge
 from ansys.edb.core.terminal.terminal import BoundaryType as GrpcBoundaryType
 from ansys.edb.core.utility.rlc import Rlc as GrpcRlc
 
@@ -66,14 +65,14 @@ class SourceExcitation:
     >>> # Create voltage source on component pins
     >>> from pyedb.grpc.database.utility.sources import Source, SourceType
     >>> source = Source(
-    >>> source_type=SourceType.Vsource,
-    >>> name="V1",
-    >>> positive_node=("U1", "VCC"),
-    >>> negative_node=("U1", "GND"),
-    >>> amplitude="1V",
-    >>> phase="0deg",
-    >>> impedance="50ohm"
-    >>> )
+    ...     source_type=SourceType.Vsource,
+    ...     name="V1",
+    ...     positive_node=("U1", "VCC"),
+    ...     negative_node=("U1", "GND"),
+    ...     amplitude="1V",
+    ...     phase="0deg",
+    ...     impedance="50ohm",
+    ... )
     >>> source_excitations.create_source_on_component([source])
 
     >>> # 2. create_port
@@ -85,21 +84,14 @@ class SourceExcitation:
     >>> # 3. create_port_on_pins
     >>> # Create circuit port between component pins
     >>> port_term = source_excitations.create_port_on_pins(
-    >>> refdes="U1",
-    >>> pins="Pin1",
-    >>> reference_pins=["GND_Pin1", "GND_Pin2"],
-    >>> impedance=50,
-    >>> port_name="Port1"
-    >>> )
+    ...     refdes="U1", pins="Pin1", reference_pins=["GND_Pin1", "GND_Pin2"], impedance=50, port_name="Port1"
+    ... )
 
     >>> # 4. create_port_on_component
     >>> # Create coaxial ports on component nets
     >>> source_excitations.create_port_on_component(
-    >>>  component="U1",
-    >>> net_list=["PCIe_RX0", "PCIe_RX1"],
-    >>> port_type=SourceType.CoaxPort,
-    >>> reference_net="GND"
-    >>> )
+    ...     component="U1", net_list=["PCIe_RX0", "PCIe_RX1"], port_type=SourceType.CoaxPort, reference_net="GND"
+    ... )
 
     >>> # 5. add_port_on_rlc_component
     >>> # Replace RLC component with circuit port
@@ -153,35 +145,26 @@ class SourceExcitation:
     >>> # 16. create_coax_port_on_component
     >>> # Create coaxial ports on component
     >>> ports = source_excitations.create_coax_port_on_component(
-    >>> ["U1", "U2"],
-    >>> ["PCIe_RX0", "PCIe_TX0"],
-    >>> delete_existing_terminal=True
-    >>> )
+    ...     ["U1", "U2"], ["PCIe_RX0", "PCIe_TX0"], delete_existing_terminal=True
+    ... )
 
     >>> # 17. create_differential_wave_port
     >>> # Create differential wave port
     >>> pos_prim = edb.modeler.primitives[0]
     >>> neg_prim = edb.modeler.primitives[1]
     >>> port_name, diff_port = source_excitations.create_differential_wave_port(
-    >>> pos_prim.id, [0, 0],
-    >>> neg_prim.id, [0, 0.2],
-    >>> "DiffPort"
-    >>> )
+    ...     pos_prim.id, [0, 0], neg_prim.id, [0, 0.2], "DiffPort"
+    ... )
 
     >>> # 18. create_wave_port
     >>> # Create wave port
-    >>> port_name, wave_port = source_excitations.create_wave_port(
-    >>> pos_prim.id, [0, 0],
-    >>> "WavePort"
-    >>> )
+    >>> port_name, wave_port = source_excitations.create_wave_port(pos_prim.id, [0, 0], "WavePort")
 
     >>> # 19. create_bundle_wave_port
     >>> # Create bundle wave port
     >>> port_name, bundle_port = source_excitations.create_bundle_wave_port(
-    >>> [pos_prim.id, neg_prim.id],
-    >>> [[0,0], [0,0.2]],
-    >>> "BundlePort"
-    >>> )
+    ...     [pos_prim.id, neg_prim.id], [[0, 0], [0, 0.2]], "BundlePort"
+    ... )
 
     >>> # 20. create_dc_terminal
     >>> # Create DC terminal
@@ -194,10 +177,8 @@ class SourceExcitation:
     >>> # 22. place_voltage_probe
     >>> # Place voltage probe between points
     >>> source_excitations.place_voltage_probe(
-    >>> "Probe1",
-    >>> "SignalNet", [0, 0], "TopLayer",
-    >>> "GND", [0.1, 0.1], "BottomLayer"
-    >>> )
+    ...     "Probe1", "SignalNet", [0, 0], "TopLayer", "GND", [0.1, 0.1], "BottomLayer"
+    ... )
 
     >>> # Save and close EDB
     >>> edb.save()
@@ -584,8 +565,11 @@ class SourceExcitation:
                     net_name = net.name
                     if net_name:
                         net_list.append(net_name)
-                except:
-                    pass
+                except Exception as e:
+                    self._logger.warning(
+                        f"A(n) {type(e).__name__} error occurred while attempting to append 'name' "
+                        f"of net {net} to list of nets {net_list}: {str(e)}"
+                    )
         if isinstance(reference_net, str) or isinstance(reference_net, Net):
             reference_net = [reference_net]
         _reference_net = [ref.name for ref in reference_net if isinstance(ref, Net)]
@@ -1656,8 +1640,7 @@ class SourceExcitation:
 
         if not source_name:
             source_name = (
-                f"Vsource_{positive_component_name}_{positive_net_name}_"
-                f"{negative_component_name}_{negative_net_name}"
+                f"Vsource_{positive_component_name}_{positive_net_name}_{negative_component_name}_{negative_net_name}"
             )
         return self.create_pin_group_terminal(
             positive_pins=pos_node_pins,
@@ -1720,8 +1703,7 @@ class SourceExcitation:
 
         if not source_name:
             source_name = (
-                f"Vsource_{positive_component_name}_{positive_net_name}_"
-                f"{negative_component_name}_{negative_net_name}"
+                f"Vsource_{positive_component_name}_{positive_net_name}_{negative_component_name}_{negative_net_name}"
             )
         return self.create_pin_group_terminal(
             positive_pins=pos_node_pins,
@@ -1865,7 +1847,7 @@ class SourceExcitation:
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> port_name, port = edb.source_excitation.create_differential_wave_port(0, [0,0], 1, [0,0.2])
+        >>> port_name, port = edb.source_excitation.create_differential_wave_port(0, [0, 0], 1, [0, 0.2])
         """
         if not port_name:
             port_name = generate_unique_name("diff")
@@ -1939,7 +1921,7 @@ class SourceExcitation:
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> port_name, port = edb.source_excitation.create_wave_port(0, [0,0])
+        >>> port_name, port = edb.source_excitation.create_wave_port(0, [0, 0])
         """
         if not port_name:
             port_name = generate_unique_name("Terminal_")
@@ -2007,7 +1989,7 @@ class SourceExcitation:
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> term = edb.source_excitation.create_edge_port_vertical(0, [0,0], reference_layer="TopLayer")
+        >>> term = edb.source_excitation.create_edge_port_vertical(0, [0, 0], reference_layer="TopLayer")
         """
         if not port_name:
             port_name = generate_unique_name("Terminal_")
@@ -2079,7 +2061,7 @@ class SourceExcitation:
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> edb.source_excitation.create_edge_port_horizontal(0, [0,0], 1, [0,0.1], "EdgePort")
+        >>> edb.source_excitation.create_edge_port_horizontal(0, [0, 0], 1, [0, 0.1], "EdgePort")
         """
         pos_edge_term = self._create_edge_terminal(prim_id, point_on_edge, port_name)
         neg_edge_term = self._create_edge_terminal(ref_prim_id, point_on_ref_edge, port_name + "_ref", is_ref=True)
@@ -2350,7 +2332,7 @@ class SourceExcitation:
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> port_name, port = edb.source_excitation.create_bundle_wave_port([0,1], [[0,0],[0,0.2]])
+        >>> port_name, port = edb.source_excitation.create_bundle_wave_port([0, 1], [[0, 0], [0, 0.2]])
         """
         if not port_name:
             port_name = generate_unique_name("bundle_port")
@@ -2453,7 +2435,7 @@ class SourceExcitation:
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> term = edb.source_excitation.get_point_terminal("Term1", "Net1", [0,0], "TopLayer")
+        >>> term = edb.source_excitation.get_point_terminal("Term1", "Net1", [0, 0], "TopLayer")
         """
         from pyedb.grpc.database.terminal.point_terminal import PointTerminal
 
@@ -2571,7 +2553,7 @@ class SourceExcitation:
         >>> edb = Edb()
         >>> poly = edb.modeler.primitives[0]
         >>> ref_poly = edb.modeler.primitives[1]
-        >>> edb.source_excitation.create_edge_port_on_polygon(poly, ref_poly, [0,0], [0.1,0])
+        >>> edb.source_excitation.create_edge_port_on_polygon(poly, ref_poly, [0, 0], [0.1, 0])
         """
         if not polygon:
             self._logger.error("No polygon provided for port {} creation".format(port_name))
@@ -2799,11 +2781,15 @@ class SourceExcitation:
         self,
         terminal: Union[PadstackInstanceTerminal, EdgeTerminal],
         ref_terminal: Union[PadstackInstanceTerminal, EdgeTerminal],
+        magnitude: Union[int, float] = 1,
+        phase: Union[int, float] = 0,
     ) -> bool:
         """Create a voltage source.
 
         Parameters
         ----------
+        name : str, optional
+            Voltage source name
         terminal : :class:`EdgeTerminal <pyedb.grpc.database.terminals.EdgeTerminal>`,
             :class:`PadstackInstanceTerminal <pyedb.grpc.database.terminals.PadstackInstanceTerminal>`,
             :class:`PointTerminal <pyedb.grpc.database.terminals.PointTerminal>`,
@@ -2814,6 +2800,10 @@ class SourceExcitation:
             :class:`PadstackInstanceTerminal <pyedb.grpc.database.terminals.PointTerminal>`,
             :class:`PinGroupTerminal <pyedb.grpc.database.terminals.PinGroupTerminal>`,
             Negative terminal of the source.
+        magnitude : int, float, optional
+            Magnitude of the source.
+        phase : int, float, optional
+            Phase of the source
 
         Returns
         -------
@@ -2832,7 +2822,8 @@ class SourceExcitation:
 
         ref_term = Terminal(self._pedb, ref_terminal)
         ref_term.boundary_type = "voltage_source"
-
+        term.magnitude = self._pedb.value(magnitude)
+        term.phase = self._pedb.value(phase)
         term.ref_terminal = ref_terminal
         return term
 
@@ -3072,8 +3063,9 @@ class SourceExcitation:
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> probe = edb.source_excitation.place_voltage_probe("Probe1", "Net1", [0,0], "TopLayer",
-        ...                                                  "GND", [0.1,0], "TopLayer")
+        >>> probe = edb.source_excitation.place_voltage_probe(
+        ...     "Probe1", "Net1", [0, 0], "TopLayer", "GND", [0.1, 0], "TopLayer"
+        ... )
         """
         p_terminal = PointTerminal.create(
             layout=self._pedb.active_layout,

pyedb/grpc/database/stackup.py

@@ -41,13 +41,12 @@ from ansys.edb.core.definition.solder_ball_property import (
 from ansys.edb.core.geometry.point3d_data import Point3DData as GrpcPoint3DData
 from ansys.edb.core.hierarchy.cell_instance import CellInstance as GrpcCellInstance
 from ansys.edb.core.hierarchy.component_group import ComponentType as GrpcComponentType
-from ansys.edb.core.layer.layer import LayerType as GrpcLayerType
-from ansys.edb.core.layer.layer import TopBottomAssociation as GrpcTopBottomAssociation
+from ansys.edb.core.layer.layer import LayerType as GrpcLayerType, TopBottomAssociation as GrpcTopBottomAssociation
 from ansys.edb.core.layer.layer_collection import (
+    LayerCollection as GrpcLayerCollection,
     LayerCollectionMode as GrpcLayerCollectionMode,
+    LayerTypeSet as GrpcLayerTypeSet,
 )
-from ansys.edb.core.layer.layer_collection import LayerCollection as GrpcLayerCollection
-from ansys.edb.core.layer.layer_collection import LayerTypeSet as GrpcLayerTypeSet
 from ansys.edb.core.layer.stackup_layer import StackupLayer as GrpcStackupLayer
 from ansys.edb.core.layout.mcad_model import McadModel as GrpcMcadModel
 
@@ -128,8 +127,9 @@ class LayerCollection(GrpcLayerCollection):
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> top_layer = edb.stackup.add_layer_top("NewTopLayer", layer_type="signal", thickness="0.1mm",
-        ... material="copper")
+        >>> top_layer = edb.stackup.add_layer_top(
+        ...     "NewTopLayer", layer_type="signal", thickness="0.1mm", material="copper"
+        ... )
         """
         thickness = Value(0.0)
         if "thickness" in kwargs:
@@ -167,8 +167,9 @@ class LayerCollection(GrpcLayerCollection):
         --------
         >>> from pyedb import Edb
         >>> edb = Edb()
-        >>> bot_layer = edb.stackup.add_layer_bottom("NewBottomLayer", layer_type="signal", thickness="0.1mm",
-        ... material="copper")
+        >>> bot_layer = edb.stackup.add_layer_bottom(
+        ...     "NewBottomLayer", layer_type="signal", thickness="0.1mm", material="copper"
+        ... )
         """
         thickness = Value(0.0)
         layer_type_map = {"dielectric": GrpcLayerType.DIELECTRIC_LAYER, "signal": GrpcLayerType.SIGNAL_LAYER}
@@ -1189,8 +1190,11 @@ class Stackup(LayerCollection):
                         sball_prop = cmp_prop.solder_ball_property
                         sball_prop.placement = GrpcSolderballPlacement.ABOVE_PADSTACK
                         cmp_prop.solder_ball_property = sball_prop
-                except:
-                    pass
+                except Exception as e:
+                    self._logger.warning(
+                        f"A(n) {type(e).__name__} error occurred while attempting to update "
+                        f"solder_ball_property for component {cmp}: {str(e)}"
+                    )
                 if cmp_type == GrpcComponentType.IC:
                     die_prop = cmp_prop.die_property
                     chip_orientation = die_prop.die_orientation
@@ -1236,8 +1240,11 @@ class Stackup(LayerCollection):
             try:
                 if val.solder_ball_height and val.placement_layer == layer_name:
                     height = val.solder_ball_height
-            except:
-                pass
+            except Exception as e:
+                self._logger.error(
+                    f"A(n) {type(e).__name__} error occurred while attempting to retrieve solder_height "
+                    f"for layer {layer_name} - Default value of 0.0 is returned: {str(e)}"
+                )
         return height
 
     def _remove_solder_pec(self, layer_name):
@@ -1330,15 +1337,21 @@ class Stackup(LayerCollection):
         >>> mounted_cmp = edb2.components.get_component_by_name("BGA")
 
         >>> vector, rotation, solder_ball_height = edb1.components.get_component_placement_vector(
-        ...                                                     mounted_component=mounted_cmp,
-        ...                                                     hosting_component=hosting_cmp,
-        ...                                                     mounted_component_pin1="A12",
-        ...                                                     mounted_component_pin2="A14",
-        ...                                                     hosting_component_pin1="A12",
-        ...                                                     hosting_component_pin2="A14")
-        >>> edb2.stackup.place_in_layout(edb1.active_cell, angle=0.0, offset_x=vector[0],
-        ...                              offset_y=vector[1], flipped_stackup=False, place_on_top=True,
-        ...                              )
+        ...     mounted_component=mounted_cmp,
+        ...     hosting_component=hosting_cmp,
+        ...     mounted_component_pin1="A12",
+        ...     mounted_component_pin2="A14",
+        ...     hosting_component_pin1="A12",
+        ...     hosting_component_pin2="A14",
+        ... )
+        >>> edb2.stackup.place_in_layout(
+        ...     edb1.active_cell,
+        ...     angle=0.0,
+        ...     offset_x=vector[0],
+        ...     offset_y=vector[1],
+        ...     flipped_stackup=False,
+        ...     place_on_top=True,
+        ... )
         """
         # if flipped_stackup and place_on_top or (not flipped_stackup and not place_on_top):
         self.adjust_solder_dielectrics()
@@ -1418,9 +1431,14 @@ class Stackup(LayerCollection):
         >>> edb2 = Edb(edbpath=targetfile2, edbversion="2021.2")
         >>> hosting_cmp = edb1.components.get_component_by_name("U100")
         >>> mounted_cmp = edb2.components.get_component_by_name("BGA")
-        >>> edb2.stackup.place_in_layout(edb1.active_cell, angle=0.0, offset_x="1mm",
-        ...                                   offset_y="2mm", flipped_stackup=False, place_on_top=True,
-        ...                                   )
+        >>> edb2.stackup.place_in_layout(
+        ...     edb1.active_cell,
+        ...     angle=0.0,
+        ...     offset_x="1mm",
+        ...     offset_y="2mm",
+        ...     flipped_stackup=False,
+        ...     place_on_top=True,
+        ... )
         """
         _angle = angle * math.pi / 180.0
 
@@ -1678,9 +1696,14 @@ class Stackup(LayerCollection):
         --------
         >>> edb1 = Edb(edbpath=targetfile1, edbversion="2021.2")
         >>> a3dcomp_path = "connector.a3dcomp"
-        >>> edb1.stackup.place_a3dcomp_3d_placement(a3dcomp_path, angle=0.0, offset_x="1mm",
-        ...                                   offset_y="2mm", flipped_stackup=False, place_on_top=True,
-        ...                                   )
+        >>> edb1.stackup.place_a3dcomp_3d_placement(
+        ...     a3dcomp_path,
+        ...     angle=0.0,
+        ...     offset_x="1mm",
+        ...     offset_y="2mm",
+        ...     flipped_stackup=False,
+        ...     place_on_top=True,
+        ... )
         """
         rotation_axis_from = GrpcPoint3DData(1.0, 0.0, 0.0)
         _angle = angle * math.pi / 180.0

pyedb/grpc/database/terminal/bundle_terminal.py

@@ -20,15 +20,20 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 
+from __future__ import annotations
+
+from typing import TYPE_CHECKING
+
+if TYPE_CHECKING:
+    from pyedb.grpc.database.hierarchy.component import Component
+    from pyedb.grpc.database.net.net import Net
 from ansys.edb.core.terminal.bundle_terminal import BundleTerminal as GrpcBundleTerminal
 from ansys.edb.core.terminal.terminal import (
+    HfssPIType as GrpcHfssPIType,
     SourceTermToGroundType as GrpcSourceTermToGroundType,
 )
-from ansys.edb.core.terminal.terminal import HfssPIType as GrpcHfssPIType
 
-from pyedb.grpc.database.hierarchy.component import Component
 from pyedb.grpc.database.layers.layer import Layer
-from pyedb.grpc.database.net.net import Net
 from pyedb.grpc.database.terminal.terminal import Terminal
 from pyedb.grpc.database.utility.rlc import Rlc
 from pyedb.grpc.database.utility.value import Value
@@ -77,6 +82,7 @@ class BundleTerminal(GrpcBundleTerminal):
         -------
         :class:`Component <pyedb.grpc.database.hierarchy.component.Component`
         """
+
         return Component(self._pedb, self.component)
 
     @property
@@ -102,6 +108,7 @@ class BundleTerminal(GrpcBundleTerminal):
         -------
         :class:`Net <pyedb.grpc.database.net.net.Net>`
         """
+
         return Net(self._pedb, self.net)
 
     @property

pyedb/grpc/database/terminal/pingroup_terminal.py

@@ -20,12 +20,17 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 
+from __future__ import annotations
+
+from typing import TYPE_CHECKING
+
+if TYPE_CHECKING:
+    from pyedb.grpc.database.net.net import Net
 from ansys.edb.core.terminal.pin_group_terminal import (
     PinGroupTerminal as GrpcPinGroupTerminal,
 )
 from ansys.edb.core.terminal.terminal import BoundaryType as GrpcBoundaryType
 
-from pyedb.grpc.database.net.net import Net
 from pyedb.grpc.database.utility.value import Value
 from pyedb.misc.decorators import deprecated_property
 
@@ -151,6 +156,8 @@ class PinGroupTerminal(GrpcPinGroupTerminal):
             Terminal Net object.
 
         """
+        from pyedb.grpc.database.net.net import Net
+
         return Net(self._pedb, super().net)
 
     @net.setter

pyedb/grpc/database/terminal/terminal.py

@@ -20,14 +20,21 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 
+from __future__ import annotations
+
+from typing import TYPE_CHECKING
+
+if TYPE_CHECKING:
+    from pyedb.grpc.database.primitive.padstack_instance import PadstackInstance
 import re
 
 from ansys.edb.core.terminal.edge_terminal import EdgeType as GrpcEdgeType
-from ansys.edb.core.terminal.terminal import BoundaryType as GrpcBoundaryType
-from ansys.edb.core.terminal.terminal import Terminal as GrpcTerminal
-from ansys.edb.core.terminal.terminal import TerminalType as GrpcTerminalType
+from ansys.edb.core.terminal.terminal import (
+    BoundaryType as GrpcBoundaryType,
+    Terminal as GrpcTerminal,
+    TerminalType as GrpcTerminalType,
+)
 
-from pyedb.grpc.database.primitive.padstack_instance import PadstackInstance
 from pyedb.grpc.database.primitive.primitive import Primitive
 from pyedb.grpc.database.utility.value import Value
 
@@ -364,13 +371,17 @@ class Terminal(GrpcTerminal):
                     return (primitive, self._pedb)
         return None  # pragma: no cover
 
-    def get_point_terminal_reference_primitive(self) -> Primitive:  # pragma : no cover
-        """Find and return the primitive reference for the point terminal or the padstack instance.
+    def get_point_terminal_reference_primitive(self) -> Primitive:
+        """
+        Find and return the primitive reference for the point terminal or the padstack instance.
 
         Returns
         -------
-        :class:`PadstackInstance <pyedb.grpc.database.primitive.padstack_instance.PadstackInstance>` or
-        :class:`Primitive <pyedb.grpc.database.primitive.primitive.Primitive>`
+        Primitive or PadstackInstance
+            The primitive reference for the point terminal or the padstack instance.
+            Returns an instance of :class:`PadstackInstance
+            <pyedb.grpc.database.primitive.padstack_instance.PadstackInstance>`
+            or :class:`Primitive <pyedb.grpc.database.primitive.primitive.Primitive>`.
         """
 
         ref_term = self.reference_terminal  # return value is type terminal

pyedb/grpc/database/utility/heat_sink.py

@@ -27,7 +27,6 @@ from ansys.edb.core.utility.value import Value as GrpcValue
 
 
 class HeatSink:
-
     """Heatsink model description.
 
     Parameters

pyedb/grpc/database/utility/hfss_extent_info.py

@@ -21,10 +21,10 @@
 # SOFTWARE.
 
 from ansys.edb.core.utility.hfss_extent_info import (
+    HfssExtentInfo as GrpcHfssExtentInfo,
     HFSSExtentInfoType as GrpcHfssExtentInfoType,
+    OpenRegionType as GrpcOpenRegionType,
 )
-from ansys.edb.core.utility.hfss_extent_info import HfssExtentInfo as GrpcHfssExtentInfo
-from ansys.edb.core.utility.hfss_extent_info import OpenRegionType as GrpcOpenRegionType
 from ansys.edb.core.utility.value import Value as GrpcValue
 
 

pyedb/grpc/database/utility/value.py

@@ -19,6 +19,7 @@
 # LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
+
 import ansys.edb.core.utility.value
 from ansys.edb.core.utility.value import Value as GrpcValue
 

pyedb/grpc/database/utility/xml_control_file.py

@@ -122,7 +122,7 @@ class ControlProperty:
                 float(value)
                 self.type = 0
             except TypeError:
-                pass
+                self.type = -1
 
     def _write_xml(self, root):
         try:
@@ -132,8 +132,11 @@ class ControlProperty:
                 double.text = str(self.value)
             else:
                 pass
-        except:
-            pass
+        except Exception as e:
+            settings.logger.error(
+                f"A(n) {type(e).__name__} error occurred while attempting to create a new sub-element {self.name} "
+                f"for element {root}: {str(e)}"
+            )
 
 
 class ControlFileMaterial: