-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed
Description
Some materials have input setup that makes no sense (i.e: in masonJarGlass/mtlxopen_pbr_surface):
- Image file paths sometimes referenced from
@textures/and other times from@..texture/and some implementations would only be able to load the latter. The default USD resolver would anchor relative paths to the directory of the file that references them, as opposed to relative to the stage root, which in the former case would have it look for the textures under/materials/textureswhich does not exist. - Color space being set
srgb_tx(which does not exist in MaterialX) instead of justsrgb(i.e:BaseColor/inputs:file) - Declaration of input without any assignment or connection (i.e:
color3f inputs:geometry_opacity) - Inputs being set within a disabled section (i.e:
coat_iorandcoat_roughnessbeing set, whilecoat_weightis left at0.0) - Inputs being set to a value and then connected to a node (i.e:
float inputs:transmission_weight) - Inputs of invalid names used (i.e:
masonJarWater/float inputs:specular_anisotropyshould bespecular_roughness_anisotropy) - Non-sensical IOR of 1 makes materials effectively pass-through (ie:
masonJarWater/float inputs:coat_ior = 1) - Subsurface settings of radius and scale are set on Thin Walled materials (i.e:
paper) which has no effect as per the spec. - Emission luminance inputs are fed from images in range 0-1 where luminance values should be in the thousands (i.e:
meetMat) - Emission luminance in
meetMatis coming in containing negative values, and OpenPBR is not (yet) mandating clamping to specified ranges in the spec. It would be better to add an explicit clamp node there. - Orange juice group is set to be invisible in the geometry usd file, but is still being rendered when coming through the USD layer composition. This is evident in Omniverse. It is only not rendering in Arnold because of some arnold-specific primVards that disable it (which should not exist in this content at all for portability). Once the arnold-specific primvars are removed, the orange juice group is also rendered in Arnold.

Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is needed