In T68239#758664, @Jacques Lucke (JacquesLucke) wrote:

If I understand it correctly, the cryptomatte passes should never be modified by some color space transformation. Is that correct?

I am unable to get hair to render - any scene from the benchmark repository (https://svn.blender.org/svnroot/bf-blender/trunk/lib/benchmarks/cycles/) that contains hair results in a crash.

@Lukas Stockner (lukasstockner97) can you upload a version of the existing that work against the current master? That would make it easier for others to experiment with it.

Just thinking aloud, this might be an opportunity for a small refactor of cycles: currently, Cycles builds the BVH before it uploads geometry vertices to the device. Most ray tracing APIs (DXR, OptiX, Embree) work the other way round. Before we introduce an OptiX specific workaround, should we change Cycles to do vertices first, BVH second? That would also help the Embree backend and potentially a future DXR/Vulkan backend when we see more vendors deliver ray tracing hardware.

@Brecht Van Lommel (brecht) should I land the patch in its current state (OIDN disabled by default) or do we want to wait for install_deps.sh to be fully working first?

In D4304#124368, @YAFU (YAFU) wrote:

Hi.
Will the compilation and installation of OIDN dependencies through install_deps.sh be automated?

There were some more implementation details in the 2019 SIGGRAPH course "My favorite samples":
https://drive.google.com/file/d/1MwphDkw6HeOzA1xcrmy_c7YvartaYjnf/view

make deps and everything else should be set up for static builds, unless I missed something.

Seems to work as intended:

• Compositor: Addressed Brecht's requests for OIDN node.

You beat me to this! :)

• build_environment: Updated to OIDN 1.0.0.

It might be useful to find a way to hide the repetition in the 64x64 pattern, since in some cases it can become visible.