Skip to content

Align video catalogue limits and OCR coverage with native evidence - #15

Merged
DrDrewCain merged 1 commit into
mainfrom
fix/video-catalogue-limits
Sep 13, 2026
Merged

DrDrewCain merged 1 commit into
mainfrom
fix/video-catalogue-limits

Conversation

@DrDrewCain

@DrDrewCain DrDrewCain commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Valid native video evidence could fail to display when a frame had over 10,000 OCR regions or supplementary Unicode characters in labels, metadata or filenames. The parser also accepted nonwhitespace recognized text that no OCR region accounted for.

Match the native 20,000 cumulative region allowance and Unicode character limits, and require region spans to account for all nonwhitespace text. Source identities, exact clocks and aggregate text/region limits remain checked.

Builds on Web PR14. Validation: 452 tests pass; strict TypeScript and production build pass. Independent native-comparison probes accept 20,000 cumulative regions and a 50,001-emoji OCR label, reject 20,001 regions, and refuse uncovered text. Five persistent regressions reproduced the failures before correction. Existing bundle-size advisory remains.

@DrDrewCain
DrDrewCain changed the base branch from feature/video-frame-understanding to main September 13, 2026 02:36
@DrDrewCain
DrDrewCain merged commit 1510924 into main Sep 13, 2026
2 checks passed
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