Wataru Tada

MaterialXView can not represent a small Subsurface Scale

1

MaterialXView can not represent a small Subsurface Scale such as 0.01 because mfp is limited to 0.1. ([mx_integrate_burley_diffusion](https://github.com/AcademySoftwareFoundation/MaterialX/blob/a255e89372e768b806c63104336dd5e67edf24fc/libraries/pbrlib/genglsl/lib/mx_microfacet_diffuse.glsl#L60)) vec3 shape = vec3(1.0) / max(mfp, 0.1); All screen shots are -Indirect...

cameraZoom affect Subsurface scattering in MaterialXView

3

cameraZoom is used to [scale the mesh](https://github.com/AcademySoftwareFoundation/MaterialX/blob/b0b42ec81e01e182e370ada6d1911c0101201cd1/source/MaterialXView/Viewer.cpp#L2287). If cameraZoom is changed, fwidth(P) in [mx_subsurface_scattering_approx](https://github.com/AcademySoftwareFoundation/MaterialX/blob/b0b42ec81e01e182e370ada6d1911c0101201cd1/libraries/pbrlib/genglsl/lib/mx_microfacet_diffuse.glsl#L81) is also changed. Was this done intentionally? Both screen shots are -Indirect Lighting OFF -Subsurface 1...

Coat is partially unaffected by Emission in MaterialXView

2

[SurfaceNodeGlsl::emitFunctionCall](https://github.com/AcademySoftwareFoundation/MaterialX/blob/322bf9bd9dc5be6265d7ead677737f36d294458e/source/MaterialXGenGlsl/Nodes/SurfaceNodeGlsl.cpp#L177) adds Emission after direct lighting calculation, so Coat is not taking partial effect of Emission. At Standard Surface, Emission is affected by Coat.  I think the current implementation...

PXR_AR_DISABLE_PLUGIN_RESOLVER raises Undefined behavior

1

### Description of Issue in _GetAvailablePrimaryResolvers() https://github.com/PixarAnimationStudios/OpenUSD/blob/2864f3d04f396432f22ec5d6928fc37d34bb4c90/pxr/usd/ar/resolver.cpp#L313 If PXR_AR_DISABLE_PLUGIN_RESOLVER is set, allAvailableResolvers uses emptyResolverList and availablePrimaryResolvers will be empty vector. in TF_VERIFY(availablePrimaryResolvers.back().type == defaultResolverType); https://github.com/PixarAnimationStudios/OpenUSD/blob/2864f3d04f396432f22ec5d6928fc37d34bb4c90/pxr/usd/ar/resolver.cpp#L347C5-L347C77 std::vector::back raises undefined behavior when...