Fix T53012: Shadow catcher creates artifacts on contact area
authorSergey Sharybin <sergey.vfx@gmail.com>
Fri, 6 Oct 2017 12:27:34 +0000 (17:27 +0500)
committerSergey Sharybin <sergey.vfx@gmail.com>
Fri, 6 Oct 2017 12:27:34 +0000 (17:27 +0500)
The issue was caused by light sample being evaluated to nan at some point.
This is root of the cause which is to be fixed, but is very hard to trace down
especially via ssh (the issue only happens on AVX2 release build). Will give it
a closer look when back to my AVX2 machine.

For until then this is a good check to have anyway, it corresponds to what's
happening in regular radiance sum.

intern/cycles/kernel/kernel_accumulate.h

index ae5f6e5..366f254 100644 (file)
@@ -557,7 +557,11 @@ ccl_device_inline void path_radiance_sum_shadowcatcher(KernelGlobals *kg,
        float path_total = average(L->path_total);
        float shadow;
 
-       if(path_total == 0.0f) {
+       if(UNLIKELY(!isfinite_safe(path_total))) {
+               kernel_assert(!"Non-finite total radiance along the path");
+               shadow = 0.0f;
+       }
+       else if(path_total == 0.0f) {
                shadow = L->shadow_transparency;
        }
        else {