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

[Texturing] Add option to correct exposure values during Texturing #656

Merged
merged 7 commits into from
Jul 15, 2019

Conversation

cvere
Copy link
Contributor

@cvere cvere commented Jul 4, 2019

Description

Add an option to correct exposure values (EV) of input images during texturing, to uniformize dataset exposures.
Do exactly the same thing than #652, but during Texturing instead of PrepareDenseScene

Features list

  • Load exposure compensation factor from "AliceVision" metadata (computed during PrepareDenseScene)
  • Correct each image exposure
  • Function to convert image colorspace, based on OpenImageIO (sRGB <-> Linear conversions)

Implementation remarks

Exposure correction must be applied in Linear colorspace while texturing must be done in sRGB colorspace. Thus, when loading images for texturing with exposure compensation, we read images in Linear colorspace, correct their exposure and then we convert them in sRGB.

cvere added 3 commits July 3, 2019 18:32
- some refactoring + remove "bandType" parameter
- add notion of exposure correction in ImagesCache constructor & necessary functions
Exposure correction must be done in linear colorspace -> add function to convert image colorspace with OIIO
@cvere cvere force-pushed the dev_EVcorrect_Texturing branch 2 times, most recently from 38c528b to 55221de Compare July 4, 2019 13:14
cvere added 4 commits July 4, 2019 15:15
check if the exposure compensation factor has been calculated & stocked in metadatas during PrepareDenseScene
Move imageIO::colorconvert to imageAlgo + some refactoring
@fabiencastan fabiencastan added this to the 2019.2 milestone Jul 5, 2019
@fabiencastan fabiencastan changed the title Dev ev correct in texturing [Texturing] Add option to correct exposure values during Texturing Jul 5, 2019
@fabiencastan fabiencastan merged commit 86af256 into develop Jul 15, 2019
@fabiencastan fabiencastan deleted the dev_EVcorrect_Texturing branch July 15, 2019 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants