Bernard Kwok

The existing topological sort method cannot be used as it's just connection order sort. (`GraphElement.topogicalSort`). This is what I tried out: ```xml def getSortedChildren(graphElement, useTopologicalSort=True): sortedNames = [] if useTopologicalSort:...

Visually, I get this. ```mermaid graph LR surfacematerial1([surfacematerial1]) style surfacematerial1 fill:#090, color:#FFF texcoord1[texcoord1] multiply1[multiply1] subgraph gradient_test gradient_test_texcoord([texcoord]) style gradient_test_texcoord fill:#09D, color:#FFF gradient_test_type([type:4]) style gradient_test_type fill:#09D, color:#FFF gradient_test_color1([color1:0.9069, 0.5283, 0]) style...

From what I see this is coming from code generation and is a known and logged issue of name scope. Sorry I don't recall which one it is.

There are bunch of UI items [here](https://github.com/AcademySoftwareFoundation/MaterialX/issues/1398). I think your case is similar to another item mentioned.

For "base" types you can use the templated method `getTypeString` in C++. An example usage can be found [here](https://github.com/AcademySoftwareFoundation/MaterialX/blob/a090c73c911cbfc57ec77aaea5a688a12729ed32/source/MaterialXCore/Unit.cpp#L268 ) or [here](https://github.com/AcademySoftwareFoundation/MaterialX/blob/a090c73c911cbfc57ec77aaea5a688a12729ed32/source/MaterialXCore/Version.cpp#L184) I do not think there are Python or...

There is no "play" functionality here nor in the desktop viewer. For the hardware case the `u_frame` shader uniform needs to be updated and bound as mentioned [here](https://github.com/AcademySoftwareFoundation/MaterialX/blob/main/documents/DeveloperGuide/ShaderGeneration.md)

@ld-kerley I like this idea of UI separation / reuse. Throwing out a possible syntax `ui` that can reference interface elements: `nodedefs` and compound `nodegraphs`: ```xml ``` Can take this...

@WaffleBoyTom, @jstone-lucasfilm : I'll take a quick look. The current logic does need some changes. It should be: 1. Are we currently inside a nodegraph: ` _isNodeGraph` 2. Have we...

This is what I've found that works: ```c++ bool isNodeGraph = _isNodeGraph; // Check if we're already inside a nodegraph if (_currUiNode) { // Check if current node is a...

Leaving a note that I reworked Manual Koster's stubs as a new repo: https://github.com/kwokcb/types-MaterialX. This approach leaves it as a type package which can be installed as desired vs part...