nominal 1.104.0__py3-none-any.whl → 1.104.1__py3-none-any.whl

CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [1.104.1](https://github.com/nominal-io/nominal-client/compare/v1.104.0...v1.104.1) (2026-01-08)
+
+
+### Bug Fixes
+
+* add asset_rid to data_review request ([#508](https://github.com/nominal-io/nominal-client/issues/508)) ([b1baf4f](https://github.com/nominal-io/nominal-client/commit/b1baf4f3745ee8a485ff0407a53ad9ee1ee9b0fc))
+
 ## [1.104.0](https://github.com/nominal-io/nominal-client/compare/v1.103.0...v1.104.0) (2026-01-08)
 
 

nominal/core/data_review.py

@@ -20,8 +20,10 @@ from typing_extensions import Self, deprecated
 
 from nominal.core import checklist, event
 from nominal.core._clientsbunch import HasScoutParams
-from nominal.core._utils.api_tools import HasRid
+from nominal.core._utils.api_tools import HasRid, rid_from_instance_or_string
+from nominal.core.asset import Asset
 from nominal.core.exceptions import NominalMethodRemovedError
+from nominal.core.run import Run
 from nominal.ts import IntegralNanosecondsUTC, _SecondsNanos
 
 
@@ -162,8 +164,81 @@ class DataReviewBuilder:
         self._integration_rids.append(integration_rid)
         return self
 
-    def add_request(self, run_rid: str, checklist_rid: str, commit: str) -> DataReviewBuilder:
-        self._requests.append(scout_datareview_api.CreateDataReviewRequest(checklist_rid, run_rid, commit))
+    @deprecated(
+        "`DataReviewBuilder.add_request` is deprecated and will be removed in a future release of the Nominal SDK. "
+        "Please use `DataReviewBuilder.execute_checklist` instead."
+    )
+    def add_request(
+        self,
+        run_rid: str,
+        checklist_rid: str,
+        commit: str | None = None,
+        *,
+        asset_rid: str | None = None,
+    ) -> DataReviewBuilder:
+        """Add a request to create a data review for the given checklist and run.
+
+        Args:
+            run_rid: RID of the Run to run the Checklist on
+            checklist_rid: RID of the checklist to execute on the Run
+            commit: Commit hash of the version of the checklist to run, or the latest version if None is provided
+            asset_rid: RID of the asset to run the checklist on within the Run (only required for multi-asset runs)
+
+        Returns:
+            DataReviewBuilder instance to continue building a data review with
+        """
+        return self.execute_checklist(run_rid, checklist_rid, commit=commit, asset=asset_rid)
+
+    def execute_checklist(
+        self,
+        run: str | Run,
+        checklist: str | checklist.Checklist,
+        *,
+        commit: str | None = None,
+        asset: str | Asset | None = None,
+    ) -> Self:
+        """Add a request to create a data review for the given checklist and run.
+
+        Args:
+            run: Instance or rid of the Run to run the Checklist on
+            checklist: Instance or rid of the checklist to execute on the Run
+            commit: Commit hash of the version of the checklist to run, or the latest version if None is provided
+            asset: Instance or rid of the asset to run the checklist on within the Run
+                NOTE: only required for multi-asset runs
+
+        Returns:
+            DataReviewBuilder instance to continue building a data review with
+
+        Raises:
+            ValueError: If the given run has multiple associated assets and no asset was specified, or
+                if the run has an asset that differs from the provided asset.
+        """
+        checklist_rid = rid_from_instance_or_string(checklist)
+        run_rid = rid_from_instance_or_string(run)
+        asset_rid = None if asset is None else rid_from_instance_or_string(asset)
+
+        raw_run = self._clients.run.get_run(self._clients.auth_header, run_rid)
+        if len(raw_run.assets) > 1 and asset is None:
+            raise ValueError(
+                f"Cannot run data review on checklist {checklist_rid} and {run_rid} without specifying `asset_rid`: "
+                f"run has {len(raw_run.assets)} assets!"
+            )
+        elif len(raw_run.assets) == 1 and asset_rid is not None:
+            raw_asset_rid = raw_run.assets[0]
+            if raw_asset_rid != asset_rid:
+                raise ValueError(
+                    f"Cannot run data review on checklist {checklist_rid} and {run_rid} with asset {asset_rid}: "
+                    f"run has a different asset {raw_asset_rid}!"
+                )
+
+        self._requests.append(
+            scout_datareview_api.CreateDataReviewRequest(
+                checklist_rid=checklist_rid,
+                run_rid=run_rid,
+                asset_rid=asset_rid,
+                commit=commit,
+            )
+        )
         return self
 
     def add_tags(self, tags: list[str]) -> DataReviewBuilder:
@@ -174,8 +249,7 @@ class DataReviewBuilder:
         """Initiates a batch data review process.
 
         Args:
-            wait_for_completion (bool): If True, waits for the data review process to complete before returning.
-                                        Default is True.
+            wait_for_completion: If True, waits for the data review process to complete before returning.
         """
         request = scout_datareview_api.BatchInitiateDataReviewRequest(
             notification_configurations=[

{nominal-1.104.0.dist-info → nominal-1.104.1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nominal
-Version: 1.104.0
+Version: 1.104.1
 Summary: Automate Nominal workflows in Python
 Project-URL: Homepage, https://nominal.io
 Project-URL: Documentation, https://docs.nominal.io

{nominal-1.104.0.dist-info → nominal-1.104.1.dist-info}/RECORD

@@ -1,4 +1,4 @@
-CHANGELOG.md,sha256=SNK-cRIerr4jSucyICY6m4WdOg1StL3J6qeqFd0TvIQ,86878
+CHANGELOG.md,sha256=NlPcTaaXpq78bnNrncsHImQ2b6R6guUk3eRPS5RDOa4,87207
 LICENSE,sha256=zEGHG9mjDjaIS3I79O8mweQo-yiTbqx8jJvUPppVAwk,1067
 README.md,sha256=KKe0dxh_pHXCtB7I9G4qWGQYvot_BZU8yW6MJyuyUHM,311
 nominal/__init__.py,sha256=rbraORnXUrNn1hywLXM0XwSQCd9UmQt20PDYlsBalfE,2167
@@ -40,7 +40,7 @@ nominal/core/checklist.py,sha256=rO1RPDYV3o2miPKF7DcCiYpj6bUN-sdtZNhJkXzkfYE,711
 nominal/core/client.py,sha256=Awt9WPkE-YXBfOwJMTL7Su8AZFJY3UMH7IKp5hI26YQ,68328
 nominal/core/connection.py,sha256=LYllr3a1H2xp8-i4MaX1M7yK8X-HnwuIkciyK9XgLtQ,5175
 nominal/core/containerized_extractors.py,sha256=fUz3-NHoNWYKqOCD15gLwGXDKVfdsW-x_kpXnkOI3BE,10224
-nominal/core/data_review.py,sha256=bEnRsd8LI4x9YOBPcF2H3h5-e12A7Gh8gQfsNUAZmPQ,7922
+nominal/core/data_review.py,sha256=Z_W1Okp_FSQDiVCk6aKb9gV0EXbE2jtiQaPqc6TaL0g,11038
 nominal/core/dataset.py,sha256=iOR-jiTrnUxTnsF_eteQCHQmbe8IZovdSzhAgq40bB8,46648
 nominal/core/dataset_file.py,sha256=8rCW6MO89MFbQ2NH0WtFWmJfRWeTxhmyuoGojuQQ4Qg,16545
 nominal/core/datasource.py,sha256=V5UahbqsCNIdml978kOHiY6boIxKxbp76KscNBpN5xc,16934
@@ -106,8 +106,8 @@ nominal/thirdparty/polars/polars_export_handler.py,sha256=hGCSwXX9dC4MG01CmmjlTb
 nominal/thirdparty/tdms/__init__.py,sha256=6n2ImFr2Wiil6JM1P5Q7Mpr0VzLcnDkmup_ftNpPq-s,142
 nominal/thirdparty/tdms/_tdms.py,sha256=m4gxbpxB9MTLi2FuYvGlbUGSyDAZKFxbM3ia2x1wIz0,8746
 nominal/ts/__init__.py,sha256=hmd0ENvDhxRnzDKGLxIub6QG8LpcxCgcyAct029CaEs,21442
-nominal-1.104.0.dist-info/METADATA,sha256=xfxtanw2lJqoSn3Nc68VIvpwdD6rkArysEa5_xP0JwE,2277
-nominal-1.104.0.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-nominal-1.104.0.dist-info/entry_points.txt,sha256=-mCLhxgg9R_lm5efT7vW9wuBH12izvY322R0a3TYxbE,66
-nominal-1.104.0.dist-info/licenses/LICENSE,sha256=zEGHG9mjDjaIS3I79O8mweQo-yiTbqx8jJvUPppVAwk,1067
-nominal-1.104.0.dist-info/RECORD,,
+nominal-1.104.1.dist-info/METADATA,sha256=pSumzRRZWU7UoEqsEQKfpc-kmXVxuB1iLChuhBhHhNA,2277
+nominal-1.104.1.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
+nominal-1.104.1.dist-info/entry_points.txt,sha256=-mCLhxgg9R_lm5efT7vW9wuBH12izvY322R0a3TYxbE,66
+nominal-1.104.1.dist-info/licenses/LICENSE,sha256=zEGHG9mjDjaIS3I79O8mweQo-yiTbqx8jJvUPppVAwk,1067
+nominal-1.104.1.dist-info/RECORD,,