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

pyedb/generic/process.py

@@ -1,6 +1,6 @@
 import os.path
 from pathlib import Path
-import subprocess
+import subprocess  # nosec B404
 
 from pyedb.generic.general_methods import is_linux
 
@@ -23,7 +23,63 @@ class SiwaveSolve(object):
         full_path = Path(self._pedb.ansys_em_path) / executable
         return str(full_path)
 
+    def __create_exec(self, type):
+        base = os.path.splitext(self._pedb.edbpath)[0]
+        txt_path = base + ".txt"
+        exec_path = base + ".exec"
+        with open(txt_path, "w") as file:
+            if type:
+                if type == "DCIR":
+                    file.write("ExecDcSim")
+                elif type == "SYZ":
+                    file.write("ExecSyzSim")
+                elif type == "CPA":
+                    file.write("ExecSentinelCpaSim")
+                elif type == "TimeCrosstalk":
+                    file.write("ExecTimeDomainCrosstalkSim")
+                elif type == "FreqCrosstalk":
+                    file.write("ExecCrosstalkSim")
+                elif type == "Impedance":
+                    file.write("ExecZ0Sim")
+
+        os.rename(txt_path, exec_path)
+        return exec_path
+
+    def solve_siwave(self, edbpath, analysis_type):
+        """Solve an SIWave setup. Only non-graphical batch mode is supported.
+
+        Parameters
+        ----------
+        analysis_type: str
+            Type of SIWave analysis to perform. Available types are "SYZ", "DCIR", "CPA", "TimeCrosstalk",
+            "FreqCrosstalk", "Impedance".
+        edbpath: str
+            Full path to the .aedb folder, siw or siwz file to be solved.
+        siwave_ng: str, optinial
+            Path to the siwave_ng. Default is the SIWave installation path.
+        """
+
+        command = [
+            self.__siwave_ng_exe_path,
+            edbpath,
+            self.__create_exec(type=analysis_type),
+            "-formatOutput",
+            "-useSubdir",
+        ]
+        try:
+            subprocess.run(command, check=True)
+        except subprocess.CalledProcessError as e:
+            raise RuntimeError(f"An error occurred when launching the solver. Please check input paths") from e
+
     def solve(self, num_of_cores=4):
+        """Solve using siwave_ng.exe
+
+        .. warning::
+            Do not execute this function with untrusted function argument, environment
+            variables or pyedb global settings.
+            See the :ref:`security guide<ref_security_consideration>` for details.
+
+        """
         exec_file = os.path.splitext(self._pedb.edbpath)[0] + ".exec"
         if os.path.exists(exec_file):
             with open(exec_file, "r+") as f:
@@ -39,16 +95,21 @@ class SiwaveSolve(object):
                     f = open(exec_file, "w")
                     f.writelines(content)
         command = [self.__siwave_ng_exe_path, self._pedb.edbpath, exec_file, "-formatOutput -useSubdir"]
-        command_ = command if os.name == "posix" else " ".join(command)
-        # p = subprocess.Popen(command_)
-        p = subprocess.Popen(command)
-        p.wait()
+        try:
+            subprocess.run(command, check=True)  # nosec
+        except subprocess.CalledProcessError as e:  # nosec
+            raise RuntimeError("An error occurred while solving") from e
 
     def export_3d_cad(
         self, format_3d="Q3D", output_folder=None, net_list=None, num_cores=4, aedt_file_name=None, hidden=False
     ):  # pragma: no cover
         """Export edb to Q3D or HFSS
 
+        .. warning::
+            Do not execute this function with untrusted function argument, environment
+            variables or pyedb global settings.
+            See the :ref:`security guide<ref_security_consideration>` for details.
+
         Parameters
         ----------
         format_3d : str, default ``Q3D``
@@ -97,10 +158,10 @@ class SiwaveSolve(object):
         command += ["-RunScriptAndExit", scriptname]
         print(command)
         try:
-            result = subprocess.run(command, check=True, capture_output=True)
+            result = subprocess.run(command, check=True, capture_output=True)  # nosec
             print(result.stdout.decode())
-        except subprocess.CalledProcessError as e:
-            print(f"Error occurred: {e.stderr.decode()}")
+        except subprocess.CalledProcessError as e:  # nosec
+            raise RuntimeError("An error occurred while exporting 3D CAD") from e
         return os.path.join(output_folder, aedt_file_name)
 
     def export_dc_report(
@@ -119,6 +180,11 @@ class SiwaveSolve(object):
     ):
         """Close EDB and solve it with Siwave.
 
+        .. warning::
+            Do not execute this function with untrusted function argument, environment
+            variables or pyedb global settings.
+            See the :ref:`security guide<ref_security_consideration>` for details.
+
         Parameters
         ----------
         siwave_project : str
@@ -215,6 +281,8 @@ class SiwaveSolve(object):
         command.append("-RunScriptAndExit")
         command.append(scriptname)
         print(command)
-        p = subprocess.Popen(command)
-        p.wait()
+        try:
+            subprocess.run(command, check=True)  # nosec
+        except subprocess.CalledProcessError as e:  # nosec
+            raise RuntimeError("An error occurred while solving with Siwave") from e
         return output_list

pyedb/grpc/database/components.py

@@ -20,9 +20,8 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
 
-"""This module contains the `Components` class.
+"""This module contains the `Components` class."""
 
-"""
 import codecs
 import json
 import math
@@ -31,8 +30,7 @@ import re
 from typing import Any, Dict, List, Optional, Tuple, Union
 import warnings
 
-from ansys.edb.core.definition.die_property import DieOrientation as GrpDieOrientation
-from ansys.edb.core.definition.die_property import DieType as GrpcDieType
+from ansys.edb.core.definition.die_property import DieOrientation as GrpDieOrientation, DieType as GrpcDieType
 from ansys.edb.core.definition.solder_ball_property import (
     SolderballShape as GrpcSolderballShape,
 )
@@ -996,8 +994,7 @@ class Components(object):
         """
         component_definition = ComponentDef.find(self._db, name)
         if component_definition.is_null:
-            from ansys.edb.core.layout.cell import Cell as GrpcCell
-            from ansys.edb.core.layout.cell import CellType as GrpcCellType
+            from ansys.edb.core.layout.cell import Cell as GrpcCell, CellType as GrpcCellType
 
             foot_print_cell = GrpcCell.create(self._pedb.active_db, GrpcCellType.FOOTPRINT_CELL, name)
             component_definition = ComponentDef.create(self._db, name, fp=foot_print_cell)
@@ -1013,8 +1010,6 @@ class Components(object):
                 if component_definition_pin.is_null:
                     self._logger.error(f"Failed to create component definition pin {name}-{pin.name}")
                     return None
-        else:
-            self._logger.warning("Found existing component definition for footprint {}".format(name))
         return component_definition
 
     def create(
@@ -1077,8 +1072,12 @@ class Components(object):
         if not compdef:
             return False
         new_cmp = GrpcComponentGroup.create(self._active_layout, component_name, compdef.name)
+        if new_cmp.is_null:
+            raise ValueError(f"Failed to create component {component_name}.")
         if hasattr(pins[0], "component") and pins[0].component:
-            hosting_component_location = pins[0].component.transform
+            hosting_component_location = None
+            if not pins[0].component.is_null:
+                hosting_component_location = pins[0].component.transform
         else:
             hosting_component_location = None
         if not len(pins) == len(compdef.component_pins):
@@ -1470,13 +1469,15 @@ class Components(object):
             sball_shape = GrpcSolderballShape.SOLDERBALL_SPHEROID
 
         cmp_property = cmp.component_property
-        if cmp.type == GrpcComponentType.IC:
+        if cmp.component_type == GrpcComponentType.IC:
             ic_die_prop = cmp_property.die_property
             ic_die_prop.die_type = GrpcDieType.FLIPCHIP
+            if not cmp.placement_layer == list(self._pedb.stackup.layers.keys())[0]:
+                chip_orientation = "chip_up"
             if chip_orientation.lower() == "chip_up":
-                ic_die_prop.orientation = GrpDieOrientation.CHIP_UP
+                ic_die_prop.die_orientation = GrpDieOrientation.CHIP_UP
             else:
-                ic_die_prop.orientation = GrpDieOrientation.CHIP_DOWN
+                ic_die_prop.die_orientation = GrpDieOrientation.CHIP_DOWN
             cmp_property.die_property = ic_die_prop
 
         solder_ball_prop = cmp_property.solder_ball_property
@@ -2228,7 +2229,7 @@ class Components(object):
         Examples
         --------
         >>> from pyedb import Edb
-        >>> edb_file = r'C:\my_edb_file.aedb'
+        >>> edb_file = r"C:\my_edb_file.aedb"
         >>> edb = Edb(edb_file)
         >>> for cmp in list(edb.components.instances.keys()):
         >>>     edb.components.deactivate_rlc_component(component=cmp, create_circuit_port=False)

pyedb/grpc/database/control_file.py

@@ -23,10 +23,12 @@
 import copy
 import os
 import re
-import subprocess
+import subprocess  # nosec B404
 import sys
 from typing import Any, Dict, List, Optional, Union
 
+from defusedxml.ElementTree import parse as defused_parse
+
 from pyedb.generic.general_methods import ET, env_path, env_value, is_linux
 from pyedb.generic.settings import settings
 from pyedb.misc.aedtlib_personalib_install import write_pretty_xml
@@ -36,6 +38,11 @@ from pyedb.misc.misc import list_installed_ansysem
 def convert_technology_file(tech_file, edbversion=None, control_file=None):
     """Convert a technology file to EDB control file (XML).
 
