diff options
author | Sebastian Herholz <sherholz> | 2022-04-28 19:03:24 +0300 |
---|---|---|
committer | Sergey Sharybin <sergey@blender.org> | 2022-04-28 19:14:06 +0300 |
commit | fbc884d2a82951c3a8c0f92993de3a0eb9907d86 (patch) | |
tree | 0404bd829d753f28226b4ba103575431b938c3ec /source/blender/bmesh/intern/bmesh_interp.h | |
parent | bd327e3bf3e0768aa2768949304571e0eb73ab80 (diff) |
Ensure BSDF evals and pdfs are zero on invalid samples
Currently, the `eval` and `pdf` are not explicitly set to zero when a BSDF sample is invalid (e.g., below the upper hemisphere), when calling
`bsdf_sample` or `bsdf_eval`. It is assumed that `eval` and `pdf` are set to zero before these functions are called, which can cause problems if not.
This patch fixes this potential problem by explicitly setting `eval` and `pdf` to zero when the sampled direction is invalid.
I also added a sanity check if `eval` and `pdf` are valid (i.e., >= 0.f).
The check is activated when build in debug mode and with the `WITH_CYCLES_DEBUG` set to `ON`.
Reviewed By: brecht, sergey
Differential Revision: https://developer.blender.org/D14776
Diffstat (limited to 'source/blender/bmesh/intern/bmesh_interp.h')
0 files changed, 0 insertions, 0 deletions