Details
Diff Detail
- Repository
- rB Blender
- Build Status
Buildable 8527 Build 8527: arc lint + arc unit
Event Timeline
For the moment, it is just disabled for Principled Hair Shader.
For Hair BSDF shader, it looks like option still has an impact on Transmission.
Having option enabled is not what is expected for an accurate geometry result.
That is the opposite : having option disabled and backfaces taken into account.
Having option enabled is not what is expected for an accurate geometry result.
Yes, it's unexpected for users that BCSDFs don't need backfaces. But that's how they work, they handle all the internal scattering in one step instead of two for efficiency.
Which is why I'm proposing to remove the option, since users don't need to be aware of such implementation details. It should render correctly by default.
I think this is correct direction to get the option removed: this is how BSDF is formulated and how it is expected to be used. Surely, in some cases the backface culling might have been used for an artistic control, but there are other ways to do artistic tweaks.