+    .. warning::
+        Do not execute this function with untrusted function argument, environment
+        variables or pyedb global settings.
+        See the :ref:`security guide<ref_security_consideration>` for details.
+
         Parameters
         ----------
         tech_file : str
@@ -58,40 +65,26 @@ def convert_technology_file(tech_file, edbversion=None, control_file=None):
         -------
         # Example 1: Converting a technology file to control file
     >>> converted_file = convert_technology_file(
-    >>> tech_file="/path/to/tech.t",
-    >>> edbversion="2025.2",
-    >>> control_file="/path/to/output.xml"
-    >>> )
+    ...     tech_file="/path/to/tech.t", edbversion="2025.2", control_file="/path/to/output.xml"
+    ... )
     >>> if converted_file:
     >>> print(f"Converted to: {converted_file}")
 
         # Example 2: Creating a material
     >>> from pyedb import ControlFileMaterial
-    >>> material = ControlFileMaterial(
-    >>> "Copper",
-    >>> {"Permittivity": 1.0, "Conductivity": 5.8e7}
-    >>> )
+    >>> material = ControlFileMaterial("Copper", {"Permittivity": 1.0, "Conductivity": 5.8e7})
 
         # Example 3: Creating a dielectric layer
     >>> from pyedb import ControlFileDielectric
-    >>> dielectric = ControlFileDielectric(
-    >>> "Core",
-    >>> {"Thickness": "0.2mm", "Material": "FR4"}
-    >>> )
+    >>> dielectric = ControlFileDielectric("Core", {"Thickness": "0.2mm", "Material": "FR4"})
 
         # Example 4: Creating a signal layer
     >>> from pyedb import ControlFileLayer
-    >>> signal_layer = ControlFileLayer(
-    >>> "TopLayer",
-    >>> {"Type": "signal", "Material": "Copper", "Thickness": "0.035mm"}
-    >>> )
+    >>> signal_layer = ControlFileLayer("TopLayer", {"Type": "signal", "Material": "Copper", "Thickness": "0.035mm"})
 
         # Example 5: Creating a via layer
     >>> from pyedb import ControlFileVia
-    >>> via_layer = ControlFileVia(
-    >>> "Via1",
-    >>> {"StartLayer": "TopLayer", "StopLayer": "BottomLayer"}
-    >>> )
+    >>> via_layer = ControlFileVia("Via1", {"StartLayer": "TopLayer", "StopLayer": "BottomLayer"})
     >>> via_layer.create_via_group = True
     >>> via_layer.tolerance = "0.1mm"
 
@@ -111,11 +104,7 @@ def convert_technology_file(tech_file, edbversion=None, control_file=None):
 
         # Example 8: Setting up simulation extents
     >>> from pyedb import ControlExtent
-    >>> extent = ControlExtent(
-    >>> type="Conforming",
-    >>> diel_hactor=0.3,
-    >>> airbox_hfactor=0.5
-    >>> )
+    >>> extent = ControlExtent(type="Conforming", diel_hactor=0.3, airbox_hfactor=0.5)
 
         # Example 9: Creating circuit ports
     >>> from pyedb import ControlCircuitPt
@@ -142,17 +131,11 @@ def convert_technology_file(tech_file, edbversion=None, control_file=None):
 
         # Example 13: Frequency sweep configuration
     >>> from pyedb import ControlFileSweep
-    >>> sweep = ControlFileSweep(
-    >>> "Sweep1", "1GHz", "10GHz", "0.1GHz",
-    >>> "Interpolating", "LinearStep", True
-    >>> )
+    >>> sweep = ControlFileSweep("Sweep1", "1GHz", "10GHz", "0.1GHz", "Interpolating", "LinearStep", True)
 
         # Example 14: Mesh operation setup
     >>> from pyedb import ControlFileMeshOp
-    >>> mesh_op = ControlFileMeshOp(
-    >>> "FineMesh", "Region1", "MeshOperationSkinDepth",
-    >>> {"Net1": "TopLayer"}
-    >>> )
+    >>> mesh_op = ControlFileMeshOp("FineMesh", "Region1", "MeshOperationSkinDepth", {"Net1": "TopLayer"})
     >>> mesh_op.skin_depth = "1um"
 
         # Example 15: Simulation setup configuration
@@ -236,10 +219,11 @@ def convert_technology_file(tech_file, edbversion=None, control_file=None):
         ]
         commands.append(command)
         commands.append(["rm", "-r", vlc_file_name + ".aedb"])
-        my_env = os.environ.copy()
         for command in commands:
-            p = subprocess.Popen(command, env=my_env)
-            p.wait()
+            try:
+                subprocess.run(command, check=True)  # nosec
+            except subprocess.CalledProcessError as e:  # nosec
+                raise RuntimeError("An error occurred while converting a technology file to edb control file") from e
         if os.path.exists(control_file):
             settings.logger.info("XML file created.")
             return control_file
@@ -290,8 +274,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:
@@ -1673,7 +1660,7 @@ class ControlFile:
         bool
             ``True`` if successful, ``False`` otherwise.
         """
-        tree = ET.parse(xml_input)
+        tree = defused_parse(xml_input)
         root = tree.getroot()
         for el in root:
             if el.tag == "Stackup":

pyedb/grpc/database/definition/materials.py

@@ -34,9 +34,9 @@ from ansys.edb.core.definition.djordjecvic_sarkar_model import (
     DjordjecvicSarkarModel as GrpcDjordjecvicSarkarModel,
 )
 from ansys.edb.core.definition.material_def import (
+    MaterialDef as GrpcMaterialDef,
     MaterialProperty as GrpcMaterialProperty,
 )
-from ansys.edb.core.definition.material_def import MaterialDef as GrpcMaterialDef
 from ansys.edb.core.definition.multipole_debye_model import (
     MultipoleDebyeModel as GrpcMultipoleDebyeModel,
 )

pyedb/grpc/database/definition/package_def.py

@@ -193,8 +193,11 @@ class PackageDef(GrpcPackageDef):
         """
         try:
             return HeatSink(self._pedb, super().heat_sink)
-        except:
-            pass
+        except Exception as e:
+            settings.logger.error(
+                f"A(n) {type(e).__name__} error occurred while attempting to access 'heatsink' "
+                f"property for object {self}: {str(e)}"
+            )
 
     @property
     @deprecated_property
@@ -222,9 +225,9 @@ class PackageDef(GrpcPackageDef):
             Fin thickness.
         """
         from ansys.edb.core.utility.heat_sink import (
+            HeatSink as GrpcHeatSink,
             HeatSinkFinOrientation as GrpcHeatSinkFinOrientation,
         )
-        from ansys.edb.core.utility.heat_sink import HeatSink as GrpcHeatSink
 
         if fin_orientation == "x_oriented":
             fin_orientation = GrpcHeatSinkFinOrientation.X_ORIENTED

pyedb/grpc/database/definition/padstack_def.py

@@ -26,18 +26,16 @@ import warnings
 from ansys.edb.core.definition.padstack_def import PadstackDef as GrpcPadstackDef
 from ansys.edb.core.definition.padstack_def_data import (
     PadGeometryType as GrpcPadGeometryType,
-)
-from ansys.edb.core.definition.padstack_def_data import (
     PadstackHoleRange as GrpcPadstackHoleRange,
+    PadType as GrpcPadType,
 )
-from ansys.edb.core.definition.padstack_def_data import PadType as GrpcPadType
 import ansys.edb.core.geometry.polygon_data
 from ansys.edb.core.geometry.polygon_data import PolygonData as GrpcPolygonData
-from ansys.edb.core.hierarchy.structure3d import MeshClosure as GrpcMeshClosure
-from ansys.edb.core.hierarchy.structure3d import Structure3D as GrpcStructure3D
+from ansys.edb.core.hierarchy.structure3d import MeshClosure as GrpcMeshClosure, Structure3D as GrpcStructure3D
 from ansys.edb.core.primitive.circle import Circle as GrpcCircle
 
 from pyedb.generic.general_methods import generate_unique_name
+from pyedb.grpc.database.primitive.circle import Circle
 from pyedb.grpc.database.utility.value import Value
 
 
@@ -127,7 +125,7 @@ class PadProperties:
             self._update_pad_parameters_parameters(geom_type=GrpcPadGeometryType.PADGEOMTYPE_POLYGON)
         else:
             raise ValueError(
-                f"Unsupported pad shape: {value}. Supported shapes are 'circle', " f"'rectangle', and 'polygon'."
+                f"Unsupported pad shape: {value}. Supported shapes are 'circle', 'rectangle', and 'polygon'."
             )
 
     @property
@@ -297,7 +295,11 @@ class PadstackDef(GrpcPadstackDef):
         List[:class:`PadstackInstance <pyedb.grpc.database.primitive.padstack_instance.PadstackInstance>`]
             List of PadstackInstance objects.
         """
