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

Update cameraSensors.db DJI Mini3Pro #1753

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

AncoraIo36
Copy link

DJI; Mini3Pro;9,79;usercontribution

Description

I add the value of width of the sensor camera lens of the drone DJI Mini3 Pro

Features list

I have improved the cameraSensors.db

DJI; Mini3Pro;9,79;usercontribution
@natowi natowi added the sensordb Sensor database update label Oct 17, 2024
@natowi
Copy link
Member

natowi commented Oct 20, 2024

Thank you for the contribution. Where did you get the sensor width from?

@natowi natowi changed the title Update cameraSensors.db Update cameraSensors.db DJI Mini3Pro Oct 20, 2024
@natowi natowi added the review label Oct 24, 2024
@AncoraIo36
Copy link
Author

Hi natowi,
My starting point was finding an explicit warning (yellow round icon) on the photos taken with my drone (DJI Mini 3 Pro) about the approximate intrinsic characteristics of the camera and the possibility of encountering degraded results.

I have updated the database of the photo sensors according to the instructions in the guide (https://github.com/alicevision/meshroom/wiki/Add-Camera-to-database). Neither on digicamdb.com nor on devicesspecifications.com I was able to find information about the drone's sensor.

I visited and searched on many websites without luck, I found on the Samsung website (https://semiconductor.samsung.com/image-sensor/mobile-image-sensor/isocell-gnk/) data related to the same type of sensor that the drone has and that with the mathematical formula suggested in the github mentioned above (xresolution * μmvalue/1000=mmwidth) allowed me to establish that data with certainty: 8.160 * 1.2/1000 = 9.792

@natowi natowi added ready and removed review labels Oct 27, 2024
@natowi
Copy link
Member

natowi commented Oct 27, 2024

Thank you AncoraIo36 :)

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.

2 participants