docs: preflop RvR high-iter rerun (bug confirmed via 1000-iter; 10000-iter killed)#164
Merged
Merged
Conversation
Empirical investigation of degenerate preflop strategy hypothesis. 100-iter (post PR #157 .so) results with initial_contributions:[50,100] config: - SB J7s root: 99.99% fold (target: 20-40% fold) - BB KdQd vs 3bb: 71.71% fold (target: <=20% fold) - BB 8s9s vs 3bb: 98.76% fold (target: ~40% fold) - Trajectory: 92.12% (10 iters) -> 99.99% (100 iters); strategy NARROWS to fold VERDICT: bug confirmed, NOT a convergence issue. Direction of travel is toward more fold not less, consistent with PR #159 RCA (leaf payoffs collapse to ~0 when initial_contributions matches the blinds, so folding becomes the no-loss dominant action). 10000-iter run was launched in parallel (still in flight at commit time at ~47 min wall, slower per-iter rate than 100-iter sample suggested); doc will be amended with full numbers when summary lands. Engine code NOT modified. Empirical measurement only. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
amaster97
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Empirical rerun of the preflop RvR solve at higher iter counts to disambiguate
"convergence issue" vs "engine bug" for the degenerate preflop strategies.
1000-iter result (10x original baseline, post-PR #157 .so)
Trajectory
Strategy monotonically narrows to fold as iters grow — opposite of what a
"needs more iters" convergence issue would show.
10000-iter run
Launched but killed externally at ~50 min wall / ~47 min CPU before writing
its summary (suspected macOS memory pressure, peak RSS ~4 GB). Verdict does
not depend on it.
Verdict
BUG CONFIRMED — NOT a convergence issue.
Bumping default iter count will NOT fix this. The leaf math must be fixed per
PR #159's RCA:
PreflopRvrState::initialignoresconfig.initial_contributions,so when the repro config passes
[50, 100]matching the blinds, the fold-at-rootleaf's chip-flow zeros out and folding becomes the no-loss dominant action.
AA at SB root plays normally (the bug doesn't affect non-fold leaves deeper in
the tree where contributions have been bumped above the blind level).
Test plan
Constraints honored