Same as in EEVEE, I don't think we want to hard-clamp >1.0 anymore.
I think this is still left over from before the "Tint is now a color" change? I don't think we want a hard limit on 1.0 now.
This should also be clamped between 0.0 and 1.0, a mix factor >1 makes no sense.
Alright, I rebased this on top of #112192 to reuse the Specular Tint parameter.
I guess we should merge that first and then rebase this branch again.
Is it still planned to be merged for 4.0?
Currently yes afaik.
For glass this actually tints the transmission color too because we use one_spectrum() - F. Is this intended? Should we use reduce_max as we do for layering?
Yeah, this is a tricky question.…
Good point about the transmission. OpenPBR does combine them (see the section about dispersion), but for now I think we can keep them separate.
As for naming - maybe "Specular Adjust"?
Ah, yeah, I wasn't sure what the ideal place for it is here.
With the updated version, this is irrelevant anyways since we don't need ior_from_f0
anymore.
Also, I agree that we should…
True, but I'm not really sure how to phrase it better without making the description overly verbose.