Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OV5640 vertical flip for T-Display S3 PRO #183

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

odudex
Copy link

@odudex odudex commented Nov 16, 2024

Just tried Jade in Liligo T-Display S3 PRO with OV5640 camera, and beyond activating the camera on sdkconfig settings, I needed to flip camera vertical axis.

@sandman21vs
Copy link

o homem é uma maquina

@JamieDriver
Copy link
Collaborator

Seems reasonable, thanks.

@totallynarwhal
Copy link

I was just hitting into this issue when trying the S3 Pro with the OV5640 camera. Thanks!

I notice that the OV5640 camera has a hard time reading the QR code from the oracle server. It didn't on my phone screen and was barely able to on my PC monitor (zooming on the page did help).

Is this due to the higher (e.g. 5MP) resolution of the OV5640? Would this be better on the GC0308 camera which is VGA?

@odudex
Copy link
Author

odudex commented Nov 30, 2024

I just tried it in stateless mode, loaded a key, and signed a PSBT via QR code. It was quick and smooth.
Although the OV5640 can offer higher resolution, it's not being used (it would be more laggy, and the UX would be worse).
Maybe other settings could be tweaked, such as exposure and sensitivity. This way, performance could match Jade's OV2640.

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.

4 participants