Wow, that is quite a list of presets. But guess people would love to have them all.
To be honest, I had no idea which ones are relevant in practice so I just took all of them.
Overall seems good (especially having tests!), just three nitpicks.
Can't we avoid the trigonometric functions here by
Nitpick: I'd prefer explicitly computing F_r
and dF_r
and then updating r
based on that. It's fairly obvious here what the respective terms are, but that spells it out more clearly.
Nitpick: I'd prefer sqr(r)
and putting these both in the same line, they're short enough and related.
Agreed, this is definitely good to have (also for physically-based metallic thin film in the future). Sorry for letting this one slip through the cracks, I'll do a proper review soon.
Okay, I think I got, !123643 should address the issue. I'll just wait for that to be done.
Yes, but the clamping should happen in bsdf_oren_nayar_setup
so that it only affects the multiscattering color correction.
We can remove the
do_diffuse
optimization flags. They are not necessary since EEVEE-Next.
Same for the other do_*
flags? If yes, that should be enough for a while.
Eventually we might…