pyedb 0.29.0__tar.gz → 0.31.0__tar.gz

{pyedb-0.29.0 → pyedb-0.31.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: pyedb
-Version: 0.29.0
+Version: 0.31.0
 Summary: Higher-Level Pythonic Ansys Electronics Data Base
 Author-email: "ANSYS, Inc." <pyansys.core@ansys.com>
 Maintainer-email: PyEDB developers <simon.vandenbrouck@ansys.com>
@@ -26,19 +26,19 @@ Requires-Dist: Rtree >= 1.2.0
 Requires-Dist: toml == 0.10.2
 Requires-Dist: scikit-rf
 Requires-Dist: ansys-sphinx-theme>=0.10.0,<1.1 ; extra == "doc"
-Requires-Dist: imageio>=2.30.0,<2.36 ; extra == "doc"
-Requires-Dist: ipython>=8.13.0,<8.28 ; extra == "doc"
+Requires-Dist: imageio>=2.30.0,<2.37 ; extra == "doc"
+Requires-Dist: ipython>=8.13.0,<8.29 ; extra == "doc"
 Requires-Dist: jupyterlab>=4.0.0,<4.3 ; extra == "doc"
 Requires-Dist: jupytext>=1.16.0,<1.17 ; extra == "doc"
 Requires-Dist: matplotlib>=3.5.0,<3.10 ; extra == "doc"
 Requires-Dist: nbsphinx>=0.9.0,<0.10 ; extra == "doc"
 Requires-Dist: nbconvert < 7.17 ; extra == "doc"
 Requires-Dist: numpydoc>=1.5.0,<1.9 ; extra == "doc"
-Requires-Dist: pypandoc>=1.10.0,<1.14 ; extra == "doc"
+Requires-Dist: pypandoc>=1.10.0,<1.15 ; extra == "doc"
 Requires-Dist: recommonmark ; extra == "doc"
 Requires-Dist: Sphinx>=7.1.0,<8.1 ; extra == "doc"
 Requires-Dist: sphinx-autobuild==2021.3.14 ; extra == "doc" and ( python_version == '3.8')
-Requires-Dist: sphinx-autobuild==2024.9.19 ; extra == "doc" and ( python_version > '3.8')
+Requires-Dist: sphinx-autobuild==2024.10.3 ; extra == "doc" and ( python_version > '3.8')
 Requires-Dist: sphinx-copybutton>=0.5.0,<0.6 ; extra == "doc"
 Requires-Dist: sphinx-gallery>=0.14.0,<0.18 ; extra == "doc"
 Requires-Dist: sphinx_design>=0.4.0,<0.7 ; extra == "doc"

{pyedb-0.29.0 → pyedb-0.31.0}/pyproject.toml

@@ -48,20 +48,20 @@ tests = [
 ]
 doc = [
     "ansys-sphinx-theme>=0.10.0,<1.1",
-    "imageio>=2.30.0,<2.36",
-    "ipython>=8.13.0,<8.28",
+    "imageio>=2.30.0,<2.37",
+    "ipython>=8.13.0,<8.29",
     "jupyterlab>=4.0.0,<4.3",
     "jupytext>=1.16.0,<1.17",
     "matplotlib>=3.5.0,<3.10",
     "nbsphinx>=0.9.0,<0.10",
     "nbconvert < 7.17",
     "numpydoc>=1.5.0,<1.9",
-    "pypandoc>=1.10.0,<1.14",
+    "pypandoc>=1.10.0,<1.15",
     # NOTE: Remove recommonmark once examples are reworked.
     "recommonmark",
     "Sphinx>=7.1.0,<8.1",
     "sphinx-autobuild==2021.3.14; python_version == '3.8'",
-    "sphinx-autobuild==2024.9.19; python_version > '3.8'",
+    "sphinx-autobuild==2024.10.3; python_version > '3.8'",
     "sphinx-copybutton>=0.5.0,<0.6",
     "sphinx-gallery>=0.14.0,<0.18",
     "sphinx_design>=0.4.0,<0.7",
@@ -91,8 +91,8 @@ default_section = "THIRDPARTY"
 src_paths = ["doc", "src", "tests"]
 
 [tool.codespell]
-skip = '*.pyc,*.txt,*.gif,*.png,*.jpg,*.js,*.html,*.doctree,*.ttf,*.woff,*.woff2,*.eot,*.mp4,*.inv,*.pickle,*.ipynb,*.a3dcomp,flycheck*,./.git/*,./.hypothesis/*,*.yml,./doc/build/*,./doc/images/*,./dist/*,*~,.hypothesis*,./doc/source/examples/*,*cover,*.dat,*.mac,*.cdb,*.CDB,build,./factory/*,PKG-INFO,*.mypy_cache/*,./_unused/*'
-ignore-words = "doc/styles/Vocab/ANSYS/accept.txt"
+skip = '*.pyc,*.txt,*.gif,*.png,*.jpg,*.js,*.html,*.doctree,*.ttf,*.woff,*.woff2,*.eot,*.mp4,*.inv,*.pickle,*.ipynb,*.a3dcomp,flycheck*,./.git/*,./.hypothesis/*,*.yml,./doc/build/*,./doc/images/*,./dist/*,*~,.hypothesis*,./doc/source/examples/*,*cover,*.dat,*.mac,*.cdb,*.CDB,build,./factory/*,PKG-INFO,*.mypy_cache/*,./_unused/*,pyproject.toml'
+ignore-words = "doc/styles/config/vocabularies/ANSYS/accept.txt"
 ignore-words-list = "renabled, sie, mot"
 enable-colors = true
 

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/__init__.py

@@ -3,14 +3,21 @@ import os
 import sys
 import warnings
 
-if os.name == "nt":
-    os.environ["PYTHONMALLOC"] = "malloc"
+# if os.name == "nt":
+#     os.environ["PYTHONMALLOC"] = "malloc"
 
 # By default we use pyedb legacy implementation
 if "PYEDB_USE_DOTNET" not in os.environ:
     os.environ["PYEDB_USE_DOTNET"] = "0"
 
 LATEST_DEPRECATED_PYTHON_VERSION = (3, 7)
+LINUX_WARNING = (
+    "Due to compatibility issues between .NET Core and libssl on some Linux versions, "
+    "for example Ubuntu 22.04, we are going to stop depending on `dotnetcore2`."
+    "Instead of using this package which embeds .NET Core 3, users will be required to "
+    "install .NET themselves. For more information, see "
+    "https://edb.docs.pyansys.com/version/stable/build_breaking_change.html"
+)
 
 
 def deprecation_warning():
@@ -39,12 +46,31 @@ def deprecation_warning():
     warnings.showwarning = existing_showwarning
 
 
+def linux_warning():
+    """Warning message informing Linux users a future breaking change is coming."""
+    # Store warnings showwarning
+    existing_showwarning = warnings.showwarning
+
+    # Define and use custom showwarning
+    def custom_show_warning(message, category, filename, lineno, file=None, line=None):
+        """Custom warning used to remove <stdin>:loc: pattern."""
+        print("{}: {}".format(category.__name__, message))
+
+    warnings.showwarning = custom_show_warning
+
+    if os.name == "posix":
+        warnings.warn(LINUX_WARNING, FutureWarning)
+
+    # Restore warnings showwarning
+    warnings.showwarning = existing_showwarning
+
+
 deprecation_warning()
 
 #
 
 pyedb_path = os.path.dirname(__file__)
-__version__ = "0.29.0"
+__version__ = "0.31.0"
 version = __version__
 
 #

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/configuration/cfg_components.py

@@ -53,9 +53,9 @@ class CfgComponents:
         self.components = [CfgComponent(**comp) for comp in components_data]
 
     def apply(self):
-        comps_in_db = self._pedb.components.components
+        comps_in_db = self._pedb.components
         for comp in self.components:
-            c_db = comps_in_db[comp.reference_designator]
+            c_db = comps_in_db.instances[comp.reference_designator]
             if comp.definition:
                 c_db.definition = comp.definition
             if comp.type:
@@ -65,7 +65,7 @@ class CfgComponents:
             if comp.port_properties:
                 c_db.port_properties = comp.port_properties
             if comp.ic_die_properties:
-                c_db.set_ic_die_properties = comp.ic_die_properties
+                c_db.ic_die_properties = comp.ic_die_properties
             if comp.pin_pair_model:
                 c_db.model_properties = {"pin_pair_model": comp.pin_pair_model}
             if comp.spice_model:

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/configuration/cfg_data.py

@@ -41,7 +41,7 @@ class CfgData(object):
 
     def __init__(self, pedb, **kwargs):
         self._pedb = pedb
-        self.general = CfgGeneral(self, kwargs.get("general", None))
+        self.general = CfgGeneral(self._pedb, kwargs.get("general", {}))
 
         self.boundaries = CfgBoundaries(self._pedb, kwargs.get("boundaries", {}))
 

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/configuration/cfg_general.py

@@ -24,11 +24,22 @@
 class CfgGeneral:
     """Manage configuration general settings."""
 
-    def __init__(self, pdata, general_dict):
-        self._pedb = pdata._pedb
-        self.spice_model_library = ""
-        self.s_parameter_library = ""
-        if general_dict:
-            self._general_dict = general_dict
-            self.spice_model_library = self._general_dict.get("spice_model_library", "")
-            self.s_parameter_library = self._general_dict.get("s_parameter_library", "")
+    def __init__(self, pedb, data):
+        self._pedb = pedb
+        self.spice_model_library = data.get("spice_model_library", "")
+        self.s_parameter_library = data.get("s_parameter_library", "")
+        self.anti_pads_always_on = data.get("anti_pads_always_on", False)
+        self.suppress_pads = data.get("suppress_pads", True)
+
+    def apply(self):
+        self._pedb.design_options.antipads_always_on = self.anti_pads_always_on
+        self._pedb.design_options.suppress_pads = self.suppress_pads
+
+    def get_data_from_db(self):
+        self.anti_pads_always_on = self._pedb.design_options.antipads_always_on
+        self.suppress_pads = self._pedb.design_options.suppress_pads
+
+        data = {}
+        data["anti_pads_always_on"] = self.anti_pads_always_on
+        data["suppress_pads"] = self.suppress_pads
+        return data

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/configuration/cfg_padstacks.py

@@ -47,12 +47,11 @@ class CfgPadstacks:
             instances_layout = self._pedb.padstacks.instances_by_name
             for inst in self.instances:
                 inst_layout = instances_layout[inst.name]
-                if inst.definition:
-                    # inst_layout.padstack_definition = inst.definition
-                    # Not supported by EDB API
-                    pass
-                if inst.backdrill_parameters:
-                    inst_layout.backdrill_parameters = inst.backdrill_parameters
+                data = dict()
+                data["backdrill_parameters"] = inst.backdrill_parameters
+                data["hole_override_enabled"] = inst.hole_override_enabled
+                data["hole_override_diameter"] = inst.hole_override_diameter
+                inst_layout.properties = data
 
     def get_data_from_db(self):
         self.definitions = []
@@ -73,14 +72,23 @@ class CfgPadstacks:
             definitions.append(i.get_attributes())
         data["definitions"] = definitions
 
-        instances_layout = self._pedb.padstacks.instances_by_name
-        for name, obj in instances_layout.items():
+        for obj in self._pedb.layout.padstack_instances:
+            temp = obj.properties
             self.instances.append(
-                Instance(name=name, definition=obj.padstack_definition, backdrill_parameters=obj.backdrill_parameters)
+                Instance(
+                    name=temp["name"],
+                    definition=temp["definition"],
+                    backdrill_parameters=temp["backdrill_parameters"],
+                    id=temp["id"],
+                    position=temp["position"],
+                    rotation=temp["rotation"],
+                    hole_override_enabled=temp["hole_override_enabled"],
+                    hole_override_diameter=temp["hole_override_diameter"],
+                )
             )
         instances = []
         for i in self.instances:
-            instances.append(i.get_attributes())
+            instances.append(i.get_attributes("id"))
         data["instances"] = instances
         return data
 
@@ -104,3 +112,8 @@ class Instance(CfgBase):
         self.name = kwargs["name"]
         self.definition = kwargs.get("definition", None)
         self.backdrill_parameters = kwargs.get("backdrill_parameters", None)
+        self.id = kwargs.get("id", None)
+        self.position = kwargs.get("position", [])
+        self.rotation = kwargs.get("rotation", None)
+        self.hole_override_enabled = kwargs.get("hole_override_enabled", None)
+        self.hole_override_diameter = kwargs.get("hole_override_diameter", None)

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/configuration/cfg_setup.py

@@ -177,6 +177,11 @@ class CfgSetups:
     def get_data_from_db(self):
         setups = []
         for _, s in self._pedb.setups.items():
+            if float(self._pedb.edbversion) < 2025.1:
+                if not s.type == "hfss":
+                    self._pedb.logger.warning("Only HFSS setups are exported in 2024 R2 and earlier version.")
+                    continue
+
             stp = {}
             if s.type == "hfss":
                 for p_name in CfgHFSSSetup(self._pedb).__dict__:

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/configuration/cfg_stackup.py

@@ -44,6 +44,7 @@ class CfgLayer(CfgBase):
         self.material = kwargs.get("material", None)
         self.fill_material = kwargs.get("fill_material", None)
         self.thickness = kwargs.get("thickness", None)
+        self.roughness = kwargs.get("roughness", None)
 
 
 class CfgStackup:
@@ -139,12 +140,7 @@ class CfgStackup:
     def get_layers_from_db(self):
         layers = []
         for name, obj in self._pedb.stackup.all_layers.items():
-            layer = {}
-            for p_name in CfgLayer().__dict__:
-                p_value = getattr(obj, p_name, None)
-                if p_value is not None:
-                    layer[p_name] = getattr(obj, p_name)
-            layers.append(layer)
+            layers.append(obj.properties)
         return layers
 
     def get_data_from_db(self):

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/configuration/configuration.py

@@ -106,6 +106,9 @@ class Configuration:
     def run(self, **kwargs):
         """Apply configuration settings to the current design"""
 
+        if self.cfg_data.general:
+            self.cfg_data.general.apply()
+
         # Configure boundary settings
         if self.cfg_data.boundaries:
             self.cfg_data.boundaries.apply()
@@ -272,6 +275,8 @@ class Configuration:
         """
         self._pedb.logger.info("Getting data from layout database.")
         data = {}
+        if kwargs.get("general", False):
+            data["general"] = self.cfg_data.general.get_data_from_db()
         if kwargs.get("stackup", False):
             data["stackup"] = self.cfg_data.stackup.get_data_from_db()
         if kwargs.get("package_definitions", False):
@@ -314,6 +319,7 @@ class Configuration:
         boundaries=True,
         s_parameters=True,
         padstacks=True,
+        general=True,
     ):
         """Export the configuration data from layout to a file.
 
@@ -345,6 +351,8 @@ class Configuration:
             Whether to export s_parameters.
         padstacks : bool
             Whether to export padstacks.
+        general : bool
+            Whether to export general information.
         Returns
         -------
         bool
@@ -352,7 +360,7 @@ class Configuration:
         data = self.get_data_from_db(
             stackup=stackup,
             package_definitions=package_definitions,
-            setups=False,
+            setups=setups,
             sources=sources,
             ports=ports,
             nets=nets,
@@ -362,6 +370,7 @@ class Configuration:
             boundaries=boundaries,
             s_parameters=s_parameters,
             padstacks=padstacks,
+            general=general,
         )
 
         file_path = file_path if isinstance(file_path, Path) else Path(file_path)

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/dotnet/edb.py

@@ -558,6 +558,10 @@ class Edb(Database):
             for cell in list(self.top_circuit_cells):
                 if cell.GetName() == self.cellname:
                     self._active_cell = cell
+        if self._active_cell is None:
+            for cell in list(self.circuit_cells):
+                if cell.GetName() == self.cellname:
+                    self._active_cell = cell
         # if self._active_cell is still None, set it to default cell
         if self._active_cell is None:
             self._active_cell = list(self.top_circuit_cells)[0]
@@ -1455,12 +1459,12 @@ class Edb(Database):
     def import_gds_file(
         self,
         inputGDS,
-        WorkDir=None,
         anstranslator_full_path="",
         use_ppe=False,
         control_file=None,
         tech_file=None,
         map_file=None,
+        layer_filter=None,
     ):
         """Import a GDS file and generate an ``edb.def`` file in the working directory.
 
@@ -1471,10 +1475,6 @@ class Edb(Database):
         ----------
         inputGDS : str
             Full path to the GDS file.
-        WorkDir : str, optional
-            Directory in which to create the ``aedb`` folder. The default value is ``None``,
-            in which case the AEDB file is given the same name as the GDS file. Only the extension
-            differs.
         anstranslator_full_path : str, optional
             Full path to the Ansys translator.
         use_ppe : bool, optional
@@ -1484,31 +1484,67 @@ class Edb(Database):
             the XML file in the same directory as the GDS file. To succeed, the XML file and GDS file must
             have the same name. Only the extension differs.
         tech_file : str, optional
-            Technology file. It uses Helic to convert tech file to xml and then imports the gds. Works on Linux only.
+            Technology file. For versions<2024.1 it uses Helic to convert tech file to xml and then imports
+            the gds. Works on Linux only.
+            For versions>=2024.1 it can directly parse through supported foundry tech files.
         map_file : str, optional
             Layer map file.
-
-        Returns
-        -------
-        bool
-            ``True`` when successful, ``False`` when failed.
+        layer_filter:str,optional
+            Layer filter file.
 
         """
-        if not is_linux and tech_file:
-            self.logger.error("Technology files are supported only in Linux. Use control file instead.")
-            return False
         control_file_temp = os.path.join(tempfile.gettempdir(), os.path.split(inputGDS)[-1][:-3] + "xml")
-        ControlFile(xml_input=control_file, tecnhology=tech_file, layer_map=map_file).write_xml(control_file_temp)
-        if self.import_layout_pcb(
-            inputGDS,
-            working_dir=WorkDir,
-            anstranslator_full_path=anstranslator_full_path,
-            use_ppe=use_ppe,
-            control_file=control_file_temp,
-        ):
-            return True
+        if float(self.edbversion) < 2024.1:
+            if not is_linux and tech_file:
+                self.logger.error("Technology files are supported only in Linux. Use control file instead.")
+                return False
+
+            ControlFile(xml_input=control_file, tecnhology=tech_file, layer_map=map_file).write_xml(control_file_temp)
+            if self.import_layout_pcb(
+                inputGDS,
+                anstranslator_full_path=anstranslator_full_path,
+                use_ppe=use_ppe,
+                control_file=control_file_temp,
+            ):
+                return True
+            else:
+                return False
         else:
-            return False
+            temp_map_file = os.path.splitext(inputGDS)[0] + ".map"
+            temp_layermap_file = os.path.splitext(inputGDS)[0] + ".layermap"
+
+            if map_file is None:
+                if os.path.isfile(temp_map_file):
+                    map_file = temp_map_file
+                elif os.path.isfile(temp_layermap_file):
+                    map_file = temp_layermap_file
+                else:
+                    self.logger.error("Unable to define map file.")
+
+            if tech_file is None:
+                if control_file is None:
+                    temp_control_file = os.path.splitext(inputGDS)[0] + ".xml"
+                    if os.path.isfile(temp_control_file):
+                        control_file = temp_control_file
+                    else:
+                        self.logger.error("Unable to define control file.")
+
+                command = [anstranslator_full_path, inputGDS, f'-g="{map_file}"', f'-c="{control_file}"']
+            else:
+                command = [
+                    anstranslator_full_path,
+                    inputGDS,
+                    f'-o="{control_file_temp}"' f'-t="{tech_file}"',
+                    f'-g="{map_file}"',
+                    f'-f="{layer_filter}"',
+                ]
+
+            result = subprocess.run(command, capture_output=True, text=True, shell=True)
+            print(result.stdout)
+            print(command)
+            temp_inputGDS = inputGDS.split(".gds")[0]
+            self.edbpath = temp_inputGDS + ".aedb"
+            return self.open_edb()
 
     def _create_extent(
         self,
@@ -2208,7 +2244,7 @@ class Edb(Database):
         pins_to_preserve = []
         nets_to_preserve = []
         if preserve_components_with_model:
-            for el in self.components.instances.values():
+            for el in self.layout.groups:
                 if el.model_type in [
                     "SPICEModel",
                     "SParameterModel",
@@ -2237,23 +2273,41 @@ class Edb(Database):
         reference_pinsts = []
         reference_prims = []
         reference_paths = []
-        for i in self.padstacks.instances.values():
-            net_name = i.net_name
-            id = i.id
+        pins_to_delete = []
+
+        def check_instances(item):
+            net_name = item.net_name
+            id = item.id
             if net_name not in all_list and id not in pins_to_preserve:
-                i.delete()
+                pins_to_delete.append(item)
             elif net_name in reference_list and id not in pins_to_preserve:
-                reference_pinsts.append(i)
-        for i in self.modeler.primitives:
-            if i:
-                net_name = i.net_name
+                reference_pinsts.append(item)
+
+        with ThreadPoolExecutor(number_of_threads) as pool:
+            pool.map(lambda item: check_instances(item), self.layout.padstack_instances)
+
+        for i in pins_to_delete:
+            i.delete()
+
+        prim_to_delete = []
+
+        def check_prims(item):
+            if item:
+                net_name = item.net_name
                 if net_name not in all_list:
-                    i.delete()
-                elif net_name in reference_list and not i.is_void:
-                    if keep_lines_as_path and i.type == "Path":
-                        reference_paths.append(i)
+                    prim_to_delete.append(item)
+                elif net_name in reference_list and not item.is_void:
+                    if keep_lines_as_path and item.type == "Path":
+                        reference_paths.append(item)
                     else:
-                        reference_prims.append(i)
+                        reference_prims.append(item)
+
+        with ThreadPoolExecutor(number_of_threads) as pool:
+            pool.map(lambda item: check_prims(item), self.modeler.primitives)
+
+        for i in prim_to_delete:
+            i.delete()
+
         self.logger.info_timer("Net clean up")
         self.logger.reset_timer()
 
@@ -2298,7 +2352,7 @@ class Edb(Database):
         if not _poly or _poly.IsNull():
             self._logger.error("Failed to create Extent.")
             return []
-        self.logger.info_timer("Expanded Net Polygon Creation")
+        self.logger.info_timer("Extent Creation")
         self.logger.reset_timer()
         _poly_list = convert_py_list_to_net_list([_poly])
         prims_to_delete = []
@@ -2381,20 +2435,17 @@ class Edb(Database):
         for pin in pins_to_delete:
             pin.delete()
 
-        self.logger.info_timer(
-            "Padstack Instances removal completed. {} instances removed.".format(len(pins_to_delete))
-        )
+        self.logger.info_timer("{} Padstack Instances deleted.".format(len(pins_to_delete)))
         self.logger.reset_timer()
 
-        # with ThreadPoolExecutor(number_of_threads) as pool:
-        #     pool.map(lambda item: clip_path(item), reference_paths)
-
-        for item in reference_paths:
-            clip_path(item)
-        for prim in reference_prims:  # removing multithreading as failing with new layer from primitive
-            clean_prim(prim)
-        # with ThreadPoolExecutor(number_of_threads) as pool:
-        #    pool.map(lambda item: clean_prim(item), reference_prims)
+        with ThreadPoolExecutor(number_of_threads) as pool:
+            pool.map(lambda item: clip_path(item), reference_paths)
+        with ThreadPoolExecutor(number_of_threads) as pool:
+            pool.map(lambda item: clean_prim(item), reference_prims)
+        # for item in reference_paths:
+        #     clip_path(item)
+        # for prim in reference_prims:  # removing multithreading as failing with new layer from primitive
+        #     clean_prim(prim)
 
         for el in poly_to_create:
             self.modeler.create_polygon(el[0], el[1], net_name=el[2], voids=el[3])
@@ -2402,7 +2453,7 @@ class Edb(Database):
         for prim in prims_to_delete:
             prim.delete()
 
-        self.logger.info_timer("Primitives cleanup completed. {} primitives deleted.".format(len(prims_to_delete)))
+        self.logger.info_timer("{} Primitives deleted.".format(len(prims_to_delete)))
         self.logger.reset_timer()
 
         i = 0
@@ -2411,7 +2462,7 @@ class Edb(Database):
                 val.edbcomponent.Delete()
                 i += 1
                 i += 1
-        self.logger.info("Deleted {} additional components".format(i))
+        self.logger.info("{} components deleted".format(i))
         if remove_single_pin_components:
             self.components.delete_single_pin_rlc()
             self.logger.info_timer("Single Pins components deleted")
@@ -3560,15 +3611,15 @@ class Edb(Database):
         """
         setups = {}
         for i in list(self.active_cell.SimulationSetups):
-            if i.GetType() == self.edb_api.utility.utility.SimulationSetupType.kHFSS:
+            if i.GetType().ToString().endswith("kHFSS"):
                 setups[i.GetName()] = HfssSimulationSetup(self, i)
-            elif i.GetType() == self.edb_api.utility.utility.SimulationSetupType.kSIWave:
+            elif i.GetType().ToString().endswith("kSIWave"):
                 setups[i.GetName()] = SiwaveSimulationSetup(self, i)
-            elif i.GetType() == self.edb_api.utility.utility.SimulationSetupType.kSIWaveDCIR:
+            elif i.GetType().ToString().endswith("kSIWaveDCIR"):
                 setups[i.GetName()] = SiwaveDCSimulationSetup(self, i)
-            elif i.GetType() == self.edb_api.utility.utility.SimulationSetupType.kRaptorX:
+            elif i.GetType().ToString().endswith("kRaptorX"):
                 setups[i.GetName()] = RaptorXSimulationSetup(self, i)
-            elif i.GetType() == self.edb_api.utility.utility.SimulationSetupType.kHFSSPI:
+            elif i.GetType().ToString().endswith("kHFSSPI"):
                 setups[i.GetName()] = HFSSPISimulationSetup(self, i)
         return setups
 

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/dotnet/edb_core/cell/hierarchy/component.py

@@ -1109,7 +1109,7 @@ class EDBComponent(Group):
             die_type = pascal_to_snake(ic_die_prop.GetType().ToString())
             temp["type"] = die_type
             if not die_type == "no_die":
-                temp["orientation"] = pascal_to_snake(ic_die_prop.GetOrientation())
+                temp["orientation"] = pascal_to_snake(ic_die_prop.GetOrientation().ToString())
                 if die_type == "wire_bond":
                     temp["height"] = ic_die_prop.GetHeightValue().ToString()
             return temp
@@ -1123,10 +1123,13 @@ class EDBComponent(Group):
         else:
             ic_die_prop = cp.GetDieProperty().Clone()
             die_type = kwargs.get("type")
+            ic_die_prop.SetType(getattr(self._edb.definition.DieType, snake_to_pascal(die_type)))
             if not die_type == "no_die":
                 orientation = kwargs.get("orientation")
                 if orientation:
-                    ic_die_prop.SetOrientation(getattr(self._edb.definition.DieType, snake_to_pascal(die_type)))
+                    ic_die_prop.SetOrientation(
+                        getattr(self._edb.definition.DieOrientation, snake_to_pascal(orientation))
+                    )
                 if die_type == "wire_bond":
                     height = kwargs.get("height")
                     if height:

{pyedb-0.29.0 → pyedb-0.31.0}/src/pyedb/dotnet/edb_core/cell/layout.py

@@ -230,11 +230,7 @@ class Layout(ObjBase):
         -------
         list of :class:`dotnet.edb_core.dotnet.primitive.PrimitiveDotNet` cast objects.
         """
-        prims = []
-        for p in self._edb_object.Primitives:
-            obj = primitive_cast(self._pedb, p)
-            prims.append(obj)
-        return prims
+        return [primitive_cast(self._pedb, p) for p in self._edb_object.Primitives]
 
     @property
     def bondwires(self):
@@ -249,14 +245,7 @@ class Layout(ObjBase):
 
     @property
     def groups(self):
-        temp = []
-        for i in list(self._edb_object.Groups):
-            group_type = i.ToString().split(".")[-1].lower()
-            if group_type == "component":
-                temp.append(EDBComponent(self._pedb, i))
-            else:
-                pass
-        return temp
+        return [EDBComponent(self._pedb, i) for i in self._edb_object.Groups if i.ToString().endswith(".Component")]
 
     @property
     def pin_groups(self):