Skip to content

Increase QR scanner processing resolution from 400px to 1024px#863

Open
jbojcic1 wants to merge 1 commit intomasterfrom
improve-qr-scanner-resolution
Open

Increase QR scanner processing resolution from 400px to 1024px#863
jbojcic1 wants to merge 1 commit intomasterfrom
improve-qr-scanner-resolution

Conversation

@jbojcic1
Copy link
Collaborator

The qr-scanner library downscales video frames to 400x400px before decoding, which is insufficient for dense QR codes. This adds a calculateScanRegion config that keeps the same scan region but increases the decode resolution to 1024x1024px, giving ~2.5x more pixels per QR module.

Closes #676

The qr-scanner library downscales video frames to 400x400px before
decoding, which is insufficient for dense QR codes. This adds a
calculateScanRegion config that keeps the same scan region but
increases the decode resolution to 1024x1024px, giving ~2.5x more
pixels per QR module.

Closes #676

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jbojcic1 jbojcic1 requested a review from gudnuf February 16, 2026 17:08
@jbojcic1 jbojcic1 self-assigned this Feb 16, 2026
@vercel
Copy link

vercel bot commented Feb 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agicash Ready Ready Preview, Comment Feb 16, 2026 5:08pm

Request Review

@supabase
Copy link

supabase bot commented Feb 16, 2026

This pull request has been ignored for the connected project hrebgkfhjpkbxpztqqke because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

Copy link
Contributor

@gudnuf gudnuf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving because it doesn't seem like it can hurt, but I'm not sure this makes it any better. I was just testing on my mac and it seems to be comparable to without this change. @jbojcic1 did you notice an improvement?

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.

In-app QR scanner does not match the native scanner's performance

2 participants

Comments