OpenColorIO-Config-ACES icon indicating copy to clipboard operation
OpenColorIO-Config-ACES copied to clipboard

[ENHANCEMENT]: Include HDR Output Transforms in the views list for Display P3

Open nick-shaw opened this issue 9 months ago • 4 comments

Apple kCGColorSpaceExtendedDisplayP3 supports float values above 1.0 for HDR/EDR display on capable Mac systems. Currently the views for the newly added Display P3 display in the Studio and CG configs only include the SDR D65 and D60 sim Output Transforms. The addition of HDR Output transforms to that list would greatly simplify working in HDR for users with suitable Apple devices.

I have previously made my own config which does exactly this, and in my (admittedly limited) testing there appear to be no issues with mixing SDR and HDR views for the same display.

nick-shaw avatar Sep 19 '23 11:09 nick-shaw

Hey @nick-shaw,

Thanks! The starting point is to have the corresponding Output Transform on the aces-dev side like I did for the SDR variant. Once this is done, we can start looking at adding it to the config.

Cheers,

KelSolaar avatar Sep 20 '23 09:09 KelSolaar

Thanks @KelSolaar

So is it the case that the config must have a 1:1 relationship with Output Transforms in aces-dev? I was imagining that given the separation of views and displays in OCIO, it would only be necessary to add to the Display P3 views list, to indicate that (on suitable devices) it was capable of showing HDR as well as SDR view transforms. Probably only ACES 1.1 - HDR Video (1000 nits & P3 lim).

nick-shaw avatar Sep 20 '23 09:09 nick-shaw

Yes, we derive the Reference transforms to build from aces-dev and use a spreadsheet that links a given ACEStransformID to the relevant OCIO BuiltinTransform and Display Colorspace: https://docs.google.com/spreadsheets/d/1SXPt-USy3HlV2G2qAvh9zit6ZCINDOlfKT07yXJdWLg/edit#gid=0

KelSolaar avatar Sep 20 '23 10:09 KelSolaar

This commit adds Display P3 1000 nit forward and inverse Output Transforms. But it is not clear to me what the file names and transform IDs should be.

Thoughts?

Tagging @scottdyer and @aforsythe.

nick-shaw avatar Sep 20 '23 12:09 nick-shaw