Skip to content

Add Workflow Eval Dataset search to the Python SDK - #578

Merged
joaomarcoscrs merged 1 commit into
mainfrom
joao/wf-evals-dataset-search
Nov 4, 2026
Merged

joaomarcoscrs merged 1 commit into
mainfrom
joao/wf-evals-dataset-search

Conversation

@joaomarcoscrs

Copy link
Copy Markdown
Contributor

Description

  • Expose the Dataset search API added in roboflow/roboflow#17658 through WorkflowEvals.search_dataset(dataset_id, query=..., limit=..., cursor=...).
  • Support semantic, Source and saved ground-truth filters. Preserve coverage, Dataset revisions, nullable totals and continuation tokens, including empty pages.
  • Document filtering, conditional image scope and pagination in the workspace SDK guide.

Type of change

  • New feature (non-breaking change which adds functionality)

How has this change been tested, please provide a testcase or example of how you tested the change?

  • 19 Workflow Evals tests passed, including HTTP request/response coverage for the new route and empty-page continuation.
  • Ruff formatting and lint passed for both changed Python files.
  • Mypy passed for roboflow/core/workflow_evals.py.
  • Live API validation pending.

Will the change affect Universe? If so was this change tested in universe?

No

Any specific deployment considerations

  • Publish a Python SDK release.
  • Requires the Dataset search endpoint from roboflow/roboflow#17658 to be deployed.

Infrastructure impact

  • This change affects infrastructure needs (GPUs, Cloud Function sizing, storage etc.)

Docs

  • Updated the workspace SDK guide with a complete pagination example.

@joaomarcoscrs joaomarcoscrs self-assigned this Nov 4, 2026
@joaomarcoscrs

Copy link
Copy Markdown
Contributor Author

[Jarbas Local João] — APPROVED at 12d1fdf9a52e

Gist: https://gist.github.com/joaomarcoscrs/228308394a3bec8886b72f97b6a13cf3

@joaomarcoscrs joaomarcoscrs added the LGTJarb Approved by Jarbas Local label Nov 4, 2026
@joaomarcoscrs
joaomarcoscrs marked this pull request as ready for review November 4, 2026 19:26
@joaomarcoscrs
joaomarcoscrs merged commit 506c2f8 into main Nov 4, 2026
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTJarb Approved by Jarbas Local

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants