Re: Review: DisplayTransform interface update


Jeremy Selan <jeremy...@...>
 

Here's an update that addresses all previous comments.
http://github.com/jeremyselan/OpenColorIO/commit/2b926355196bcd1690f9dd5a924c3d04ff24cd53

This commit updates the APIs to use our newer terminology + functionality:
- Display - a virtual or physical display device
- View - a meaningful view of the reference space on a Display

It also changes the .ocio config format:

displays:
DCIP3:
- !<View> {name: Raw, colorspace: nc10}
- !<View> {name: Log, colorspace: lg10}
- !<View> {name: Film, colorspace: p3dci8}
sRGB:
- !<View> {name: Raw, colorspace: nc10}
- !<View> {name: Log, colorspace: lg10}
- !<View> {name: Film, colorspace: srgb8}
Cheese:
- !<View> {name: Raw, colorspace: nc10}
- !<View> {name: Log, colorspace: lg10}
active_displays: [sRGB, DCIP3, Cheese]
active_views: [Film, Log, Raw]

(The old format is still allowed for backwards compatibility purposes.)

New environment variables are exposed:
$OCIO_ACTIVE_DISPLAYS
$OCIO_ACTIVE_VIEWS
which allow these lists to be pruned / prioritized at runtime.

Upon this commit being approved, I will put out updated configurations
that match this spec.

-- Jeremy

Join ocio-dev@lists.aswf.io to automatically receive all group messages.