forked from bartvdbraak/blender
Cycles: Initialize rng_state for split kernel
Because the split kernel can render multiple samples in parallel it is necessary to have everything initialized before rendering of any samples begins. The code that normally handles initialization of `rng_state` (`kernel_path_trace_setup()`) only does so for the first sample, which was causing artifacts in the split kernel due to uninitialized `rng_state` for some samples. Note that because the split kernel can render samples in parallel this means that the split kernel is incompatible with the LCG.
This commit is contained in:
parent
cd7d5669d1
commit
223f45818e
@ -126,7 +126,7 @@ void KERNEL_FUNCTION_FULL_NAME(data_init)(
|
||||
*use_queues_flag = 0;
|
||||
}
|
||||
|
||||
/* zero the tiles pixels if this is the first sample */
|
||||
/* zero the tiles pixels and initialize rng_state if this is the first sample */
|
||||
if(start_sample == 0) {
|
||||
parallel_for(kg, i, sw * sh * kernel_data.film.pass_stride) {
|
||||
int pixel = i / kernel_data.film.pass_stride;
|
||||
@ -139,6 +139,14 @@ void KERNEL_FUNCTION_FULL_NAME(data_init)(
|
||||
|
||||
*(buffer + index) = 0.0f;
|
||||
}
|
||||
|
||||
parallel_for(kg, i, sw * sh) {
|
||||
int x = sx + i % sw;
|
||||
int y = sy + i / sw;
|
||||
|
||||
int index = (offset + x + y*stride);
|
||||
*(rng_state + index) = hash_int_2d(x, y);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user