nominal 1.109.0__py3-none-any.whl → 1.111.0__py3-none-any.whl

CHANGELOG.md

@@ -1,5 +1,36 @@
 # Changelog
 
+## [1.111.0](https://github.com/nominal-io/nominal-client/compare/v1.110.0...v1.111.0) (2026-01-29)
+
+
+### Features
+
+* add RID mapping logging to migration utils ([#593](https://github.com/nominal-io/nominal-client/issues/593)) ([bf19a8b](https://github.com/nominal-io/nominal-client/commit/bf19a8b99f176d0d25586c9f134278a684cedcd2))
+* copy over bounds in clone dataset method ([#597](https://github.com/nominal-io/nominal-client/issues/597)) ([49f0cc1](https://github.com/nominal-io/nominal-client/commit/49f0cc119895768f48fd94a70719ad516bf51a94))
+
+
+### Bug Fixes
+
+* remove logical series service references ([#591](https://github.com/nominal-io/nominal-client/issues/591)) ([ec3ce3b](https://github.com/nominal-io/nominal-client/commit/ec3ce3bc6ddfad7a3e07cfc51f0681bbb719f952))
+
+
+### Documentation
+
+* update avro stream docstring with new schema types ([#594](https://github.com/nominal-io/nominal-client/issues/594)) ([690e5a3](https://github.com/nominal-io/nominal-client/commit/690e5a34dc782601df13d72034c452844fc644a4))
+
+## [1.110.0](https://github.com/nominal-io/nominal-client/compare/v1.109.0...v1.110.0) (2026-01-23)
+
+
+### Features
+
+* add data review/checklist methods to assets and runs ([#567](https://github.com/nominal-io/nominal-client/issues/567)) ([4b080d9](https://github.com/nominal-io/nominal-client/commit/4b080d96700fc51555b69a093d592cb03b92e073))
+* add video clone capability in experimental and refactor video methods ([#576](https://github.com/nominal-io/nominal-client/issues/576)) ([f58a89c](https://github.com/nominal-io/nominal-client/commit/f58a89c1fe2220677f439fdf306f0c743e441548))
+
+
+### Bug Fixes
+
+* properly handle dataset files in the new deletion states ([#590](https://github.com/nominal-io/nominal-client/issues/590)) ([e71bcc9](https://github.com/nominal-io/nominal-client/commit/e71bcc9f8fd752257dc2b33789380f0ffb7c5410))
+
 ## [1.109.0](https://github.com/nominal-io/nominal-client/compare/v1.108.0...v1.109.0) (2026-01-23)
 
 

nominal/core/_checklist_types.py

@@ -0,0 +1,48 @@
+from __future__ import annotations
+
+from enum import IntEnum
+
+from nominal_api import scout_api
+
+
+class Priority(IntEnum):
+    P0 = 0
+    P1 = 1
+    P2 = 2
+    P3 = 3
+    P4 = 4
+
+    @classmethod
+    def _from_conjure(cls, priority: scout_api.Priority) -> Priority:
+        match priority.name:
+            case "P0":
+                return cls.P0
+            case "P1":
+                return cls.P1
+            case "P2":
+                return cls.P2
+            case "P3":
+                return cls.P3
+            case "P4":
+                return cls.P4
+            case _:
+                raise ValueError(f"unknown priority '{priority}', expected one of {list(cls)}")
+
+    def _to_conjure(self) -> scout_api.Priority:
+        match self:
+            case Priority.P0:
+                return scout_api.Priority.P0
+            case Priority.P1:
+                return scout_api.Priority.P1
+            case Priority.P2:
+                return scout_api.Priority.P2
+            case Priority.P3:
+                return scout_api.Priority.P3
+            case Priority.P4:
+                return scout_api.Priority.P4
+            case _:
+                raise ValueError(f"unknown priority '{self}', expected one of {list(Priority)}")
+
+
+def _conjure_priority_to_priority(priority: scout_api.Priority) -> Priority:
+    return Priority._from_conjure(priority)

nominal/core/_clientsbunch.py

@@ -27,7 +27,6 @@ from nominal_api import (
     storage_datasource_api,
     storage_writer_api,
     timeseries_channelmetadata,
-    timeseries_logicalseries,
     timeseries_metadata,
     upload_api,
 )
@@ -124,7 +123,6 @@ class ClientsBunch:
     dataexport: scout_dataexport_api.DataExportService
     datasource: scout_datasource.DataSourceService
     ingest: ingest_api.IngestService
-    logical_series: timeseries_logicalseries.LogicalSeriesService
     run: scout.RunService
     units: scout.UnitsService
     upload: upload_api.UploadService
@@ -165,7 +163,6 @@ class ClientsBunch:
             dataexport=client_factory(scout_dataexport_api.DataExportService),
             datasource=client_factory(scout_datasource.DataSourceService),
             ingest=client_factory(ingest_api.IngestService),
-            logical_series=client_factory(timeseries_logicalseries.LogicalSeriesService),
             run=client_factory(scout.RunService),
             units=client_factory(scout.UnitsService),
             upload=client_factory(upload_api.UploadService),

nominal/core/_utils/api_tools.py

@@ -5,13 +5,15 @@ import importlib.metadata
 import logging
 import platform
 import sys
-from typing import Any, Generic, Mapping, Protocol, Sequence, TypeAlias, TypedDict, TypeVar, runtime_checkable
+from typing import Any, Generic, Literal, Mapping, Protocol, Sequence, TypeAlias, TypedDict, TypeVar, runtime_checkable
 
-from nominal_api import scout_compute_api, scout_run_api
+from nominal_api import scout_asset_api, scout_compute_api, scout_run_api
 from typing_extensions import NotRequired, Self
 
 from nominal._utils.dataclass_tools import update_dataclass
 
+ScopeTypeSpecifier: TypeAlias = Literal["connection", "dataset", "video"]
+
 logger = logging.getLogger(__name__)
 
 T = TypeVar("T")
@@ -94,3 +96,15 @@ def create_api_tags(tags: Mapping[str, str] | None = None) -> dict[str, scout_co
         return {}
 
     return {key: scout_compute_api.StringConstant(literal=value) for key, value in tags.items()}
+
+
+def filter_scopes(
+    scopes: Sequence[scout_asset_api.DataScope], scope_type: ScopeTypeSpecifier
+) -> Sequence[scout_asset_api.DataScope]:
+    return [scope for scope in scopes if scope.data_source.type.lower() == scope_type]
+
+
+def filter_scope_rids(scopes: Sequence[scout_asset_api.DataScope], scope_type: ScopeTypeSpecifier) -> Mapping[str, str]:
+    return {
+        scope.data_scope_name: getattr(scope.data_source, scope_type) for scope in filter_scopes(scopes, scope_type)
+    }

nominal/core/_video_types.py

@@ -0,0 +1,16 @@
+from dataclasses import dataclass
+
+from nominal.ts import IntegralNanosecondsUTC
+
+
+@dataclass(init=True, repr=False, eq=False, order=False, unsafe_hash=False)
+class McapVideoDetails:
+    mcap_channel_locator_topic: str
+
+
+@dataclass(init=True, repr=False, eq=False, order=False, unsafe_hash=False)
+class TimestampOptions:
+    starting_timestamp: IntegralNanosecondsUTC
+    ending_timestamp: IntegralNanosecondsUTC
+    scaling_factor: float
+    true_framerate: float

nominal/core/asset.py

@@ -4,7 +4,7 @@ import datetime
 import logging
 from dataclasses import dataclass, field
 from types import MappingProxyType
-from typing import Iterable, Literal, Mapping, Protocol, Sequence, TypeAlias
+from typing import Iterable, Mapping, Protocol, Sequence, TypeAlias
 
 from nominal_api import (
     event,
@@ -15,6 +15,7 @@ from nominal_api import (
 )
 from typing_extensions import Self
 
+from nominal.core import data_review, streaming_checklist
 from nominal.core._clientsbunch import HasScoutParams
 from nominal.core._event_types import EventType, SearchEventOriginType
 from nominal.core._utils.api_tools import (
@@ -22,7 +23,10 @@ from nominal.core._utils.api_tools import (
     Link,
     LinkDict,
     RefreshableMixin,
+    ScopeTypeSpecifier,
     create_links,
+    filter_scope_rids,
+    filter_scopes,
     rid_from_instance_or_string,
 )
 from nominal.core._utils.pagination_tools import search_runs_by_asset_paginated
@@ -35,25 +39,10 @@ from nominal.core.video import Video, _create_video, _get_video
 from nominal.ts import IntegralNanosecondsDuration, IntegralNanosecondsUTC, _SecondsNanos
 
 ScopeType: TypeAlias = Connection | Dataset | Video
-ScopeTypeSpecifier: TypeAlias = Literal["connection", "dataset", "video"]
 
 logger = logging.getLogger(__name__)
 
 
-def _filter_scopes(
-    scopes: Sequence[scout_asset_api.DataScope], scope_type: ScopeTypeSpecifier
-) -> Sequence[scout_asset_api.DataScope]:
-    return [scope for scope in scopes if scope.data_source.type.lower() == scope_type]
-
-
-def _filter_scope_rids(
-    scopes: Sequence[scout_asset_api.DataScope], scope_type: ScopeTypeSpecifier
-) -> Mapping[str, str]:
-    return {
-        scope.data_scope_name: getattr(scope.data_source, scope_type) for scope in _filter_scopes(scopes, scope_type)
-    }
-
-
 @dataclass(frozen=True)
 class Asset(_DatasetWrapper, HasRid, RefreshableMixin[scout_asset_api.Asset]):
     rid: str
@@ -70,6 +59,7 @@ class Asset(_DatasetWrapper, HasRid, RefreshableMixin[scout_asset_api.Asset]):
         Video._Clients,
         Attachment._Clients,
         Event._Clients,
+        data_review.DataReview._Clients,
         HasScoutParams,
         Protocol,
     ):
@@ -94,11 +84,11 @@ class Asset(_DatasetWrapper, HasRid, RefreshableMixin[scout_asset_api.Asset]):
         return response[self.rid]
 
     def _list_dataset_scopes(self) -> Sequence[scout_asset_api.DataScope]:
-        return _filter_scopes(self._get_latest_api().data_scopes, "dataset")
+        return filter_scopes(self._get_latest_api().data_scopes, "dataset")
 
     def _scope_rids(self, scope_type: ScopeTypeSpecifier) -> Mapping[str, str]:
         asset = self._get_latest_api()
-        return _filter_scope_rids(asset.data_scopes, scope_type)
+        return filter_scope_rids(asset.data_scopes, scope_type)
 
     def update(
         self,
@@ -540,6 +530,32 @@ class Asset(_DatasetWrapper, HasRid, RefreshableMixin[scout_asset_api.Asset]):
             origin_types=origin_types,
         )
 
+    def search_data_reviews(
+        self,
+        runs: Sequence[Run | str] | None = None,
+    ) -> Sequence[data_review.DataReview]:
+        """Search for data reviews associated with this Asset. See nominal.core.client.search_data_reviews
+        for details.
+        """
+        return list(
+            data_review._iter_search_data_reviews(
+                self._clients,
+                assets=[self.rid],
+                runs=[rid_from_instance_or_string(run) for run in (runs or [])],
+            )
+        )
+
+    def list_streaming_checklists(self) -> Sequence[str]:
+        """List all Streaming Checklists associated with this Asset. See
+        nominal.core.client.list_streaming_checklists for details.
+        """
+        return list(
+            streaming_checklist._iter_list_streaming_checklists(
+                self._clients,
+                asset_rid=self.rid,
+            )
+        )
+
     def remove_attachments(self, attachments: Iterable[Attachment] | Iterable[str]) -> None:
         """Remove attachments from this asset.
         Does not remove the attachments from Nominal.

nominal/core/bounds.py

@@ -2,7 +2,7 @@ from __future__ import annotations
 
 from dataclasses import dataclass
 
-from nominal_api import scout_catalog
+from nominal_api import datasource, scout_catalog
 from typing_extensions import Self
 
 from nominal.ts import (
@@ -22,3 +22,10 @@ class Bounds:
             start=_SecondsNanos.from_api(bounds.start).to_nanoseconds(),
             end=_SecondsNanos.from_api(bounds.end).to_nanoseconds(),
         )
+
+    def _to_conjure(self) -> scout_catalog.Bounds:
+        return scout_catalog.Bounds(
+            type=datasource.TimestampType.ABSOLUTE,
+            start=_SecondsNanos.from_nanoseconds(self.start).to_api(),
+            end=_SecondsNanos.from_nanoseconds(self.end).to_api(),
+        )

nominal/core/channel.py

@@ -15,7 +15,6 @@ from nominal_api import (
     storage_series_api,
     timeseries_channelmetadata,
     timeseries_channelmetadata_api,
-    timeseries_logicalseries_api,
 )
 from typing_extensions import Self
 
@@ -298,20 +297,6 @@ class Channel(RefreshableMixin[timeseries_channelmetadata_api.ChannelMetadata]):
             _clients=clients,
         )
 
-    @classmethod
-    def _from_conjure_logicalseries_api(
-        cls, clients: _Clients, series: timeseries_logicalseries_api.LogicalSeries
-    ) -> Self:
-        channel_data_type = ChannelDataType._from_conjure(series.series_data_type) if series.series_data_type else None
-        return cls(
-            name=series.channel,
-            data_source=series.data_source_rid,
-            unit=series.unit,
-            description=series.description,
-            data_type=channel_data_type,
-            _clients=clients,
-        )
-
     @classmethod
     def _from_channel_metadata_api(
         cls, clients: _Clients, channel: timeseries_channelmetadata_api.ChannelMetadata

nominal/core/checklist.py

@@ -2,10 +2,11 @@ from __future__ import annotations
 
 from dataclasses import dataclass, field
 from datetime import timedelta
-from typing import Literal, Mapping, Protocol, Sequence
+from typing import Mapping, Protocol, Sequence
 
 from nominal_api import (
-    scout_api,
+    event,
+    scout,
     scout_checklistexecution_api,
     scout_checks_api,
     scout_datareview_api,
@@ -13,11 +14,11 @@ from nominal_api import (
 )
 from typing_extensions import Self
 
+from nominal.core import run as core_run
 from nominal.core._clientsbunch import HasScoutParams
 from nominal.core._utils.api_tools import HasRid, rid_from_instance_or_string
 from nominal.core.asset import Asset
 from nominal.core.data_review import DataReview
-from nominal.core.run import Run
 from nominal.ts import _to_api_duration
 
 
@@ -30,13 +31,17 @@ class Checklist(HasRid):
     labels: Sequence[str]
     _clients: _Clients = field(repr=False)
 
-    class _Clients(DataReview._Clients, HasScoutParams, Protocol):
+    class _Clients(HasScoutParams, Protocol):
         @property
         def checklist(self) -> scout_checks_api.ChecklistService: ...
         @property
         def checklist_execution(self) -> scout_checklistexecution_api.ChecklistExecutionService: ...
         @property
         def datareview(self) -> scout_datareview_api.DataReviewService: ...
+        @property
+        def event(self) -> event.EventService: ...
+        @property
+        def run(self) -> scout.RunService: ...
 
     @classmethod
     def _from_conjure(cls, clients: _Clients, checklist: scout_checks_api.VersionedChecklist) -> Self:
@@ -53,7 +58,7 @@ class Checklist(HasRid):
             _clients=clients,
         )
 
-    def execute(self, run: Run | str, commit: str | None = None) -> DataReview:
+    def execute(self, run: core_run.Run | str, commit: str | None = None) -> DataReview:
         """Execute a checklist against a run.
 
         Args:
@@ -152,26 +157,7 @@ class Checklist(HasRid):
         """Returns a link to the page for this checklist in the Nominal app"""
         return f"{self._clients.app_base_url}/checklists/{self.rid}"
 
-    def preview_for_run_url(self, run: Run | str) -> str:
+    def preview_for_run_url(self, run: core_run.Run | str) -> str:
         """Returns a link to the page for previewing this checklist on a given run in the Nominal app"""
         run_rid = rid_from_instance_or_string(run)
         return f"{self.nominal_url}?previewRunRid={run_rid}"
-
-
-Priority = Literal[0, 1, 2, 3, 4]
-
-
-_priority_to_conjure_map: dict[Priority, scout_api.Priority] = {
-    0: scout_api.Priority.P0,
-    1: scout_api.Priority.P1,
-    2: scout_api.Priority.P2,
-    3: scout_api.Priority.P3,
-    4: scout_api.Priority.P4,
-}
-
-
-def _conjure_priority_to_priority(priority: scout_api.Priority) -> Priority:
-    inverted_map = {v: k for k, v in _priority_to_conjure_map.items()}
-    if priority in inverted_map:
-        return inverted_map[priority]
-    raise ValueError(f"unknown priority '{priority}', expected one of {_priority_to_conjure_map.values()}")

nominal/core/client.py

@@ -48,11 +48,8 @@ from nominal.core._utils.multipart import (
     upload_multipart_io,
 )
 from nominal.core._utils.pagination_tools import (
-    list_streaming_checklists_for_asset_paginated,
-    list_streaming_checklists_paginated,
     search_assets_paginated,
     search_checklists_paginated,
-    search_data_reviews_paginated,
     search_datasets_paginated,
     search_runs_by_asset_paginated,
     search_runs_paginated,
@@ -84,7 +81,7 @@ from nominal.core.containerized_extractors import (
     FileExtractionInput,
     FileOutputFormat,
 )
-from nominal.core.data_review import DataReview, DataReviewBuilder
+from nominal.core.data_review import DataReview, DataReviewBuilder, _iter_search_data_reviews
 from nominal.core.dataset import (
     Dataset,
     _create_dataset,
@@ -97,6 +94,7 @@ from nominal.core.exceptions import NominalConfigError, NominalError, NominalMet
 from nominal.core.filetype import FileType, FileTypes
 from nominal.core.run import Run, _create_run
 from nominal.core.secret import Secret
+from nominal.core.streaming_checklist import _iter_list_streaming_checklists
 from nominal.core.unit import Unit, _available_units
 from nominal.core.user import User
 from nominal.core.video import Video, _create_video
@@ -1162,21 +1160,17 @@ class NominalClient:
         )
         return list(self._iter_search_assets(query))
 
-    def _iter_list_streaming_checklists(self, asset: str | None) -> Iterable[str]:
-        if asset is None:
-            return list_streaming_checklists_paginated(self._clients.checklist_execution, self._clients.auth_header)
-        return list_streaming_checklists_for_asset_paginated(
-            self._clients.checklist_execution, self._clients.auth_header, asset
-        )
-
-    def list_streaming_checklists(self, asset: Asset | str | None = None) -> Iterable[str]:
+    def list_streaming_checklists(self, asset: Asset | str | None = None) -> Sequence[str]:
         """List all Streaming Checklists.
 
         Args:
             asset: if provided, only return checklists associated with the given asset.
+
+        Returns:
+            All streaming checklist RIDs that match the provided conditions
         """
-        asset = None if asset is None else rid_from_instance_or_string(asset)
-        return list(self._iter_list_streaming_checklists(asset))
+        asset_rid = None if asset is None else rid_from_instance_or_string(asset)
+        return list(_iter_list_streaming_checklists(self._clients, asset_rid))
 
     def data_review_builder(self) -> DataReviewBuilder:
         return DataReviewBuilder([], [], [], _clients=self._clients)
@@ -1220,27 +1214,29 @@ class NominalClient:
         responses = self._clients.event.batch_get_events(self._clients.auth_header, list(rids))
         return [Event._from_conjure(self._clients, response) for response in responses]
 
-    def _iter_search_data_reviews(
-        self,
-        assets: Sequence[Asset | str] | None = None,
-        runs: Sequence[Run | str] | None = None,
-    ) -> Iterable[DataReview]:
-        for review in search_data_reviews_paginated(
-            self._clients.datareview,
-            self._clients.auth_header,
-            assets=[rid_from_instance_or_string(asset) for asset in assets] if assets else None,
-            runs=[rid_from_instance_or_string(run) for run in runs] if runs else None,
-        ):
-            yield DataReview._from_conjure(self._clients, review)
-
     def search_data_reviews(
         self,
         assets: Sequence[Asset | str] | None = None,
         runs: Sequence[Run | str] | None = None,
     ) -> Sequence[DataReview]:
-        """Search for any data reviews present within a collection of runs and assets."""
+        """Search for data reviews meeting the specified filters.
+        Filters are ANDed together, e.g. `(data_review.asset == asset) AND (data_review.run == run)`
+
+        Args:
+            assets: List of assets that must be associated with a data review to be included.
+            runs: List of runs that must be associated with a data review to be included.
+
+        Returns:
+            All data reviews which match all of the provided conditions
+        """
         # TODO (drake-nominal): Expose checklist_refs to users
-        return list(self._iter_search_data_reviews(assets, runs))
+        return list(
+            _iter_search_data_reviews(
+                clients=self._clients,
+                assets=[rid_from_instance_or_string(asset) for asset in assets] if assets else None,
+                runs=[rid_from_instance_or_string(run) for run in runs] if runs else None,
+            )
+        )
 
     def search_events(
         self,

nominal/core/data_review.py

@@ -3,7 +3,7 @@ from __future__ import annotations
 from dataclasses import dataclass, field
 from datetime import timedelta
 from time import sleep
-from typing import Protocol, Sequence
+from typing import TYPE_CHECKING, Iterable, Protocol, Sequence
 
 from nominal_api import (
     event as event_api,
@@ -18,14 +18,19 @@ from nominal_api import (
 )
 from typing_extensions import Self, deprecated
 
-from nominal.core import checklist, event
+from nominal.core._checklist_types import Priority, _conjure_priority_to_priority
 from nominal.core._clientsbunch import HasScoutParams
 from nominal.core._utils.api_tools import HasRid, rid_from_instance_or_string
-from nominal.core.asset import Asset
+from nominal.core._utils.pagination_tools import search_data_reviews_paginated
+from nominal.core.event import Event
 from nominal.core.exceptions import NominalMethodRemovedError
-from nominal.core.run import Run
 from nominal.ts import IntegralNanosecondsUTC, _SecondsNanos
 
+if TYPE_CHECKING:
+    from nominal.core.asset import Asset
+    from nominal.core.checklist import Checklist
+    from nominal.core.run import Run
+
 
 @dataclass(frozen=True)
 class DataReview(HasRid):
@@ -66,8 +71,10 @@ class DataReview(HasRid):
             _clients=clients,
         )
 
-    def get_checklist(self) -> checklist.Checklist:
-        return checklist.Checklist._from_conjure(
+    def get_checklist(self) -> "Checklist":
+        from nominal.core.checklist import Checklist
+
+        return Checklist._from_conjure(
             self._clients,
             self._clients.checklist.get(self._clients.auth_header, self.checklist_rid, commit=self.checklist_commit),
         )
@@ -83,7 +90,7 @@ class DataReview(HasRid):
             "use 'nominal.core.DataReview.get_events()' instead",
         )
 
-    def get_events(self) -> Sequence[event.Event]:
+    def get_events(self) -> Sequence[Event]:
         """Retrieves the list of events for the data review."""
         data_review_response = self._clients.datareview.get(self._clients.auth_header, self.rid).check_evaluations
         all_event_rids = [
@@ -93,7 +100,7 @@ class DataReview(HasRid):
             for event_rid in check.state._generated_alerts.event_rids
         ]
         event_response = self._clients.event.batch_get_events(self._clients.auth_header, all_event_rids)
-        return [event.Event._from_conjure(self._clients, data_review_event) for data_review_event in event_response]
+        return [Event._from_conjure(self._clients, data_review_event) for data_review_event in event_response]
 
     def reload(self) -> DataReview:
         """Reloads the data review from the server."""
@@ -137,7 +144,7 @@ class CheckViolation:
     name: str
     start: IntegralNanosecondsUTC
     end: IntegralNanosecondsUTC | None
-    priority: checklist.Priority | None
+    priority: Priority | None
 
     @classmethod
     def _from_conjure(cls, check_alert: scout_datareview_api.CheckAlert) -> CheckViolation:
@@ -147,7 +154,7 @@ class CheckViolation:
             name=check_alert.name,
             start=_SecondsNanos.from_api(check_alert.start).to_nanoseconds(),
             end=_SecondsNanos.from_api(check_alert.end).to_nanoseconds() if check_alert.end is not None else None,
-            priority=checklist._conjure_priority_to_priority(check_alert.priority)
+            priority=_conjure_priority_to_priority(check_alert.priority)
             if check_alert.priority is not scout_api.Priority.UNKNOWN
             else None,
         )
@@ -192,7 +199,7 @@ class DataReviewBuilder:
     def execute_checklist(
         self,
         run: str | Run,
-        checklist: str | checklist.Checklist,
+        checklist: str | Checklist,
         *,
         commit: str | None = None,
         asset: str | Asset | None = None,
@@ -273,3 +280,17 @@ def poll_until_completed(
     data_reviews: Sequence[DataReview], interval: timedelta = timedelta(seconds=2)
 ) -> Sequence[DataReview]:
     return [review.poll_for_completion(interval) for review in data_reviews]
+
+
+def _iter_search_data_reviews(
+    clients: DataReview._Clients,
+    assets: Sequence[str] | None = None,
+    runs: Sequence[str] | None = None,
+) -> Iterable[DataReview]:
+    for review in search_data_reviews_paginated(
+        clients.datareview,
+        clients.auth_header,
+        assets=assets,
+        runs=runs,
+    ):
+        yield DataReview._from_conjure(clients, review)