Skip to content

Let workflow-eval --wait run without a time limit and show progress - #562

Draft
joaomarcoscrs wants to merge 1 commit into
mainfrom
joao/feat/workflow-eval-wait-unlimited
Draft

joaomarcoscrs wants to merge 1 commit into
mainfrom
joao/feat/workflow-eval-wait-unlimited

Conversation

@joaomarcoscrs

@joaomarcoscrs joaomarcoscrs commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

What

  • wait_for_run(timeout=None or 0) waits without limit; positive timeouts still raise TimeoutError, SDK default stays 3600 s.
  • workflow-eval run get/start --wait: --timeout 0 means no limit; without --timeout, Runs on a local inference server (executionMode: browser) wait without limit and server Runs keep 3600 s.
  • Progress line on stderr at most every 10 s while waiting (suppressed with --json / --quiet).
  • submit_capture: runtime required in body; optional HTTP timeout since scoring runs inside the request.
  • CLI-COMMANDS.md updated.

Why

F1 of "evals on device": local runs (~3 h on a local inference server) must be driven by the CLI in the background, without the browser and without hitting the 1 h wait cap.

How tested

pytest (61 passed, 1 skipped), ruff format --check, ruff check, mypy.

Related (F1)

- wait_for_run(timeout=None or 0) waits without limit; positive timeouts
  still raise TimeoutError and the SDK default stays 3600 s.
- run get/start --wait: --timeout 0 means no limit; without --timeout,
  Runs on a local inference server (executionMode browser) wait without
  limit and server Runs keep 3600 s.
- Print a progress line on stderr at most every 10 s while waiting,
  suppressed with --json and --quiet.
- submit_capture: runtime is required in the body; accept an optional
  HTTP timeout because scoring runs inside the request.
@joaomarcoscrs joaomarcoscrs self-assigned this Oct 22, 2026
Comment thread roboflow/cli/handlers/workflow_eval.py Dismissed
@joaomarcoscrs

Copy link
Copy Markdown
Contributor Author

[Jarbas Local João] — APPROVED at fa7dd958ee94

Gist: https://gist.github.com/joaomarcoscrs/1c89efa727628a195da6e6eb47768ed2

This branch has not been deployed

No deployments
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