Yes, that is a little bit of unfortunate naming. I did not want to shadow the function parameter sc
, but maybe that is fine as its immediately cast to pc
and we can refer to pc
everywhere…
This is inside a check that already checks for either LABEL_PORTAL
or LABEL_TRANSPARENT
(so should not disable MIS on diffuse?), and based on my tests I believe we want to disable MIS for…
This is indeed based on the transparent BSDF code. While adding OSL support I also kept it close to the transparent code, which made the OSL changes very simple: osl_closure_portal_bsdf_setup
…
Yes, that makes it slightly more obvious if either has to change in the future.
This should be fixed now by #117735.
Materials don't influence each other anymore and the "Transparent Shadows" checkbox does what it says, Transparent BSDFs are now not transparent to shadows…
I think I found an unrelated bug with transparent shadows during my tests: If there are any materials that contain transparent closures and have the "Transparent Shadows" checkbox set, all other…
Yes, that one is a bit strange. I did not find a better place to disable transparent shadows for rays intersecting with portals, since I am reusing transparent_extinction, which is used in…