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

New panorama pipeline for fisheye images #749

Merged
merged 270 commits into from
Jul 16, 2020
Merged

Conversation

fabiencastan
Copy link
Member

@fabiencastan fabiencastan commented Feb 26, 2020

Description

Add support for HDRI creation from fisheye images.

Features list

  • New Intrinsic: EquidistantRadial
  • New dedicated BA with symbolic derivatives
  • New fisheye circle estimation
  • Split LdrToHdr into LdrToHdrSampling, LdrToHdrCalibration, LdrToHdrMerge
  • Improve LdrToHdr calibration quality

Implementation remarks

  • Change "Intrinsics" inheritance tree and use distortion objects

- new param maxTotalPoints
- samplesFolder is required only if the method is not linear
- PanoramaWarping: export extra info (panorama size and warp tile
offset) in metadata (instead of an additional json file)
- PanoramaCompositing: now requires an input SfM to be able to load
warped images
The UID generation has been updated, so if there is no unique ID in the
image metadata, the full path of the image will be used.
This breaks the ways we generate identical UID for GT evaluation.
So with this PR, the GT comparaison uses the number extracted from the
filename to create the UID.
- Sampling/Calibration/Merge supports the case of NO multi-bracketing
- Rename the output param of LdrToHdrMerge
- Add an index offset to control the bracket index to use as the
reference for the target exposure
- The notion of target views is now only used in LdrToHdrMerge (and no
more in sampling/calibration)
@fabiencastan fabiencastan merged commit 33d6553 into develop Jul 16, 2020
@fabiencastan fabiencastan deleted the dev/panoramaFisheye branch July 16, 2020 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants