Repository navigation
Conversation
… scenes' triggers; the Oreburgh Gym's inside Checkpoints 00.01 to 00.03 replay recorded stick holds. This writes the same walks down as lines on the Twinleaf, Route 201 (a decomp stand-in until the re-derived grid lands), Verity Lakefront and Sandgem grids, corner to corner, each line ending on the tile that fires the next scene. The scenes' trigger rectangles are read off the Diamond decomp's coord events (Barry's door, his tall-grass warning, the lake's entrance, Rowan on the way back, Dawn at Sandgem), and a test plans every stretch as one straight leg and checks each line steps into its scene on its last tile only. Route records for the three checkpoints trace on the grids. The Oreburgh Gym's inside is decoded so 01.04's "north until a trainer stops us" traces too. The segment code keeps its replays; a TODO points at the lines. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
This was referenced Nov 1, 2026
Closed
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
#111's decomp grids win: Segment00Lines drops its Route 201 stand-in for Route201::grid() (7 rows shorter at the top) and SandgemTown::grid() is now its whole cell (4 rows taller). The 00.01-00.03 lines, triggers, named tiles and route records move by those offsets; their world tiles are unchanged. Done first on #121 (64afa5f); this PR owns the lines. This PR now builds on #111. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine
changed the base branch from
bdsp/event-sandgem-center
to
bdsp/maps-rederive
November 1, 2026 17:25
dayfine
added a commit
that referenced
this pull request
Nov 1, 2026
…08f8c) into bdsp/mvp-v3 #111 now holds the grid fixes and #112 the 00.01-00.03 lines on them, both first made here; their content matches this branch's, so only nav/shared's newer LegPlanner (an opt-in wild-stop weight, 0 by default) and #111's map grid tests arrive. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine
added a commit
that referenced
this pull request
Nov 1, 2026
…n its own file Test_Route202CatchGrass, Test_SandgemCenterApproach and Test_TrainingToSandgem moved to #111 (PokemonBDSP_AutoStory_MapGrid_Tests, merged in), so Segment00_Tests.cpp is #112's again; Test_IndoorsAnyLight, v3's own, moves to PokemonBDSP_AutoStory_Indoors_Tests.cpp. No program code changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
…nned-walks Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
The Gym's grid, which this PR added, moves to OreburghGym.json like #111's maps (converted once, cell for cell); its C++ table goes and OreburghGym::grid() returns story_grid("OreburghGym"). bdsp_maps.py embeds it, and the test data is regenerated with it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine
added a commit
that referenced
this pull request
Nov 1, 2026
…sp/mvp-v3 #111 and #112 now hold the maps as JSON, the built-in test copy, the generator and the JSON-to-grid loader (MapGrids), which were this branch's; the C++ tables are gone. What stays here: the program loads Packages' files at start, checks each against the built-in copy, and installs them for story_grid() (MapFiles); StoryMaps goes back to the X::grid() accessors. The glyph test moved to #111 with the loader. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
…) into bdsp/segment00-planned-walks The generated test data goes with #111's swap; the Oreburgh Gym, this PR's map, is committed as Maps/OreburghGym.json beside #111's 15. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine
added a commit
that referenced
this pull request
Nov 1, 2026
…uilt in by CMake) into bdsp/mvp-v3 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine
added a commit
that referenced
this pull request
Nov 1, 2026
…s are the only copy The user's decision (2026-11-01): the maps stay files in this repository (#111, #112), built into the program; story_grid() reads only those. The start-up load of Packages' PokemonBDSP/Maps and its "differs from this build's copy" check go, with their test; the lab deploy no longer copies map files. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
…dsp/segment00-planned-walks Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine
added a commit
that referenced
this pull request
Nov 1, 2026
…approach) into bdsp/mvp-v3 #111 now has the two walks into Sandgem; this branch's version of the heal (the same two walks, then the taps west below the door) is kept. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
Owner
Author
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
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.
00.01 to 00.03 still replay recorded stick holds; this writes the same walks down as lines on the decomp grids and checks them offline, without touching the segment code (task #22).
Segment00Lines: five lines from the house's doorstep to Dawn at Sandgem's entrance, corner to corner, on Twinleaf, Route 201 (a decomp stand-in until the re-derived grid lands), the Verity Lakefront and Sandgem; the scenes' trigger rectangles read off the Diamond decomp's coord events.Testing: new
Test_Segment00Lines(every stretch plans as one straight leg; each line enters its scene's rectangle on its last tile only) andTest_OreburghGymGrid;Test_RouteRecordsnow traces 14 records. Checked with the standalone planner harness and the code-health scan; no console run.🤖 Generated with Claude Code
https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E