Skip to content

FRLG: RNG plan log lines and RngPlan tests - #188

Draft
dayfine wants to merge 5 commits into
frlg/rng-planfrom
frlg/rng-plan-log
Draft

dayfine wants to merge 5 commits into
frlg/rng-planfrom
frlg/rng-plan-log

Conversation

@dayfine

@dayfine dayfine commented Nov 3, 2026

Copy link
Copy Markdown
Owner

Depends on #187 (base frlg/rng-plan) and #186 (frlg/rng-plan-inputs merged in); the diff also shows #186's files until it merges into the base.

Log lines for the RNG plan, and the tests for RngPlan that check them.

  • PokemonFRLG_RngPlanLog: one line per attempt (Target Seed, Advances, nature, IVs, the SIDs it's shiny for, both waits), plus the TID, SID Helper, result and confirmed-SID lines a program logs.
  • RngPlan's tests (candidate weights, attempt order, Starter RNG's timing limits) land here because they also check the attempt lines.
  • Split out of FRLG AutoStory: plan Starter RNG attempts from SID Helper's candidates #182 unchanged.

Testing

  • New RngPlan and RngPlanLog unit tests; syntax-checked against this branch (with Windows Clang's missing-field warning on); code health 0 failures.
  • Not run on hardware.

🤖 Generated with Claude Code

https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E

dayfine and others added 3 commits November 2, 2026 19:18
Split out of #182 unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
# Conflicts:
#	SerialPrograms/Source/VideoGames/PokemonFRLG/PokemonFRLG_Tests.cpp
#	SerialPrograms/cmake/SourceFiles.cmake
Split out of #182 unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
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