module-qc-database-tools 2.9.0__tar.gz → 2.10.0__tar.gz

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/.pre-commit-config.yaml

@@ -1,6 +1,7 @@
 ci:
   autoupdate_commit_msg: "chore: update pre-commit hooks"
   autofix_commit_msg: "style: pre-commit fixes"
+  autoupdate_schedule: "monthly"
 
 exclude: |
   (?x)^(

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/.zenodo.json

@@ -1,8 +1,8 @@
 {
   "description": " general python tool for interfacing with databases during module QC tests for the ATLAS ITk project",
   "license": "MIT",
-  "title": "atlas-itk-pixel-modules/mqdbt: v2.9.0",
-  "version": "v2.9.0",
+  "title": "atlas-itk-pixel-modules/mqdbt: v2.10.0",
+  "version": "v2.10.0",
   "upload_type": "software",
   "creators": [
     {
@@ -80,7 +80,7 @@
   "related_identifiers": [
     {
       "scheme": "url",
-      "identifier": "https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools/-/tree/v2.9.0",
+      "identifier": "https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools/-/tree/v2.10.0",
       "relation": "isSupplementTo"
     }
   ]

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/CITATION.cff

@@ -54,6 +54,6 @@ keywords:
 license:
 - mit
 message: If you use this software, please cite it using the metadata from this file.
-title: 'atlas-itk-pixel-modules/mqdbt: v2.9.0'
+title: 'atlas-itk-pixel-modules/mqdbt: v2.10.0'
 type: software
-version: v2.9.0
+version: v2.10.0

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/PKG-INFO

@@ -1,11 +1,11 @@
 Metadata-Version: 2.4
 Name: module-qc-database-tools
-Version: 2.9.0
+Version: 2.10.0
 Summary: Python wrapper to interface with LocalDB and Production DB for common tasks for pixel modules.
 Project-URL: Homepage, https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools
 Project-URL: Bug Tracker, https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools/issues
 Project-URL: Source, https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools
-Project-URL: Documentation, https://atlas-itk-pixel-mqdbt.docs.cern.ch/2.9/
+Project-URL: Documentation, https://atlas-itk-pixel-mqdbt.docs.cern.ch/v2.10/
 Author-email: Jay Chan <jay.chan@cern.ch>
 Maintainer-email: Giordon Stark <kratsg@gmail.com>, Elisabetta Pianori <elisabetta.pianori@cern.ch>, Lingxin Meng <lingxin.meng@cern.ch>
 License: Copyright (c) 2022 ATLAS ITk Pixel Modules
@@ -50,7 +50,7 @@ Requires-Dist: itkdb>=0.6.17
 Requires-Dist: itksn>=0.4.2
 Requires-Dist: jsbeautifier
 Requires-Dist: jsondiff
-Requires-Dist: module-qc-data-tools>=1.4.0
+Requires-Dist: module-qc-data-tools>=1.5.0
 Requires-Dist: packaging
 Requires-Dist: pandas
 Requires-Dist: pyarrow
@@ -61,7 +61,7 @@ Requires-Dist: typing-extensions>=4.0; python_version < '3.10'
 Requires-Dist: urllib3<2,>=1.26.20; 'el7.x86_64' in platform_release
 Description-Content-Type: text/markdown
 
-# Module QC Database Tools v2.9.0
+# Module QC Database Tools v2.10.0
 
 The package to regisiter ITkPixV1.1 modules, and generate YARR configs from ITk
 production database using `itkdb` API.

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/README.md

@@ -1,4 +1,4 @@
-# Module QC Database Tools v2.9.0
+# Module QC Database Tools v2.10.0
 
 The package to regisiter ITkPixV1.1 modules, and generate YARR configs from ITk
 production database using `itkdb` API.

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/docs/history.md

@@ -12,8 +12,20 @@ and this project adheres to
 
 **_Changed:_**
 
+- increase mqdt version to v1.5.0 and use centralised cold/warm stage lists
+  (!148)
+- update `run-full-qc` (!146)
+  - use typer everywhere for better self-documentation, remove argparse
+  - only use controller config provided by hw config, remove the `-r` option
+  - get LDB config from hw config
+  - consistent long options using double-dash
+
 **_Added:_**
 
+- extend `fetch-reference-iv` to HV groups (!151)
+- add an option `-c` in `ls` so that it can list the current location of
+  components (!149)
+
 **_Fixed:_**
 
 ## [2.9.0](https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools/-/tags/v2.9.0) - 2025-10-29 ## {: #mqdbt-v2.9.0 }

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/docs/install.md

@@ -90,7 +90,7 @@ package.
 $ python -m venv venv
 $ source venv/bin/activate
 $ python -m pip install -U pip
-$ python -m pip install -U pip module-qc-database-tools==2.9.0
+$ python -m pip install -U pip module-qc-database-tools==2.10.0
 ```
 
 ### via uv
@@ -99,7 +99,7 @@ $ python -m pip install -U pip module-qc-database-tools==2.9.0
 $ python -m venv venv
 $ source venv/bin/activate
 $ python -m pip install uv
-$ uv pip install module-qc-database-tools==2.9.0
+$ uv pip install module-qc-database-tools==2.10.0
 ```
 
 # Usage

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/pyproject.toml

@@ -35,7 +35,7 @@ dependencies = [
     "arrow",
     "packaging",
     "typing_extensions >=4.0; python_version<'3.10'",
-    "module_qc_data_tools>=1.4.0",
+    "module_qc_data_tools>=1.5.0",
 ]
 classifiers = [
     "License :: OSI Approved :: MIT License",
@@ -60,7 +60,7 @@ classifiers = [
 Homepage = "https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools"
 "Bug Tracker" = "https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools/issues"
 Source = "https://gitlab.cern.ch/atlas-itk/pixel/module/module-qc-database-tools"
-"Documentation" = "https://atlas-itk-pixel-mqdbt.docs.cern.ch/2.9/"
+"Documentation" = "https://atlas-itk-pixel-mqdbt.docs.cern.ch/v2.10/"
 
 [project.scripts]
 module-qc-database-tools = "module_qc_database_tools.cli:app"
@@ -70,7 +70,11 @@ registerComponent =  "module_qc_database_tools.cli.register_component:app"
 influxdbtool-retrieve = "module_qc_database_tools.cli.influxdbtool_retrieve:main"
 localdbtool-retrieve = "module_qc_database_tools.cli.localdbtool_retrieve:main"
 localdbtool-upload = "module_qc_database_tools.cli.localdbtool_upload:main"
-run-full-qc = "module_qc_database_tools.cli.run_full_qc:main"
+run-full-qc = "module_qc_database_tools.cli.run_full_qc:app"
+
+[dependency-groups]
+dev = [ { include-group = "test" } ]
+test = [ "pytest" ]
 
 [tool.hatch.version]
 source = "vcs"
@@ -247,9 +251,9 @@ filterwarnings = [
   "error",
   'ignore:unclosed file:ResourceWarning',  # itkdb
 ]
-
 testpaths = ["tests"]
 log_cli_level = "INFO"
+log_level = "INFO"
 
 [tool.repo-review]
 ignore = [

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/src/module_qc_database_tools/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '2.9.0'
-__version_tuple__ = version_tuple = (2, 9, 0)
+__version__ = version = '2.10.0'
+__version_tuple__ = version_tuple = (2, 10, 0)
 
 __commit_id__ = commit_id = None

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/src/module_qc_database_tools/cli/globals.py

@@ -206,14 +206,14 @@ OPTIONS["serial_number"]: str = typer.Option(
     ...,
     "--sn",
     "--serial-number",
-    help="Module serial number",
+    help="Component serial number",
     callback=sn_callback,
 )
 OPTIONS["serial_numbers"]: list[str] = typer.Option(
     ...,
     "--sns",
     "--serial-numbers",
-    help="Module serial numbers. Example for list of SNs: --sns SN1 --sns SN2 --sns SN3....",
+    help="Component serial numbers. Example for list of SNs: --sns SN1 --sns SN2 --sns SN3....",
     callback=sns_callback,
 )
 OPTIONS["stage"]: str = typer.Option(..., "-stage", "--stage", help="Stage to use")
@@ -305,7 +305,7 @@ OPTIONS_serial_number = Annotated[
     typer.Option(
         "--sn",
         "--serial-number",
-        help="Module serial number",
+        help="Component serial number",
         callback=sn_callback,
     ),
 ]

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/src/module_qc_database_tools/cli/localdbtool_upload.py

@@ -19,7 +19,9 @@ from datetime import datetime, timezone
 from pathlib import Path
 
 from module_qc_data_tools.utils import (
+    cold_stages,
     get_chip_type_from_config,
+    warm_stages,
 )
 
 from module_qc_database_tools import exceptions, yarr
@@ -513,14 +515,8 @@ def writeScan(self, i_dir):
                     stage = conn_json.get("stage")
 
                     branch = "default"
-                    warm_list = [
-                        "WARM",
-                        "PARYLENE_UNMASKING",
-                        "WIREBOND_PROTECTION",
-                        "THERMAL_CYCLES",
-                        "LONG_TERM_STABILITY_TEST",
-                    ]
-                    cold_list = ["COLD"]
+                    warm_list = warm_stages
+                    cold_list = cold_stages
 
                     if any(phrase in stage for phrase in warm_list):
                         branch = "warm"

{module_qc_database_tools-2.9.0 → module_qc_database_tools-2.10.0}/src/module_qc_database_tools/cli/run_full_qc.py

@@ -1,6 +1,5 @@
 from __future__ import annotations
 
-import argparse
 import json
 import logging
 import os
@@ -9,6 +8,7 @@ import subprocess
 import sys
 from datetime import datetime
 from pathlib import Path
+from typing import Optional
 
 # from typing import Optional
 import itksn
@@ -19,7 +19,10 @@ from module_qc_data_tools.utils import (
     get_layer_from_sn,
 )
 
-# app = typer.Typer(context_settings=CONTEXT_SETTINGS)
+from module_qc_database_tools.cli.globals import CONTEXT_SETTINGS, OPTIONS, Protocol
+from module_qc_database_tools.cli.utils import load_localdb_config_from_hw
+
+app = typer.Typer(context_settings=CONTEXT_SETTINGS)
 
 # Logging
 logger = logging.getLogger(__name__)
@@ -38,20 +41,23 @@ class FullQCConf:
         ## commandline
         self.BASEDIR = Path.home()
         self.MODULESN = ""
-        self.LDBHOST = "localhost"
-        self.LDBPORT = "5000"
-        self.PROTOCOL = "http"
+        ## required
         self.CONNECTIVITY = (
             self.BASEDIR / self.MODULESN / f"{self.MODULESN}_{self.TYPE}.json"
         )
-        self.CONTROLLER = "configs/controller/specCfg-rd53b-16x1.json"
         self.CONFIG = self.BASEDIR / "configs" / "new_hw_config.json"
+        ## optional
         self.MEASCONFIG = ""
         self.OUTPUT = self.BASEDIR / self.MODULESN
+        self.LDBHOST = "localhost"
+        self.LDBPORT = 5000
+        self.PROTOCOL = "http"
+        self.TAGS = []
 
         ## fixed
         ## will be overwritten from HW config
         self.YARR = self.BASEDIR / "Yarr"
+        self.CONTROLLER = Path("configs/controller/specCfg-rd53b-16x1.json")
         ## will be overwritten from module connectivity file path
         self.MODULEDIR = self.BASEDIR / self.MODULESN
         ## will be overwritten depending on module SN
@@ -86,7 +92,7 @@ class FullQCConf:
             str(self.OUTPUT),
         ]
         if self.MEASCONFIG:
-            cmd.extend(["-cm", self.MEASCONFIG])
+            cmd.extend(["--cm", self.MEASCONFIG])
 
         subprocess.run(cmd, check=True)
 
@@ -113,21 +119,24 @@ class FullQCConf:
 
         # Upload measurements and generate analysis results
         ana_results = ana_dir / f"{self.MODULESN}.json"
+        cmd = [
+            "mqdbt",
+            "upload-measurement",
+            "--path",
+            str(last_meas_path),
+            "--host",
+            self.LDBHOST,
+            "--port",
+            str(self.LDBPORT),
+            "--protocol",
+            self.PROTOCOL,
+            "--out",
+            str(ana_results),
+        ]
+        tags = [f"--tag {item}" for item in self.TAGS]
+        cmd.extend(tags)
         subprocess.run(
-            [
-                "mqdbt",
-                "upload-measurement",
-                "--path",
-                str(last_meas_path),
-                "--host",
-                self.LDBHOST,
-                "--port",
-                self.LDBPORT,
-                "--protocol",
-                self.PROTOCOL,
-                "--out",
-                str(ana_results),
-            ],
+            cmd,
             check=True,
         )
 
@@ -164,74 +173,6 @@ class FullQCConf:
 
         return 0
 
-    def get_args(self):
-        """
-        Function using argparse to parse commandline arguments.
-        """
-        parser = argparse.ArgumentParser()
-        parser.add_argument(
-            "-b", "--basedir", type=Path, default=self.BASEDIR, help="Base directory."
-        )
-        parser.add_argument("-sn", "--modulesn", type=str, help="Module serial number.")
-        parser.add_argument(
-            "-m",
-            "--module-connectivity",
-            default=self.CONNECTIVITY,
-            type=Path,
-            help="Path to module connectivity file.",
-        )
-        parser.add_argument(
-            "-r",
-            "--controller",
-            type=Path,
-            default=self.CONTROLLER,
-            help="Path to module connectivity file.",
-        )
-        parser.add_argument(
-            "-c",
-            "--config",
-            type=Path,
-            required=True,
-            help="Path to hardware config file",
-        )
-        parser.add_argument(
-            "-cm", "--measconfig", type=Path, help="Path to measurement config file"
-        )
-        parser.add_argument(
-            "-o",
-            "--output",
-            type=Path,
-            default=self.OUTPUT,
-            help="Path to output directory",
-        )
-        parser.add_argument("--port", type=str, default=self.LDBPORT, help="LDB port")
-        parser.add_argument("--host", type=str, default=self.LDBHOST, help="LDB host")
-        parser.add_argument(
-            "--protocol",
-            type=str,
-            default=self.PROTOCOL,
-            help="Protocol - http or https.",
-        )
-
-        tests = parser.add_argument_group("tests")
-        tests.add_argument(
-            "-iv",
-            action="store_true",
-            help="Run sensor leakage current vs bias voltage.",
-        )
-        tests.add_argument(
-            "-cal", action="store_true", help="Run CALibration measurements."
-        )
-        tests.add_argument("-mht", action="store_true", help="Run Minimum Health Test.")
-        tests.add_argument("-tun", action="store_true", help="Run TUNing.")
-        tests.add_argument(
-            "-pfa", action="store_true", help="Run Pixel Failure Analysis."
-        )
-        args = parser.parse_args()
-
-        logger.info(args)
-        return parser, args
-
     def run_scan(self, scan_name, scan_output, tag):
         """
         Function to concatenate commands and run YARR scan
@@ -411,27 +352,90 @@ class FullQCConf:
         return 0
 
 
+@app.command()
 def main(
     # MODULESN: Optional[str] = OPTIONS["serial_number"],
-    # OUTPUT: Optional[Path] = typer.Option(
-    # None,
-    # "-o",
-    # "--outdir",
-    # help="Path to output directory.",
-    # exists=False,
-    # writable=True,
-    # ),
-    # LDBHOST: str = OPTIONS["host"],
-    # LDBPORT: int = OPTIONS["port"],
-    # PROTOCOL: str = OPTIONS["protocol"],
+    CONNECTIVITY: Path = typer.Option(
+        ...,
+        "-m",
+        "--module-connectivity",
+        help="path to the module connectivity. Used also to identify the module SN, and to set the default output directory",
+        exists=True,
+        file_okay=True,
+        readable=True,
+        writable=True,
+        resolve_path=True,
+    ),
+    CONFIG: Path = typer.Option(
+        ...,
+        "-c",
+        "--config",
+        help="Hardware Config file path",
+        exists=True,
+        file_okay=True,
+        readable=True,
+        resolve_path=True,
+        is_eager=True,  # must be eagerly evaluated first so we can check it in config_callback()
+    ),
+    MODULESN: Optional[str] = typer.Option(  # noqa: UP045
+        None,
+        "--sn",
+        "--serial-number",
+        help="Module serial number",
+    ),
+    BASEDIR: Optional[Path] = OPTIONS["base_dir"],  # noqa: UP045
+    OUTPUT: Optional[Path] = typer.Option(  # noqa: UP045
+        None,
+        "-o",
+        "--output",
+        help="Path to output directory.",
+        exists=False,
+        writable=True,
+    ),
+    MEASCONFIG: Optional[Path] = typer.Option(  # noqa: UP045
+        None,
+        "--cm",
+        "--measconfig",
+        help="Path to measurement config file.",
+        exists=True,
+        file_okay=True,
+        readable=True,
+    ),
+    LDBHOST: str = OPTIONS["host"],
+    LDBPORT: int = OPTIONS["port"],
+    PROTOCOL: Protocol = OPTIONS["protocol"],
+    TAGS: list[str] = OPTIONS["tags"],
+    iv: bool = typer.Option(
+        False,
+        "--iv",
+        help="Measure sensor leakage current vs bias voltage.",
+    ),
+    cal: bool = typer.Option(
+        False,
+        "--cal",
+        help="Run calibration measurements, e.g. ADC calibration, analog readback, SLDO, Vcal calibration and injection capacitance.",
+    ),
+    mht: bool = typer.Option(
+        False,
+        "--mht",
+        help="Run scans required for Minimum Health Test.",
+    ),
+    tun: bool = typer.Option(
+        False,
+        "--tun",
+        help="Run scans required for Tuning.",
+    ),
+    pfa: bool = typer.Option(
+        False,
+        "--pfa",
+        help="Run scans required for Pixel Failure Analysis.",
+    ),
     # dry_run: bool = OPTIONS["dry_run"],
-    # CONTROLLER: Optional[Path] = OPTIONS["controller_config"],
-    # CONNECTIVITY: Path = OPTIONS["module_connectivity"],
-    # CONFIG: Path = OPTIONS["config_hw"],
     # _verbosity: LogLevel = OPTIONS["verbosity"]
 ):
     """
-    Main function to execute all QC scans.
+    Execute all QC scans if no test is specified.
+    If any tests are specified, only those are run.
 
     For instructions in case of core column issues, see https://codimd.web.cern.ch/s/iTKWoRkuP .
 
@@ -447,69 +451,64 @@ def main(
 
     qc = FullQCConf()
 
-    parser, args = qc.get_args()
-
-    if args.module_connectivity:
-        qc.CONNECTIVITY = args.module_connectivity
-
-    if args.config:
-        qc.CONFIG = args.config
-        with Path.open(qc.CONFIG, encoding="utf-8") as _hwconf:
-            qc.YARR = json.load(_hwconf)["yarr"]["run_dir"]
-
-    check_existence = [qc.CONNECTIVITY, qc.CONFIG]
-
-    for item in check_existence:
-        if not Path.exists(item):
-            logger.error("Path does not exist: %s!", item)
-            sys.exit(2)
-
-    if args.modulesn:
-        qc.MODULESN = args.modulesn
-
-    if args.basedir:
-        qc.BASEDIR = args.basedir
-
-    if args.controller:
-        qc.CONTROLLER = args.controller
-
-    if args.output:
-        qc.OUTPUT = args.output
-
-    if args.port:
-        qc.LDBPORT = args.port
-
-    if args.host:
-        qc.LDBHOST = args.host
-
-    if args.protocol:
-        qc.PROTOCOL = args.protocol
+    qc.CONNECTIVITY = CONNECTIVITY or qc.CONNECTIVITY
+    qc.CONFIG = CONFIG or qc.CONFIG
+    with Path.open(qc.CONFIG, encoding="utf-8") as _hwconf:
+        config_hw = json.load(_hwconf)
+        qc.YARR = Path(config_hw["yarr"]["run_dir"])
+        qc.CONTROLLER = Path(config_hw["yarr"]["controller"])
+
+    qc.MODULESN = MODULESN
+    qc.BASEDIR = BASEDIR or qc.BASEDIR
+    qc.MEASCONFIG = MEASCONFIG
+    qc.OUTPUT = OUTPUT or qc.OUTPUT
+
+    # Load hardware config and override parameters if provided
+    hw_config_data = load_localdb_config_from_hw(qc.CONFIG)
+    host = hw_config_data.get("host", LDBHOST)
+    port = hw_config_data.get("port", LDBPORT)
+    protocol = Protocol(hw_config_data.get("protocol", PROTOCOL.value))
+    # Merge tags from config with CLI tags
+    config_tags = hw_config_data.get("tags", [])
+
+    qc.LDBPORT = port or LDBPORT
+    qc.LDBHOST = host or LDBHOST
+    qc.PROTOCOL = protocol or PROTOCOL.value
+    qc.TAGS = list(set(TAGS + config_tags))
 
     if not qc.MODULESN:
         qc.MODULESN = (
             str(qc.CONNECTIVITY).rsplit("/", maxsplit=1)[-1].split("_", maxsplit=1)[0]
         )
-
     qc.CHIPTYPE = get_chip_type_from_serial_number(qc.MODULESN).lower()
     qc.TYPE = get_config_type_from_connectivity_path(qc.CONNECTIVITY)
     qc.LPTYPE = "_".join([qc.TYPE.split("_", maxsplit=1)[0], "LP"])
     qc.MODULEDIR = qc.CONNECTIVITY.parent
 
-    _tests = [group for group in parser._action_groups if "tests" in group.title]  # pylint: disable=protected-access
-    _tests = _tests[0]  ## type <class 'argparse._ArgumentGroup'>
-    logger.debug(_tests)
+    _tests = []
+    if iv:
+        _tests.append("iv")
+    if cal:
+        _tests.append("cal")
+    if mht:
+        _tests.append("mht")
+    if tun:
+        _tests.append("tun")
+    if pfa:
+        _tests.append("pfa")
 
     logger.info("Hardware config: %s", qc.CONFIG)
     logger.info("Module connectivity: %s", qc.CONNECTIVITY)
     logger.info("Output directory: %s", qc.OUTPUT)
-
+    logger.debug(locals())
     try:
         _cmd = ["analysis-check-kshunt-in-chip-config", "-c", qc.MODULEDIR]
         subprocess.run(_cmd, check=True)
     except Exception as e:  # pylint: disable=broad-exception-caught
         logger.warning("!!! Not able to update k-factors in the chip config %s !!!", e)
 
-    if not any(getattr(args, item.dest) for item in _tests._group_actions):  # pylint: disable=protected-access
+    ## if no test specified, then run full QC
+    if not any([iv, cal, mht, tun, pfa]):
         logger.info("No single tests specified, running full QC.")
         ## if not digital, run measure(IV)
         info = itksn.parse(qc.MODULESN.encode("utf-8"))
@@ -533,12 +532,9 @@ def main(
         qc.pfa()
         logger.info("Script completed successfully.")
     else:
-        for item in _tests._group_actions:  # pylint: disable=protected-access
-            ## e.g. _StoreTrueAction(option_strings=['-iv'], dest='iv', nargs=0, const=True, default=False, type=None, choices=None, required=False, help='Run sensor leakage current vs bias voltage.', metavar=None)
-            logger.debug(item)
-            logger.debug(type(item))
-
-            if getattr(args, item.dest):
+        for _index, item in enumerate(_tests):
+            # Run eye diagram only if the test is not IV
+            if item != "iv":
                 # Run eye diagram
                 subprocess.run(
                     [
@@ -551,8 +547,16 @@ def main(
                     cwd=qc.YARR,
                     check=True,
                 )
-                exec(f"qc.{item.dest}()")  # pylint: disable=exec-used
+            else:
+                # Skip IV if digital module
+                info = itksn.parse(qc.MODULESN.encode("utf-8"))
+                if "digital" in info.component_code.title().lower():
+                    logger.warning(
+                        "%s is a digital module, will not run IV.", qc.MODULESN
+                    )
+                    continue
+            exec(f"qc.{item}()")  # pylint: disable=exec-used
 
 
 if __name__ == "__main__":
-    main()
+    typer.run(main)