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

[cameraSensors] fix Samsung SM-G998B #1449

Merged
merged 2 commits into from
Mar 30, 2024

Conversation

ICIbrahim
Copy link
Contributor

Apologies for my previous mistake, had confounded focal length with sensor width for the Samsung S21 Ultra 108MP lens. This should now be correct based on Samsung Device Specifications: https://www.samsung.com/uk/support/mobile-devices/what-are-the-latest-features-of-the-galaxy-s21-series-camera/

Description

Features list

Implementation remarks

Apologies for my previous mistake, had confounded focal length with sensor width for the Samsung S21 Ultra 108MP lens. This should now be correct based on Samsung Device Specifications: https://www.samsung.com/uk/support/mobile-devices/what-are-the-latest-features-of-the-galaxy-s21-series-camera/
@almarouk almarouk added the sensordb Sensor database update label Jun 5, 2023
@natowi natowi added the ready label Aug 24, 2023
natowi
natowi previously approved these changes Aug 24, 2023
@natowi
Copy link
Member

natowi commented Aug 24, 2023

Image resolution: 12000x9000; 0.8㎛ pixel size -> 9.6mm

@fabiencastan
Copy link
Member

@ICIbrahim Could you validate the new values updated by @natowi are working in your use case?

@fabiencastan fabiencastan changed the title Update cameraSensors.db [cameraSensors] fix Samsung SM-G998B Mar 30, 2024
@fabiencastan fabiencastan added this to the 3.3.0 milestone Mar 30, 2024
@fabiencastan fabiencastan merged commit b58c922 into alicevision:develop Mar 30, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready sensordb Sensor database update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants