Skip to content

Fix documented default round limit - #44

Open
ssynb wants to merge 1 commit into
AMAP-ML:mainfrom
ssynb:docs/fix-default-max-rounds
Open

ssynb wants to merge 1 commit into
AMAP-ML:mainfrom
ssynb:docs/fix-default-max-rounds

Conversation

@ssynb

@ssynb ssynb commented Aug 17, 2026

Copy link
Copy Markdown

Summary

  • update the English and Chinese configuration tables to show the actual default of 25 rounds
  • update both CLI reference tables to the same value

Evidence

DEFAULT_MAX_ROUNDS is 25 in src/lh_harness/types.py, run.max_rounds defaults to 25 in src/lh_harness/config.py, and the CLI derives its fallback and help text from that constant. The READMEs still documented 30.

Validation

  • git diff --check
  • compared all four documented values against src/lh_harness/types.py and src/lh_harness/config.py

No runtime code is changed, so no test suite was run.

prax211 added a commit to cogniziocompany/LongHorizon-Harness that referenced this pull request Sep 6, 2026
…uns + QA ref fix AMAP-ML#44/AMAP-ML#45

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
prax211 added a commit to cogniziocompany/LongHorizon-Harness that referenced this pull request Sep 6, 2026
 closed; promotion run fixes AMAP-ML#44-AMAP-ML#49

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
prax211 added a commit to cogniziocompany/LongHorizon-Harness that referenced this pull request Sep 8, 2026
…ML#44)

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