hugin-devs team mailing list archive

[tmodes <2064625@xxxxxxxxxxxxxxxxxx>]

Okay, thanks for the test file. I found the report with the digital zoom ratio: https://bugs.launchpad.net/hugin/+bug/2000019
Here the digital zoom ratio is required for a correct fov.

The problem is that Apple does not follow the EXIF standard. 
The EXIF standard says under FocalLengthIn35mmFilm:
- Digital zoom ratio is not taken into account.
But Apple takes the digital zoom ratio into account and write also the digital zoom ratio.

So not sure how to work around this violation of the standard. Best
would be if Apple would follow the standard, but I have no hope that
this will happen.

-- 
You received this bug notification because you are a member of Hugin
Developers, which is subscribed to Hugin.
https://bugs.launchpad.net/bugs/2064625

Title:
  35mm focal length on iPhone SE3 2nd generation over corrercted

Status in Hugin:
  Incomplete

Bug description:
  This is for an iPhone SE 3rd generation. Looking at the exif data for
  a zoomed picture I can see:

  Lens Specification  |3.990000, 3.990000, 1.8, 1.8
  Lens Make           |Apple
  Lens Model          |iPhone SE (3rd generation) back camera 3.99mm f/1.8
  Digital Zoom Ratio  |1.951
  Focal Length in 35mm|55

  For a normal picture:

  Focal Length in 35mm|28
  Scene Capture Type  |Standard
  Lens Specification  |3.990000, 3.990000, 1.8, 1.8
  Lens Make           |Apple
  Lens Model          |iPhone SE (3rd generation) back camera 3.99mm f/1.8

  Turns out the reported focal length for a zoomed picture is already
  corrected for the digital zoom factor so
  src/hugin_base/panodata/SrcPanoImage.cpp:542 over corrects by
  multiplying again with the digital zoom factor ending up with an hFOV
  too small. I just commented out line 542 and now when I add the zoomed
  picture I get an hFOV of 34.9 (which is what I would expect) instead
  of 18.3 (wrong).

  --- a/src/hugin_base/panodata/SrcPanoImage.cpp  Wed May 01 19:41:11 2024 +0200
  +++ b/src/hugin_base/panodata/SrcPanoImage.cpp  Thu May 02 15:18:02 2024 +0200
  @@ -539,7 +539,7 @@
       const float digitalZoom = Exiv2Helper::getExiv2ValueDouble(exifData, "Exif.Photo.DigitalZoomRatio");
       if (eFocalLength35 > 0 && digitalZoom > 1)
       {
  -        eFocalLength35 *= digitalZoom;
  +    //    eFocalLength35 *= digitalZoom;
       };
       float focalLength=0;
       //The various methods to detmine crop factor

To manage notifications about this bug go to:
https://bugs.launchpad.net/hugin/+bug/2064625/+subscriptions