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

DCP error management #1328

Merged
merged 2 commits into from
Jan 14, 2023
Merged

DCP error management #1328

merged 2 commits into from
Jan 14, 2023

Conversation

demoulinv
Copy link
Contributor

Description

Throw an error instead of a warning when trying to load a non existing DCP profile.

cameraInit: rawColorInterpretation automatically switch to librawWhiteBalancing mode and throw a warning when a DCP profile is required but no error is requested if it is missing and no database or an erroneous one is given as parameter or if the expected profile does not exist in the database.

Add working color space as a parameter in sampling and merging HDR exec.

Features list

Implementation remarks

Throw an error instead of a warning when trying to load a non existing DCP profile.
rawColorInterpretation: In cameraInit, automatically switch to librawWhiteBalancing mode when a DCP profile is required but no error is requested if it is missing and no database or an erroneous one is given as parameter or if the expected profile does not exist in the database.
Add working color space as a parameter in sampling and merging HDR exec.
@demoulinv demoulinv requested a review from fabiencastan January 6, 2023 17:27
Copy link
Member

@fabiencastan fabiencastan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also merge and sampling modifications should be done in a separate commit.

src/software/pipeline/main_cameraInit.cpp Outdated Show resolved Hide resolved
src/software/pipeline/main_cameraInit.cpp Outdated Show resolved Hide resolved
src/software/pipeline/main_cameraInit.cpp Show resolved Hide resolved
src/software/pipeline/main_cameraInit.cpp Outdated Show resolved Hide resolved
@fabiencastan fabiencastan added this to the 2.5.0 milestone Jan 14, 2023
@fabiencastan fabiencastan changed the title Missing DCP error management DCP error management Jan 14, 2023
@fabiencastan fabiencastan merged commit c3bba3a into develop Jan 14, 2023
@fabiencastan fabiencastan deleted the dev/dcpManagementErrorUpdate branch January 14, 2023 17:42
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