http://www.miikahweb.com/feed.php MiikaHweb - Blender Tmp-ocio-v2 Branch http://www.miikahweb.com/ MiikaHweb - Blender Tmp-ocio-v2 Branch fi Wed, 27 Oct 21 17:05:56 +0000 MiikaHweb RSS contact at miikahweb.com <![CDATA[[28f799d]: OpenColorIO: address review comments]]> http://www.miikahweb.com/en/blender/git-logs/commit/28f799df9260f3717300b802f8bd8784060e15cd 28f799d
Branches : tmp-ocio-v2
Author : Brecht Van Lommel (brecht)
----------------------------------------
OpenColorIO: address review comments

----------------------------------------
6 Paths Changed]]>
Wed, 03 Feb 2021 14:12:28 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/28f799df9260f3717300b802f8bd8784060e15cdBrecht Van Lommel
<![CDATA[[6adb1fd]: Windows: OCIO2 build fixes]]> http://www.miikahweb.com/en/blender/git-logs/commit/6adb1fd37c5dfbcec3a4fc1516790c48d0855e71 6adb1fd
Branches : tmp-ocio-v2
Author : Ray molenkamp (LazyDodo)
----------------------------------------
Windows: OCIO2 build fixes

Nothing too interesting here, the static imports define
changed as did some of the library names/locations

----------------------------------------
3 Paths Changed]]>
Mon, 01 Feb 2021 17:47:44 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/6adb1fd37c5dfbcec3a4fc1516790c48d0855e71Ray molenkamp
<![CDATA[[0eb26e3]: OpenColorIO: update processors and transforms for version 2...]]> http://www.miikahweb.com/en/blender/git-logs/commit/0eb26e349d7e671c1a293932b168ee753458a19e 0eb26e3
Branches : tmp-ocio-v2
Author : Brecht Van Lommel (brecht)
----------------------------------------
OpenColorIO: update processors and transforms for version 2.0

CPU processors now need to be created to do CPU processing. These are cached
internally, but the cache lookup is not fast enough to execute per pixel or
texture sample, so for performance these are now also exposed in the C API.

The C API for transform will no longer be needed afer all changes, so remove
it to simplify the API and fallback implementation.

----------------------------------------
9 Paths Changed]]>
Mon, 01 Feb 2021 13:46:15 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/0eb26e349d7e671c1a293932b168ee753458a19eBrecht Van Lommel
<![CDATA[[1b01957]: OpenColorIO: remove default display workaround]]> http://www.miikahweb.com/en/blender/git-logs/commit/1b01957bb5961e9ac27fcf566b3e45238bd18473 1b01957
Branches : tmp-ocio-v2
Author : Brecht Van Lommel (brecht)
----------------------------------------
OpenColorIO: remove default display workaround

A fix for this is in 2.0 (and recent 1.1.x versions), no need for this anymore.

----------------------------------------
1 Path Changed]]>
Mon, 01 Feb 2021 14:42:46 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/1b01957bb5961e9ac27fcf566b3e45238bd18473Brecht Van Lommel
<![CDATA[[4e35285]: OpenColorIO: update GPU display shader for version 2.0]]> http://www.miikahweb.com/en/blender/git-logs/commit/4e352851e4dc47d22325db3308d91b879958bcd0 4e35285
Branches : tmp-ocio-v2
Author : Brecht Van Lommel (brecht)
----------------------------------------
OpenColorIO: update GPU display shader for version 2.0

To avoid baking exposure and gamma into the GLSL shader and requiring slow
recompiles when tweaking, we manually apply them in the shader. This leads
to some logic duplicaton between the CPU and GPU display processor, but it
seems unavoidable.

Caching was also changed. Previously this was done both on the imbuf and
opencolorio module levels. Now it's all done in the opencolorio module by
simply matching color space names. We no longer use cacheIDs from OpenColorIO
since computing them is expensive, and they are unlikely to match now that
more is baked into the shader code.

Shaders can now use multiple 2D textures, 3D textures and uniforms, rather
than a single 3D texture. So allocating and binding those adds some code.

Color space conversions for blending with overlays is now hardcoded in the
shader. This was using harcoded numbers anyway, if this every becomes a
general OpenColorIO transform it can be changed, but for now there is no
point to add code complexity.

----------------------------------------
6 Paths Changed]]>
Mon, 01 Feb 2021 14:42:46 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/4e352851e4dc47d22325db3308d91b879958bcd0Brecht Van Lommel
<![CDATA[[7c8cbc6]: OpenColorIO: use aces_interchange role to detect CIE XYZ values...]]> http://www.miikahweb.com/en/blender/git-logs/commit/7c8cbc67abdd0e2293ca810a3832749bb6031649 7c8cbc6
Branches : tmp-ocio-v2
Author : Brecht Van Lommel (brecht)
----------------------------------------
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.

----------------------------------------
7 Paths Changed]]>
Mon, 01 Feb 2021 14:42:46 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/7c8cbc67abdd0e2293ca810a3832749bb6031649Brecht Van Lommel
<![CDATA[[84f4e9b]: Build: OpenColorIO 2.0.0 support for make deps and install_deps...]]> http://www.miikahweb.com/en/blender/git-logs/commit/84f4e9b4313b1595bfd4bda4add23e9aa72bcc7b 84f4e9b
Branches : tmp-ocio-v2
Author : Brecht Van Lommel (brecht)
----------------------------------------
Build: OpenColorIO 2.0.0 support for make deps and install_deps.sh

* Tinyxml was replaced with Expat, adding a new dependency.
* Yaml-cpp is now built as a dependency on Unix, as was already done on Windows.
* Removed currently unused LCMS code, couldn't be bothered to update and test it.
* Pystring remains built as part of OCIO itself, since it has no good build system.

We currently require OpenColorIO 2.0.0 and the upcoming code changes have no
compatibility with 1.x. Compatibility could be added, but it's rather complicated
and I only want to do the work if it's really needed.

The install_deps.sh support was implemented by Jeroen, I have no tested it myself.

The Windows code was updated, but I have not tested if it actually works, it
almost certainly will not.

----------------------------------------
14 Paths Changed]]>
Mon, 01 Feb 2021 13:46:14 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/84f4e9b4313b1595bfd4bda4add23e9aa72bcc7bBrecht Van Lommel
<![CDATA[[a5139ce]: OpenColorIO: update display transforms for version 2.0]]> http://www.miikahweb.com/en/blender/git-logs/commit/a5139ce82c4aeecc09cf44dc4967920c00b3a419 a5139ce
Branches : tmp-ocio-v2
Author : Brecht Van Lommel (brecht)
----------------------------------------
OpenColorIO: update display transforms for version 2.0

Needs a bit more manual work constructing the transform. LegacyViewingPipeline
could also have been used, but isn't really any simpler and since it's legacy
we better not rely on it.

This moves more logic into the opencolorio module, to simplify the API. There is
no need to wrap a dozen functions just to be able to do this in C rather than C++.
It's also tightly coupled to the GPU shader logic, and so should be in the same
module.

----------------------------------------
6 Paths Changed]]>
Mon, 01 Feb 2021 14:42:46 +0000 http://www.miikahweb.com/en/blender/git-logs/commit/a5139ce82c4aeecc09cf44dc4967920c00b3a419Brecht Van Lommel