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

[image] Fix erroneous color space detection for images that contain "raw" in their filepath #1728

Merged
merged 1 commit into from
Jul 22, 2024

Conversation

demoulinv
Copy link
Contributor

Description

This PR fixes the following bug:
For non raw images (for instance jpeg format), without any color space indication in the metadata, if the image filepath contains "raw" then the Raw color space is detected by the ocio function using the filepath as a hint to guess the color space.
Because no conversion is defined in the ocio.config file from the raw color space the image reading fails.

Features list

  • Use only the filename instead of the full filepath to guess the image color space in order to limit misinterpretation for all color spaces.
  • Do not consider the Raw color space when it is returned by the ocio function.

Implementation remarks

@cbentejac cbentejac changed the title Bugfix on color space detection [image] Fix erroneous color space detection for images that have "raw" in their filepaths Jul 22, 2024
@cbentejac cbentejac changed the title [image] Fix erroneous color space detection for images that have "raw" in their filepaths [image] Fix erroneous color space detection for images that have "raw" in their filepath Jul 22, 2024
@cbentejac cbentejac merged commit 0f66f01 into develop Jul 22, 2024
3 of 4 checks passed
@cbentejac cbentejac deleted the bugfix/colorspaceDetection branch July 22, 2024 14:28
@cbentejac cbentejac changed the title [image] Fix erroneous color space detection for images that have "raw" in their filepath [image] Fix erroneous color space detection for images that contain "raw" in their filepath Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants