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

Add support for ASWF Color Interop Forum recommendation #127

Open
doug-walker opened this issue Jul 20, 2024 · 4 comments
Open

Add support for ASWF Color Interop Forum recommendation #127

doug-walker opened this issue Jul 20, 2024 · 4 comments
Assignees
Labels
configs for separation purposes

Comments

@doug-walker
Copy link
Contributor

doug-walker commented Jul 20, 2024

The ASWF Color Interop Forum has developed a recommendation for color spaces that should be supported for texture assets and CG rendering. The draft of the document is here.

The proposed names should be added to the OCIO ACES configs, either as the color space name or as an alias. The suffix "Texture" should be replaced with "Scene-referred".

We may want to revisit the naming of "sRGB Encoded P3-D65 - Texture" and make that an alias.

The color spaces to add are AdobeRGB, Linear AdobeRGB, and CIE-XYZ-D65 Scene-referred.

At the same time, support should be added for any MaterialX 1.39 names which are not already part of the configs. This means adding "adobergb" and "lin_adobergb". We should discuss whether to make "none" an alias of Raw. The "g18_ap1" is deprecated and does not need to be added. The rest of them are already in the configs as aliases.

This applies to both the CG and Studio configs for ACES.

@doug-walker doug-walker added the configs for separation purposes label Jul 20, 2024
@doug-walker doug-walker added this to the OCIO 2.4.0 milestone Jul 20, 2024
@KelSolaar
Copy link
Contributor

Should be addressed in #129!

@doug-walker
Copy link
Contributor Author

Confirmed in studio-config-v2.2.0_aces-v1.3_ocio-v2.4.ocio and cg-config-v2.2.0_aces-v1.3_ocio-v2.4.ocio now include these names and aliases as of PR #129.

The ASWF CIF recommendation is not final yet, so will leave this issue open for now in case any tweaks are made to the naming.

@doug-walker
Copy link
Contributor Author

@KelSolaar , per the discussion at the latest Interop Forum meeting, please change the following user-facing names as follows:

sRGB - Scene-referred --> sRGB Encoded Rec.709 (sRGB)
Gamma 2.2 Rec.709 - Scene-referred --> Gamma 2.2 Encoded Rec.709
Gamma 1.8 Rec.709 - Scene-referred --> Gamma 1.8 Encoded Rec.709
sRGB Encoded AP1 - Scene-referred --> sRGB Encoded AP1
Gamma 2.2 AP1 - Scene-referred --> Gamma 2.2 Encoded AP1
sRGB Encoded P3-D65 - Scene-referred --> sRGB Encoded P3-D65
AdobeRGB - Scene-referred --> Gamma 2.2 Encoded AdobeRGB

@KelSolaar
Copy link
Contributor

List was missing Gamma 2.4 Encoded Rec.709 and this should leave us with CIE XYZ-D65 - Scene-referred only having the Scene-referred suffix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
configs for separation purposes
Projects
None yet
Development

No branches or pull requests

2 participants