search

alicevision / AliceVision

Photogrammetric Computer Vision Framework
http://alicevision.org
Other
2.89k stars 807 forks source link

Update Fairphone cameraSensor #1654

Closed woutkolkman closed 4 months ago

woutkolkman commented 5 months ago

Fairphone 5 primary sensor (not ultra-wide). If I made a mistake, please let me know, thanks.

natowi commented 4 months ago

Main Camera: 50MP Sony IMX 800 sensor. 1/1.49" 1.0μm-2µm(pixel binning), 8768 x 6144 pixels -> 8.76

@woutkolkman Where did you get your value from?

woutkolkman commented 4 months ago

@natowi I'm not actually too sure about the value, because I don't have that much experience in sensor specs. I should have stated that.

There might also be conflicting sources online, I think I've made an error here.

The value you've posted does have a logical calculation, so I'd go with that.

woutkolkman commented 4 months ago

@fabiencastan I'm afraid the value was still 8.14, not 8.76, I'm not sure if this is a problem.

fabiencastan commented 4 months ago

Oops, I was thinking that the update was done. Could you submit a new PR with the fix?