Skip to content

Mark turn-budget exhaustion as truncated - #72

Merged
flukeskywalker merged 3 commits into
prodfrom
agent/mark-turn-budget-truncated
Sep 9, 2026
Merged

flukeskywalker merged 3 commits into
prodfrom
agent/mark-turn-budget-truncated

Conversation

@nightlessbaron

Copy link
Copy Markdown

Summary

  • interpret Harbor exit_status=LimitsExceeded as rollout truncation
  • mark only the final sample TRUNCATED for multi-sample trajectories
  • retain agent metadata and all earlier completed turns

Validation

  • focused helper tests: 5 passed
  • added end-to-end generation coverage for merged and multi-sample variants
  • Ruff and isort passed
  • Black passed on the changed source, new test, and changed integration-test range
  • git diff --check passed

The full integration case could not collect locally because its fixture attempted an uncached Hugging Face tokenizer lookup in the network-restricted environment; CI will run it in the repository environment.

@nightlessbaron
nightlessbaron marked this pull request as ready for review August 28, 2026 17:21
@nightlessbaron
nightlessbaron requested a review from a team August 28, 2026 17:21
@flukeskywalker

Copy link
Copy Markdown

It seems merge_samples is not imported, according to CI

@flukeskywalker
flukeskywalker merged commit 341e38f into prod Sep 9, 2026
4 checks passed
@flukeskywalker
flukeskywalker deleted the agent/mark-turn-budget-truncated branch September 9, 2026 21:54
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.

2 participants