Blender Git Commit Log

Git Commits -> Revision 7c8cbc6

Revision 7c8cbc6 by Brecht Van Lommel (tmp-ocio-v2)
February 1, 2021, 14:42 (GMT)
OpenColorIO: use aces_interchange role to detect CIE XYZ values

We need standard CIE XYZ values for rendering effects like blackbody emission.
The relation to the scene linear role is based on OpenColorIO configuration.

In OpenColorIO 2.0 configs roles can no longer have the same name as color
spaces, which means our XYZ role and colorspace in the configuration give an
error.

Instead use the new standard aces_interchange role, which relates scene linear
to a known scene referred color space. Compatibility with the old XYZ role is
preserved, if the configuration file has no conflicting names.

Also includes a non-functional change to the configuraton file to use an
XYZ-to-ACES matrix instead of REC709-to-ACES, makes debugging a little easier
since the matrix is the same one we have in the code now and that is also
found easily in the ACES specs.

There is a new cie_xyz_d65_interchange role that we should add support for,
for interchange of display referred colorspace (as opposed to scene referred).
I couldn't immediately understand how that is supposed to be defined, will
probably leave that for another patch unless someone knows how.

Commit Details:

Full Hash: 7c8cbc67abdd0e2293ca810a3832749bb6031649
Parent Commit: 4e35285
Lines Changed: +153, -64

1 Added Path:

/release/datafiles/colormanagement/luts/xyz_to_aces.spimtx (+3, -0) (View)

2 Deleted Paths:

/release/datafiles/colormanagement/luts/aces_to_xyz.spimtx (+0, -3)
/release/datafiles/colormanagement/luts/rec709_to_aces.spimtx (+0, -3)

4 Modified Paths:

/intern/cycles/render/shader.cpp (+87, -37) (Diff)
/intern/cycles/render/shader.h (+2, -0) (Diff)
/intern/opencolorio/ocio_impl.cc (+55, -18) (Diff)
/release/datafiles/colormanagement/config.ocio (+6, -3) (Diff)
By: Miika HämäläinenLast update: Nov-07-2014 14:18MiikaHweb | 2003-2021