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

ingenic-osdrv-t31: Improve image quality by replacing gc2053-t31.bin sensor param file from stock fw #966

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

gtxaspec
Copy link
Contributor

Update gc2053-t31.bin with sensor param file from stock FW. The result is Improved image quality: better contrast, saturation and luminance.

…ock FW

Update `gc2053-t31.bin` with sensor param file from stock FW.  The result is Improved image quality: better contrast, saturation and luminance.
Copy link
Collaborator

@ZigFisher ZigFisher left a comment

Choose a reason for hiding this comment

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

Thank you.
This patch will require additional testing as devices with this sensor are already in use and we need to make sure that nothing will be broken.

@gtxaspec
Copy link
Contributor Author

gtxaspec commented Aug 22, 2023

@ZigFisher Thank You!

If anyone else would be able to test on their devices, the feedback would be appreciated. I tested on:

Wyze V3
AtomCam 2

@viktorxda
Copy link
Collaborator

Just from very quick test on a Wyze V3, the image quality is indeed improved.
I assume this is only a image parameter calibration file, something similar is used for Sigmastar as well.

@ZigFisher ZigFisher merged commit b6ceb9c into OpenIPC:master Aug 22, 2023
81 checks passed
@ZigFisher
Copy link
Collaborator

Test passed, thank you all

themactep referenced this pull request in themactep/thingino-firmware Oct 29, 2023
@gtxaspec gtxaspec deleted the gc2053-sensor-cal branch November 6, 2023 07:05
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.

3 participants