Skip to content

docs: correct manager/auditor episode timeout defaults in both READMEs - #65

Open
Huang-404-Q wants to merge 1 commit into
AMAP-ML:mainfrom
Huang-404-Q:fix-readme-budget-timeouts
Open

Huang-404-Q wants to merge 1 commit into
AMAP-ML:mainfrom
Huang-404-Q:fix-readme-budget-timeouts

Conversation

@Huang-404-Q

Copy link
Copy Markdown

What

The \[run.timeouts\] tables in both READMEs documented 600 as the default episode timeout for manager and auditor. The actual default is 300 seconds:

  • src/lh_harness/cli.py \_BUDGET_OPTIONS = (("manager", 300), ("gui_executor", 1800), ("cli_executor", 1800), ("auditor", 300))
  • src/lh_harness/config.py CONFIG_TEMPLATE writes manager = 300 / auditor = 300
  • lh-harness run --help prints the same defaults (they are derived from the same constant)

Why

An operator budgeting on the documented 600s would see Manager and Auditor episodes stop at 5 minutes and look for a bug that does not exist. The two executor rows (1800) were already correct.

How I checked

  • Compared all four documented values against cli.py and config.py
  • git diff --check clean; full test suite passes (405 passed, 1 skipped)

Documentation only — no behavior change.

prax211 added a commit to cogniziocompany/LongHorizon-Harness that referenced this pull request Sep 5, 2026
…nd auth-broker (AMAP-ML#65)

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant