You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I get the idea of the system being to match the PBR shader node which makes sense, however MANY of the mtlx materials are made using the standard surface.
The standard shader system needs to map to it but a procedural one doesn't really.
I see that the JSON conversion works for "unsupported" graphs however it doesn't generate a material for it nor assign it to assets breaking the gltf.
I suggest a few options:
Put the onus on the end user on how they handle standard vs PBR surfaces. By passing this along renderers/loaders will map the output to their system. This is the least impact and lets most standard existing MaterialX assets (both standard surface and GLTF_PBR) work
Put conversion into the converter library. If a standard surface is detected, inject The Existing Conversion Node
The downside with this is it forces all systems to use the conversion library which they may not do.
The text was updated successfully, but these errors were encountered:
I get the idea of the system being to match the PBR shader node which makes sense, however MANY of the mtlx materials are made using the standard surface.
The standard shader system needs to map to it but a procedural one doesn't really.
I am working on bringing the extension to threeJS and we already support importing MaterialX
mtlx
files directly: https://threejs.org/examples/?q=materialx#webgpu_loader_materialxThese are all standard surface files.
I see that the JSON conversion works for "unsupported" graphs however it doesn't generate a material for it nor assign it to assets breaking the gltf.
I suggest a few options:
Put the onus on the end user on how they handle standard vs PBR surfaces. By passing this along renderers/loaders will map the output to their system. This is the least impact and lets most standard existing MaterialX assets (both standard surface and GLTF_PBR) work
Put conversion into the converter library. If a standard surface is detected, inject The Existing Conversion Node
The downside with this is it forces all systems to use the conversion library which they may not do.
The text was updated successfully, but these errors were encountered: