Code cleanup: remove SOBOL_SKIP hack, seems no longer needed.
authorBrecht Van Lommel <brechtvanlommel@gmail.com>
Mon, 18 Sep 2017 22:38:22 +0000 (00:38 +0200)
committerBrecht Van Lommel <brechtvanlommel@gmail.com>
Wed, 20 Sep 2017 17:38:08 +0000 (19:38 +0200)
intern/cycles/kernel/kernel_random.h

index eb23c77d6a4e0ac966910aeab16f50d6417c4b63..11798d87cb53189f1746ff13e4bc7cd57357a867 100644 (file)
@@ -30,12 +30,6 @@ CCL_NAMESPACE_BEGIN
 
 #ifdef __SOBOL__
 
-/* Skip initial numbers that are not as well distributed, especially the
- * first sequence is just 0 everywhere, which can be problematic for e.g.
- * path termination.
- */
-#define SOBOL_SKIP 64
-
 ccl_device uint sobol_dimension(KernelGlobals *kg, int index, int dimension)
 {
        uint result = 0;
@@ -73,7 +67,7 @@ ccl_device_forceinline float path_rng_1D(KernelGlobals *kg,
 
 #ifdef __SOBOL__
        /* Sobol sequence value using direction vectors. */
-       uint result = sobol_dimension(kg, sample + SOBOL_SKIP, dimension);
+       uint result = sobol_dimension(kg, sample, dimension);
        float r = (float)result * (1.0f/(float)0xFFFFFFFF);
 
        /* Cranly-Patterson rotation using rng seed */