Repository navigation
Conversation
This was referenced Oct 22, 2026
Closed
Open
dayfine
force-pushed
the
bbq/failure-screenshots
branch
from
October 27, 2026 06:12
f07244e to
e1d2a2d
Compare
dayfine
force-pushed
the
bbq/diagnostics-and-baseline
branch
from
October 27, 2026 06:12
0ceac54 to
c8ba680
Compare
dayfine
added a commit
that referenced
this pull request
Oct 27, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine
force-pushed
the
bbq/diagnostics-and-baseline
branch
from
October 28, 2026 03:35
c8ba680 to
7d24d5f
Compare
dayfine
force-pushed
the
bbq/failure-screenshots
branch
from
October 28, 2026 03:35
e1d2a2d to
72099ba
Compare
When a catch quest times out entering battle, or a quest is still on the panel after an attempt, the logs say what failed but not what the screen showed. Add save_bbq_snapshot(), which saves the current frame to DEBUG_PATH()/BBQ/ as <timestamp>-<quest>-<reason>.png and logs a "BBQ event=snapshot" line with the outcome. Unlike dump_image(), it stays on the local machine and sends nothing to telemetry. Call it at the battle-entry timeout and at failed quest verification. Add PokemonSV::BBQSnapshot tests that save into a folder that does not exist yet and check the file, path and log line, and that check a save that cannot be written returns no path and logs saved=0. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E
dayfine
force-pushed
the
bbq/diagnostics-and-baseline
branch
from
October 28, 2026 08:08
7d24d5f to
7cf661b
Compare
dayfine
force-pushed
the
bbq/failure-screenshots
branch
from
October 28, 2026 08:08
72099ba to
cc69593
Compare
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
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.
Part of PokemonAutomation/ComputerControl#263.
When a catch quest times out entering battle, or a quest is still on the board after an attempt, the logs say what failed but not what the screen showed.
save_bbq_snapshot()saves the current frame at those two points toDEBUG_PATH()/BBQ/as<timestamp>-<quest>-<reason>.pngand logs where it went; unlikedump_image(), nothing is sent to telemetry. TwoPokemonSV::BBQSnapshottests inPokemonSV_BBQTests.cppcover a save into a new folder and a save that cannot be written; the native BBQ runner passes 16 of 16.🤖 Generated with Claude Code
https://claude.ai/code/session_01AKYzhrMnPK61XHWEch4a6E