Skip to content

fix: resume valid requested modes from partial summaries - #29

Merged
cuizy3 merged 2 commits into
google-research:mainfrom
rudycelekli:fix/resume-requested-modes
Nov 2, 2026
Merged

cuizy3 merged 2 commits into
google-research:mainfrom
rudycelekli:fix/resume-requested-modes

Conversation

@rudycelekli

Copy link
Copy Markdown
Contributor

A summary can contain a completed parallel mode and a failed sequential selector because independent mode records accumulate in the same file. The resume gate rejects the global partial status before examining the requested modes, so subsequent parallel-only runs repeatedly execute an already valid result while the unrequested sequential failure keeps the merged summary partial.

Allow partial summaries to reach the existing requested-mode and artifact-integrity checks. Completed requested modes can resume; failed requested modes still retry, and base-failed or malformed summaries remain excluded.

Validation uses real offline Demo execution with a selector that fails only for sequential selection. The baseline parallel-only retry executes again; fixed retries skip with unchanged executor and proposer counts. Explicitly requesting the failed sequential mode still retries and reports partial. Focused regression and all 6 native tests pass; whole Ruff, touched formatting, and diff checks pass under an OS network fence. No paid model/executor work.

@cuizy3
cuizy3 merged commit be21d03 into google-research:main Nov 2, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants