fixed drafts in ls-remote

Author: marwaneltoukhy

ipm/common.py

@@ -221,7 +221,21 @@ def get_verified_ip_info(Self, ip_name=None, include_drafts=False, local_file=No
                 logger.print_err(f"IP {ip_name} not found in the release list.")
                 exit(1)
         else:
-            return data
+            # When listing all IPs, filter out IPs with only draft releases if include_drafts is False
+            result = {}
+            for ip_name, ip_info in data.items():
+                releases = ip_info.get("release", {})
+                if not include_drafts:
+                    filtered_releases = {
+                        version: info
+                        for version, info in releases.items()
+                        if not info.get("draft", False)
+                    }
+                    if filtered_releases:
+                        result[ip_name] = {**ip_info, "release": filtered_releases}
+                else:
+                    result[ip_name] = ip_info
+            return result
 
     @staticmethod
     def get_installed_ips(ip_root):
@@ -1245,14 +1259,16 @@ def check_ip_root_dir(ip_root) -> bool:
         return True
 
 
-def list_verified_ips(category=None, technology=None):
-    """creates a table of all verified remote ips
+def list_verified_ips(category=None, technology=None, include_drafts=False, local_file=None):
+    """creates a table of all verified remote ips or from a local file
 
     Args:
         category (str, optional): filter the ips by category. Defaults to None.
         technology (str, optional): filter the ips by technology. Defaults to None.
+        include_drafts (bool, optional): include draft IPs. Defaults to False.
+        local_file (str, optional): path to local verified_IPs.json. Defaults to None.
     """
-    verified_ips = IPInfo.get_verified_ip_info()
+    verified_ips = IPInfo.get_verified_ip_info(include_drafts=include_drafts, local_file=local_file)
     ip_list = []
     logger = Logger()
     check_for_updates(logger)

ipm/manage.py

@@ -111,14 +111,24 @@ def uninstall(
     required=False,
     help="Optionally provide the technology (sky130A, gf180mcuC)",
 )
-def ls_remote_cmd(category, technology):
+@click.option(
+    "--include-drafts",
+    is_flag=True,
+    help="Include draft IPs in the listing",
+)
+@click.option(
+    "--local-file",
+    required=False,
+    help="Path to local verified_IPs.json file",
+)
+def ls_remote_cmd(category, technology, include_drafts, local_file):
     """Lists all verified IPs in ipm's database"""
-    ls_remote(category, technology)
+    ls_remote(category, technology, include_drafts, local_file)
 
 
-def ls_remote(category, technology):
+def ls_remote(category, technology, include_drafts=False, local_file=None):
     """Lists all verified IPs in ipm's database"""
-    list_verified_ips(category, technology)
+    list_verified_ips(category, technology, include_drafts=include_drafts, local_file=local_file)
 
 
 @click.command("info")

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "cf-ipm"
-version = "1.2.1"
+version = "1.2.2"
 description = "Package manager for Open Source ASIC IPs"
 authors = ["ChipFoundry <marwan.abbas@chipfoundry.io>"]
 readme = "README.md"

verified_IPs.json

@@ -53,7 +53,7 @@
                 "bus": [
                     "WB"
                 ],
-                "type": "soft",
+                "type": "hard",
                 "width": "387.87",
                 "height": "303.315",
                 "cell_count": "0",
@@ -70,7 +70,7 @@
                 "bus": [
                     "WB"
                 ],
-                "type": "soft",
+                "type": "hard",
                 "width": "387.87",
                 "height": "303.315",
                 "cell_count": "0",
@@ -3242,6 +3242,7 @@
                 "height": "0.0",
                 "cell_count": "0",
                 "clock_freq_mhz": "48",
+                "draft": true,
                 "supply_voltage": [
                     "n/a"
                 ]