hugin-devs team mailing list archive

[tmodes <2064625@xxxxxxxxxxxxxxxxxx>]

** Changed in: hugin
    Milestone: None => 2024.0beta1

** Changed in: hugin
       Status: Fix Committed => Fix Released

-- 
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:
  Fix Released

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