-        return [i for i in list(self._pedb.padstacks.instances.values()) if i.padstack_def.name == self.name]
+        return [
+            i
+            for i in list(self._pedb.padstacks.instances.values())
+            if not i.is_null and i.padstack_def.name == self.name
+        ]
 
     @property
     def layers(self) -> list[str]:
@@ -720,7 +722,7 @@ class PadstackDef(GrpcPadstackDef):
                         net_name=via.net_name,
                     )
                 else:
-                    GrpcCircle.create(
+                    Circle(self._pedb).create(
                         layout,
                         self.start_layer,
                         via.net,
@@ -735,7 +737,7 @@ class PadstackDef(GrpcPadstackDef):
                         net_name=via.net_name,
                     )
                 else:
-                    GrpcCircle.create(
+                    Circle(self._pedb).create(
                         layout,
                         self.stop_layer,
                         via.net,
@@ -748,7 +750,7 @@ class PadstackDef(GrpcPadstackDef):
                     if layer_name == via.start_layer or started:
                         start = layer_name
                         stop = layer_names[layer_names.index(layer_name) + 1]
-                        cloned_circle = GrpcCircle.create(
+                        cloned_circle = Circle(self._pedb).create(
                             layout,
                             start,
                             via.net,
@@ -756,7 +758,7 @@ class PadstackDef(GrpcPadstackDef):
                             Value(pos[1]),
                             Value(rad1),
                         )
-                        cloned_circle2 = GrpcCircle.create(
+                        cloned_circle2 = Circle(self._pedb).create(
                             layout,
                             stop,
                             via.net,
@@ -771,7 +773,7 @@ class PadstackDef(GrpcPadstackDef):
                         s3d.add_member(cloned_circle2)
                         if not self.data.material.value:
                             self._pedb.logger.warning(
-                                f"Padstack definution {self.name} has no material defined." f"Defaulting to copper"
+                                f"Padstack definution {self.name} has no material defined.Defaulting to copper"
                             )
                             self.data.material = "copper"
                         s3d.set_material(self.data.material.value)

pyedb/grpc/database/hfss.py

@@ -23,6 +23,7 @@
 """
 This module contains the ``EdbHfss`` class.
 """
+
 import math
 import warnings
 
@@ -1259,11 +1260,7 @@ class Hfss(object):
         )
         from ansys.edb.core.simulation_setup.simulation_setup import (
             Distribution as GrpcDistribution,
-        )
-        from ansys.edb.core.simulation_setup.simulation_setup import (
             FrequencyData as GrpcFrequencyData,
-        )
-        from ansys.edb.core.simulation_setup.simulation_setup import (
             SweepData as GrpcSweepData,
         )
 

pyedb/grpc/database/hierarchy/component.py

@@ -28,14 +28,13 @@ import warnings
 from ansys.edb.core.definition.component_model import (
     NPortComponentModel as GrpcNPortComponentModel,
 )
-from ansys.edb.core.definition.die_property import DieOrientation as GrpcDieOrientation
-from ansys.edb.core.definition.die_property import DieType as GrpcDieType
+from ansys.edb.core.definition.die_property import DieOrientation as GrpcDieOrientation, DieType as GrpcDieType
 from ansys.edb.core.definition.solder_ball_property import SolderballShape
 from ansys.edb.core.geometry.polygon_data import PolygonData as GrpcPolygonData
 from ansys.edb.core.hierarchy.component_group import (
     ComponentGroup as GrpcComponentGroup,
+    ComponentType as GrpcComponentType,
 )
-from ansys.edb.core.hierarchy.component_group import ComponentType as GrpcComponentType
 from ansys.edb.core.hierarchy.netlist_model import NetlistModel as GrpcNetlistModel
 from ansys.edb.core.hierarchy.pin_pair_model import PinPairModel as GrpcPinPairModel
 from ansys.edb.core.hierarchy.sparameter_model import (
@@ -49,7 +48,9 @@ from ansys.edb.core.terminal.padstack_instance_terminal import (
     PadstackInstanceTerminal as GrpcPadstackInstanceTerminal,
 )
 from ansys.edb.core.utility.rlc import Rlc as GrpcRlc
+import numpy as np
 
+from pyedb.generic.general_methods import get_filename_without_extension
 from pyedb.grpc.database.hierarchy.pin_pair_model import PinPairModel
 from pyedb.grpc.database.hierarchy.s_parameter_model import SparamModel
 from pyedb.grpc.database.hierarchy.spice_model import SpiceModel
@@ -60,15 +61,6 @@ from pyedb.grpc.database.terminal.padstack_instance_terminal import (
 )
 from pyedb.grpc.database.utility.value import Value
 
-try:
-    import numpy as np
-except ImportError:
-    warnings.warn(
-        "The NumPy module is required to run some functionalities of EDB.\n"
-        "Install with \n\npip install numpy\n\nRequires CPython."
-    )
-from pyedb.generic.general_methods import get_filename_without_extension
-
 
 class Component(GrpcComponentGroup):
     """Manages EDB functionalities for components.
@@ -1102,7 +1094,7 @@ class Component(GrpcComponentGroup):
                 return False
         if not reference_net:
             self._pedb.logger.warning(
-                f"No reference net provided for S parameter file {file_path}, net `GND` is " f"assigned by default"
+                f"No reference net provided for S parameter file {file_path}, net `GND` is assigned by default"
             )
             reference_net = "GND"
         n_port_model = GrpcNPortComponentModel.find_by_name(self.component_def, name)

pyedb/grpc/database/hierarchy/pingroup.py

@@ -20,16 +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 pyedb.grpc.database.primitive.padstack_instance import PadstackInstance
 from typing import Union
 
 from ansys.edb.core.hierarchy.pin_group import PinGroup as GrpcPinGroup
 from ansys.edb.core.terminal.terminal import BoundaryType as GrpcBoundaryType
 
 from pyedb.generic.general_methods import generate_unique_name
-from pyedb.grpc.database.hierarchy.component import Component
-from pyedb.grpc.database.net.net import Net
-from pyedb.grpc.database.primitive.padstack_instance import PadstackInstance
 from pyedb.grpc.database.terminal.pingroup_terminal import PinGroupTerminal
 from pyedb.grpc.database.utility.value import Value
 
@@ -63,10 +67,13 @@ class PinGroup(GrpcPinGroup):
         :class:`Component <pyedb.grpc.database.hierarchy.component.Component>`
             Pin group component.
         """
+
         return Component(self._pedb, super().component)
 
     @component.setter
     def component(self, value):
+        from pyedb.grpc.database.hierarchy.component import Component
+
         if isinstance(value, Component):
             super(PinGroup, self.__class__).component.__set__(self, value)
 
@@ -78,6 +85,8 @@ class PinGroup(GrpcPinGroup):
         -------
         Dict[:class:`PadstackInstance <pyedb.grpc.database.primitive.padstack_instance.PadstackInstance>`].
         """
+        from pyedb.grpc.database.primitive.padstack_instance import PadstackInstance
+
         return {i.name: PadstackInstance(self._pedb, i) for i in super().pins}
 
     @property
@@ -88,10 +97,14 @@ class PinGroup(GrpcPinGroup):
         -------
         :class:`Net <ansys.edb.core.net.net.Net>`.
         """
+        from pyedb.grpc.database.net.net import Net
+
         return Net(self._pedb, super().net)
 
     @net.setter
     def net(self, value):
+        from pyedb.grpc.database.net.net import Net
+
         if isinstance(value, Net):
             super(PinGroup, self.__class__).net.__set__(self, value)
 

pyedb/grpc/database/layers/layer.py

@@ -6,8 +6,7 @@
 
 from __future__ import absolute_import
 
-from ansys.edb.core.layer.layer import Layer as GrpcLayer
-from ansys.edb.core.layer.layer import LayerType as GrpcLayerType
+from ansys.edb.core.layer.layer import Layer as GrpcLayer, LayerType as GrpcLayerType
 
 
 class Layer(GrpcLayer):