pyedb 0.56.0__py3-none-any.whl → 0.58.0__py3-none-any.whl

pyedb/grpc/database/primitive/rectangle.py

@@ -21,11 +21,14 @@
 # SOFTWARE.
 
 
+from typing import Union
+
 from ansys.edb.core.primitive.rectangle import (
+    Rectangle as GrpcRectangle,
     RectangleRepresentationType as GrpcRectangleRepresentationType,
 )
-from ansys.edb.core.primitive.rectangle import Rectangle as GrpcRectangle
 
+from pyedb.grpc.database.layers.layer import Layer
 from pyedb.grpc.database.primitive.primitive import Primitive
 from pyedb.grpc.database.utility.value import Value
 
@@ -33,9 +36,10 @@ from pyedb.grpc.database.utility.value import Value
 class Rectangle(GrpcRectangle, Primitive):
     """Class representing a rectangle object."""
 
-    def __init__(self, pedb, edb_object):
-        Primitive.__init__(self, pedb, edb_object)
-        GrpcRectangle.__init__(self, edb_object.msg)
+    def __init__(self, pedb, edb_object=None):
+        if edb_object:
+            Primitive.__init__(self, pedb, edb_object)
+            GrpcRectangle.__init__(self, edb_object.msg)
         self._pedb = pedb
         self._mapping_representation_type = {
             "center_width_height": GrpcRectangleRepresentationType.CENTER_WIDTH_HEIGHT,
@@ -71,6 +75,103 @@ class Rectangle(GrpcRectangle, Primitive):
         else:
             super().representation_type = self._mapping_representation_type[value]
 
+    def create(
+        self,
+        layout=None,
+        layer: Union[str, Layer] = None,
+        net: Union[str, "Net"] = None,
+        rep_type: str = "center_width_height",
+        param1: float = None,
+        param2: float = None,
+        param3: float = None,
+        param4: float = None,
+        corner_rad: float = 0.0,
+        rotation: float = 0.0,
+    ):
+        """
+        Create a rectangle primitive in the specified layout, layer, and net with the given parameters.
+
+        Parameters
+        ----------
+        layout : Layout, optional
+            The layout in which the rectangle will be created. If not provided, the active layout of the `pedb` instance
+            will be used.
+        layer : Union[str, Layer], optional
+            The layer in which the rectangle will be created. This parameter is required and must be specified.
+        net : Union[str, Net], optional
+            The net to which the rectangle will belong. This parameter is required and must be specified.
+        rep_type : str, optional
+            The representation type of the rectangle. Options are `"center_width_height"` or `"lower_left_upper_right"`.
+            The default value is `"center_width_height"`.
+        param1 : float, optional
+            The first parameter defining the rectangle. Its meaning depends on the `rep_type`.
+        param2 : float, optional
+            The second parameter defining the rectangle. Its meaning depends on the `rep_type`.
+        param3 : float, optional
+            The third parameter defining the rectangle. Its meaning depends on the `rep_type`.
+        param4 : float, optional
+            The fourth parameter defining the rectangle. Its meaning depends on the `rep_type`.
+        corner_rad : float, optional
+            The corner radius of the rectangle. The default value is `0.0`.
+        rotation : float, optional
+            The rotation angle of the rectangle in degrees. The default value is `0.0`.
+
+        Returns
+        -------
+        Rectangle
+            The created rectangle primitive.
+
+        Raises
+        ------
+        ValueError
+            If the `layer` parameter is not provided.
+        ValueError
+            If the `net` parameter is not provided.
+
+        Notes
+        -----
+        - The created rectangle is added to the modeler primitives of the `pedb` instance.
+        - The `rep_type` determines how the parameters are interpreted:
+            - `"center_width_height"`: `param1` and `param2` represent the center coordinates, while `param3` and
+            `param4` represent the width and height.
+            - `"lower_left_upper_right"`: `param1` and `param2` represent the lower-left corner coordinates, while
+            `param3` and `param4` represent the upper-right corner coordinates.
+        """
+        if not layout:
+            layout = self._pedb.active_layout
+        if not layer:
+            raise ValueError("Layer must be provided.")
+        if not net:
+            raise ValueError("Net must be provided.")
+
+        rep_type_mapping = {
+            "center_width_height": GrpcRectangleRepresentationType.CENTER_WIDTH_HEIGHT,
+            "lower_left_upper_right": GrpcRectangleRepresentationType.LOWER_LEFT_UPPER_RIGHT,
+        }
+        rep_type = rep_type_mapping.get(rep_type, GrpcRectangleRepresentationType.INVALID_RECT_TYPE)
+        edb_object = super().create(
+            layout=layout,
+            layer=layer,
+            net=net,
+            rep_type=rep_type,
+            param1=Value(param1),
+            param2=Value(param2),
+            param3=Value(param3),
+            param4=Value(param4),
+            corner_rad=Value(corner_rad),
+            rotation=Value(rotation),
+        )
+        # keep cache synced
+        new_rect = Rectangle(self._pedb, edb_object)
+        self._pedb.modeler._add_primitive(new_rect)
+        return new_rect
+
+    def delete(self):
+        """Delete the rectangle primitive from the layout."""
+        # Remove from cache
+        self._pedb.modeler._remove_primitive(self)
+        super().delete()
+
     def get_parameters(self):
         """Get coordinates parameters.
 

pyedb/grpc/database/simulation_setup/hfss_general_settings.py

@@ -23,8 +23,6 @@
 
 from ansys.edb.core.simulation_setup.hfss_simulation_settings import (
     AdaptType as GrpcAdaptType,
-)
-from ansys.edb.core.simulation_setup.hfss_simulation_settings import (
     HFSSGeneralSettings as GrpcHFSSGeneralSettings,
 )
 

pyedb/grpc/database/simulation_setup/hfss_settings_options.py

@@ -23,11 +23,7 @@
 
 from ansys.edb.core.simulation_setup.hfss_simulation_settings import (
     BasisFunctionOrder as GrpcBasisFunctionOrder,
-)
-from ansys.edb.core.simulation_setup.hfss_simulation_settings import (
     HFSSSettingsOptions as GrpcHFSSSettingsOptions,
-)
-from ansys.edb.core.simulation_setup.hfss_simulation_settings import (
     SolverType as GrpcSolverType,
 )
 

pyedb/grpc/database/simulation_setup/hfss_simulation_setup.py

@@ -30,6 +30,7 @@ from ansys.edb.core.simulation_setup.hfss_simulation_settings import (
 from ansys.edb.core.simulation_setup.hfss_simulation_setup import (
     HfssSimulationSetup as GrpcHfssSimulationSetup,
 )
+from ansys.edb.core.simulation_setup.mesh_operation import LengthMeshOperation as GrpcLengthMeshOperation
 
 from pyedb.generic.general_methods import generate_unique_name
 from pyedb.grpc.database.simulation_setup.sweep_data import SweepData
@@ -411,3 +412,81 @@ class HfssSimulationSetup(GrpcHfssSimulationSetup):
             else:
                 self._pedb.logger.error("Failed to add frequency sweep data")
                 return False
+
+    def auto_mesh_operation(
+        self,
+        trace_ratio_seeding: float = 3,
+        signal_via_side_number: int = 12,
+        power_ground_via_side_number: int = 6,
+    ) -> bool:
+        """
+        Automatically create and apply a length-based mesh operation for all nets in the design.
+
+        The method inspects every signal net, determines the smallest trace width, and
+        seeds a :class:`GrpcLengthMeshOperation` whose maximum element length is
+        ``smallest_width * trace_ratio_seeding``. Signal vias (padstack instances) are
+        configured with the requested number of polygon sides, while power/ground vias
+        are updated through the global ``num_via_sides`` advanced setting.
+
+        Parameters
+        ----------
+        trace_ratio_seeding : float, optional
+            Ratio used to compute the maximum allowed element length from the
+            smallest trace width found in the design.  The resulting length is
+            ``min_width * trace_ratio_seeding``.  Defaults to ``3``.
+        signal_via_side_number : int, optional
+            Number of sides (i.e. faceting resolution) assigned to **signal**
+            padstack instances that belong to the nets being meshed.
+            Defaults to ``12``.
+        power_ground_via_side_number : int, optional
+            Number of sides assigned to **power/ground** vias via the global
+            ``advanced.num_via_sides`` setting.  Defaults to ``6``.
+
+        Returns
+        -------
+        bool
+
+        Raises
+        ------
+        ValueError
+            If the design contains no terminals, making mesh seeding impossible.
+
+        Notes
+        -----
+        * Only primitives of type ``"path"`` are considered when determining the
+          smallest trace width.
+        * Every ``(net, layer, sheet)`` tuple required by the mesher is
+          automatically populated; sheet are explicitly marked as ``False``.
+        * Existing contents of :attr:`mesh_operations` are **replaced** by the
+          single new operation.
+
+        Examples
+        --------
+        >>> setup = edbapp.setups["my_setup"]
+        >>> setup.auto_mesh_operation(trace_ratio_seeding=4, signal_via_side_number=16)
+        >>> setup.mesh_operations[0].max_length
+        '2.5um'
+        """
+        net_for_mesh_seeding = list(set([term.net.name for term in list(self._pedb.terminals.values())]))
+        if not net_for_mesh_seeding:
+            raise ValueError("No terminals found to seed the mesh operation.")
+        meshop = GrpcLengthMeshOperation(name=f"{self.name}_AutoMeshOp")
+        layer_info = []
+        smallest_width = 1e3
+        for net in net_for_mesh_seeding:
+            traces = [prim for prim in self._pedb.modeler.primitives_by_net[net] if prim.type == "path"]
+            _width = min([trace.width for trace in traces], default=1e3)
+            if _width < smallest_width:
+                smallest_width = _width
+            layers = list(set([trace.layer.name for trace in traces]))
+            for layer in layers:
+                layer_info.append((net, layer, False))
+            for inst in self._pedb.padstacks.instances_by_net[net]:
+                inst.side_number = signal_via_side_number
+        meshop.max_length = f"{round(float((smallest_width * trace_ratio_seeding)), 9) * 1e6}um"
+        meshop.net_layer_info = layer_info
+        self.mesh_operations = [meshop]
+        self.settings.advanced.num_via_sides = power_ground_via_side_number
+        if self.mesh_operations:
+            return True
+        return False

pyedb/grpc/database/simulation_setup/siwave_cpa_simulation_setup.py

@@ -758,6 +758,7 @@ class SIWaveCPASimulationSetup:
         self._pedb = pedb
         self._channel_setup = ChannelSetup(pedb)
         self._solver_options = SolverOptions(pedb)
+        self.type = "cpa"
         if isinstance(siwave_cpa_setup_class, SIwaveCpaSetup):
             self._apply_cfg_object(siwave_cpa_setup_class)
         else:

pyedb/grpc/database/simulation_setup/sweep_data.py

@@ -22,11 +22,9 @@
 
 from ansys.edb.core.simulation_setup.simulation_setup import (
     Distribution as GrpcDistribution,
-)
-from ansys.edb.core.simulation_setup.simulation_setup import (
     FrequencyData as GrpcFrequencyData,
+    SweepData as GrpcSweepData,
 )
-from ansys.edb.core.simulation_setup.simulation_setup import SweepData as GrpcSweepData
 
 
 class SweepData(GrpcSweepData):

pyedb/grpc/database/siwave.py

@@ -24,6 +24,7 @@
 This module contains these classes: ``CircuitPort``, ``CurrentSource``, ``EdbSiwave``,
 ``PinGroup``, ``ResistorSource``, ``Source``, ``SourceType``, and ``VoltageSource``.
 """
+
 import os
 from typing import Any, Dict, Optional, Union
 import warnings
@@ -31,11 +32,9 @@ import warnings
 from ansys.edb.core.database import ProductIdType as GrpcProductIdType
 from ansys.edb.core.simulation_setup.simulation_setup import (
     Distribution as GrpcDistribution,
-)
-from ansys.edb.core.simulation_setup.simulation_setup import (
     FrequencyData as GrpcFrequencyData,
+    SweepData as GrpcSweepData,
 )
-from ansys.edb.core.simulation_setup.simulation_setup import SweepData as GrpcSweepData
 
 from pyedb.grpc.database.simulation_setup.siwave_cpa_simulation_setup import (
     SIWaveCPASimulationSetup,
@@ -335,7 +334,7 @@ class Siwave(object):
             Use :func:`pyedb.grpc.core.excitations._check_gnd` instead.
         """
         warnings.warn(
-            "`_check_gnd` is deprecated and is now located here " "`pyedb.grpc.core.excitations._check_gnd` instead.",
+            "`_check_gnd` is deprecated and is now located here `pyedb.grpc.core.excitations._check_gnd` instead.",
             DeprecationWarning,
         )
         return self._pedb.source_excitation._check_gnd(component_name)
@@ -563,9 +562,7 @@ class Siwave(object):
         >>> success = edbapp.siwave.create_exec_file(add_ac=True, add_syz=True)
         >>> # Create exec file with Touchstone export
         >>> success = edbapp.siwave.create_exec_file(
-        ...     add_ac=True,
-        ...     export_touchstone=True,
-        ...     touchstone_file_path="C:/temp/my_touchstone.s2p"
+        ...     add_ac=True, export_touchstone=True, touchstone_file_path="C:/temp/my_touchstone.s2p"
         ... )
         """
         workdir = os.path.dirname(self._pedb.edbpath)
@@ -644,17 +641,13 @@ class Siwave(object):
         >>> from pyedb import Edb
         >>> edbapp = Edb("myaedbfolder", edbversion="2021.2")
         >>> # Add SYZ analysis with linear sweep from 1kHz to 10GHz
-        >>> setup = edbapp.siwave.add_siwave_syz_analysis(
-        ...     start_freq=1e3,
-        ...     stop_freq=10e9,
-        ...     distribution="linear"
-        ... )
+        >>> setup = edbapp.siwave.add_siwave_syz_analysis(start_freq=1e3, stop_freq=10e9, distribution="linear")
         >>> # Add SYZ analysis with decade sweep
         >>> setup = edbapp.siwave.add_siwave_syz_analysis(
         ...     start_freq=1e3,
         ...     stop_freq=10e9,
         ...     distribution="decade_count",
-        ...     step_freq=10  # 10 points per decade
+        ...     step_freq=10,  # 10 points per decade
         ... )
         """
         setup = self._pedb.create_siwave_syz_setup()

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()
@@ -505,22 +486,22 @@ class SourceExcitation:
 
     def create_port_on_component(
         self,
-        component: Union[str, List[str]],
+        component: Union[str, Component],
         net_list: Union[str, List[str]],
-        port_type: SourceType,
+        port_type: str = "coax_port",
         do_pingroup: Optional[bool] = True,
         reference_net: Optional[str] = None,
         port_name: Optional[List[str]] = None,
-        solder_balls_height: Optional[float] = None,
-        solder_balls_size: Optional[float] = None,
-        solder_balls_mid_size: Optional[float] = None,
+        solder_balls_height: Union[float, str] = None,
+        solder_balls_size: Union[float, str] = None,
+        solder_balls_mid_size: Union[float, str] = None,
         extend_reference_pins_outside_component: Optional[bool] = False,
     ) -> List[str]:
         """Create ports on a component.
 
         Parameters
         ----------
-        component : str or  self._pedb.component
+        component : str or Component
             EDB component or str component name.
         net_list : str or list of string.
             List of nets where ports must be created on the component.
@@ -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))
@@ -2679,7 +2661,7 @@ class SourceExcitation:
             for __pin in pins_name:
                 if __pin in pins:
                     pin = pins[__pin]
-                    term_name = f"{pin.component.name}_{pin.net.name}_{pin.component}"
+                    term_name = f"{pin.component.name}_{pin.net.name}_{pin.component.name}"
                     start_layer, stop_layer = pin.get_layer_range()
                     if start_layer:
                         positive_terminal = PadstackInstanceTerminal.create(
@@ -3081,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,