Skip to content

FRLG summary reader fixes - #1365

Merged
Mysticial merged 5 commits into
PokemonAutomation:mainfrom
theastrogoth:frlg-statsreader-fixes
Aug 8, 2026
Merged

FRLG summary reader fixes#1365
Mysticial merged 5 commits into
PokemonAutomation:mainfrom
theastrogoth:frlg-statsreader-fixes

Conversation

@theastrogoth

@theastrogoth theastrogoth commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

This includes a few fixes to the FRLG summary reader:

  • switches to only saving debug dumps on failed reads rather than on every read
  • saves the full raw frame in addition to the crops and blurred+thresholded images
  • removes the paths that used to be specific to Paddle, which would need multi-filtering added to be reliable. I'd never used/tested/maintained them, and I think the waterfill + template matching approach is consistent enough to rely on in all cases

The last point doesn't address Patrick's issues with PaddleOCR, since it didn't change the nature reader

@Mysticial

Copy link
Copy Markdown
Collaborator

I see that you're yeeting some of the Tesseract/Paddle branching. Make sure both of them still work.

@theastrogoth

Copy link
Copy Markdown
Contributor Author

Confirmed working with both libraries (neither is used for digit reading now, since it's just template matching).

The Paddle branches weren't working for certain capture cards anyways, since they didn't have a multi-filter step

@Mysticial

Copy link
Copy Markdown
Collaborator

Will merge when CI finishes green.

@Mysticial
Mysticial merged commit a53a468 into PokemonAutomation:main Aug 8, 2026
6 of 7 checks passed
@theastrogoth
theastrogoth deleted the frlg-statsreader-fixes branch August 13, 2026 13:46
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.

2 participants