@LukasStockner There is a problem with your change. Because num_closure is set to 2 in shader_graph.cpp, when it tries to allocate a transparent closure in bsdf_microfacet_hair_setup() it would…
- Fixed the blue=0 issue
- Adjusted versioning - now the IOR 1.2 trick is only used when the Coat input was dynamic (in this case, we avoid needing a Math node). If the input has a fixed value,…
For naming, now about "Random Walk (non-organic)" and "Random Walk (organic)"? The same arguments about complex interface layers also apply to e.g. plants, but not to e.g. marble, so that might be…
Thanks for the review!
Blender crashes when using EEVEE with this pull request on the Metal backend for Blender (Tested with a M1 Pro). I haven't done much debugging on the Metal EEVEE side…
Okay, updated version here:
- Fixed sampling bug in GGX entry bounce
- Removed shadowing-masking term from GGX entry to avoid darkening (we only care about the direction distribution here, so…
- Rebased
- Added Coat IOR as discussed above
- Added descriptions to Coat node sockets
- Tint now reacts to the Coat input
One remaining detail: When creating a new file, it appears that…
Now for the roughness. Remember, this boundary is not directly in relationship with the specular component. We want a compound effect through multiple layers of dead cells (stratum corneum),…
This is probably unintentional so that's why I'm reporting it.
Changing the
Coat Tint
colour while the coat parameter is set to0
still has an impact on the material. ![Clear coat…
@Christophe-Hery, you may have feedback on this.
We've briefly talked at SIGGRAPH. This PR is only about parametrization, I've got a second branch (no PR yet) for updating how the entry…