[Bf-blender-cvs] [d029399e6b6] master: Code cleanup: remove SOBOL_SKIP hack, seems no longer needed.

Brecht Van Lommel noreply at git.blender.org
Wed Sep 20 20:00:38 CEST 2017


Commit: d029399e6b6d221766ccee49f4814f3b925d6ae8
Author: Brecht Van Lommel
Date:   Tue Sep 19 00:38:22 2017 +0200
Branches: master
https://developer.blender.org/rBd029399e6b6d221766ccee49f4814f3b925d6ae8

Code cleanup: remove SOBOL_SKIP hack, seems no longer needed.

===================================================================

M	intern/cycles/kernel/kernel_random.h

===================================================================

diff --git a/intern/cycles/kernel/kernel_random.h b/intern/cycles/kernel/kernel_random.h
index eb23c77d6a4..11798d87cb5 100644
--- a/intern/cycles/kernel/kernel_random.h
+++ b/intern/cycles/kernel/kernel_random.h
@@ -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 */



More information about the Bf-blender-cvs mailing list