Transmission depth and subsurface radius don't have suggested bounds about openpbr HOT 5 CLOSED

Fair enough, we can put a soft-max and see if we have feedback on it. For sliders in scene units, 1 seems reasonable indeed.
A paragraph on the topic could be useful as well.

from openpbr.

@virtualzavie Do you mind to make a pull request for this?

from openpbr.

This is difficult since it depends on the scene units, and the length scale of the object being rendered. A depth of 1e6 would be reasonable when rendering a 1km wide body of water in mm units, say, though that's probably atypical.

However if you make the upper limit too large, then the user ends up with a useless slider most of the time. In Maya transmission_depth (soft) maxes out at 100.0, so if you're working in the default cm units with some volume with a MFP of O(1cm) (e.g. orange juice), the slider is functionally useless.

Similarly for subsurface_radius which (soft) maxes out at 10.0 while skin MFP is typically O(1mm).

I think this is really a UI problem though, i.e. the solution is better slider UIs where you can easily adjust the range for convenience.

In the meantime, I'd propose a soft-max of 1.0. At least then for typical objects/units (where MFP and units are roughly in the same ballpark) the slider will do something useful over most of the range, and it's a soft-max so seems more appropriate for it to be too small than too large (i.e. it's more natural to increase it as needed, rather than decrease it from some large default value).

from openpbr.

Will do.

from openpbr.

Thanks for this recommendation, @virtualzavie, as well as for the fix in #114. I'll go ahead and mark this as resolved.

from openpbr.