KhronosGroup / KhronosGroup/glTF

EXT_texture_procedurals_mx_1_39 Support Standard Surface

Open
#2,465 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
HTML
Stars
7.8k
Forks
1.2k
Avg merge
17h 26m
Merged PRs (30d)
5

Description

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_materialx

These 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:

1. 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

2. Put conversion into the converter library. If a standard surface is detected, inject [The Existing Conversion Node](https://github.com/KhronosGroup/glTF-MaterialX-Converter/blob/main/tests/data/shader_translation.gltf)
The downside with this is it forces all systems to use the conversion library which they may not do.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the JSON conversion behavior for unsupported graphs and the linked tests/data/shader_translation.gltf example in the glTF-MaterialX-Converter. The issue needs a decision between renderer-side handling and converter-side injection; done means standard-surface graphs produce and assign a usable material without breaking the glTF.

Written by the indexing model from the issue text.

Assessment

Tech stack
three.js
Domain
computer-graphics
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.