-
Notifications
You must be signed in to change notification settings - Fork 352
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Web Viewer: When loading file that contains both surfacematerial and a nodegraph with single output, only one material is displayed #1633
Comments
Some "rambling" thoughts on this :). Whatever is done it would be good to have consistent workflows between all sample viewers / editors if plausible. The simple logic currently on load is
The geometry is whatever geometry is currently loaded though the default is a shaderball. The current state for desktop is:
Thus:
I think the choice of how to add this in depends on the intent of the viewer / editor. For:
|
To clarify, with "show all materials" I mean that they and their properties should be listed in the GUI, not "shown on an object in the scene". I would like to be able to click a little button next to each of these names to adjust what is displayed on the ShaderBall. If the model has a stronger opinion (materialassign being used) then that should always win. For the file I attached, that would mean
Does that make more sense? |
Yes. Thanks for the clarification. The button click option sounds equivalent to what I called "solo"ing. I think it should be allowable to override material assignments otherwise if you have all geometry assigned and another material in the same file you can never view it -- but this could be an option. |
Relevant Slack discussion: https://academysoftwarefdn.slack.com/archives/C0230LWBE2X/p1704323996349749
The attached file contains two materials, one implicit (single-output nodegraph) and one explicit (named surfacematerial).
According to the discussion, this is valid.
two-materials.mtlx.zip
I think it would be good if the viewer
The text was updated successfully, but these errors were encountered: