Bernard Kwok

Every atomic and compound node (graph) in a shading network can have 1 or more classifications. Atomic ones include classifying things like closures / shaders, texture etc. After building the...

> I'd like to bump this issue as I'm not aware of a workaround, and it basically breaks all subgraph-focused workflows as far as I can tell. For this particular...

@jstone-lucasfilm, the test suite turns off shader transform insertion by default. I didn't see any firewall test if it's on so I think you can get double transforms if you...

Yes. My mistake. Still might want to put in guards for renderers that parse the colorspace image meta-data. Think I used to do this for Arnold.

Adding `smothstep` as a candidate for edge case docs based on this Slack post: https://academysoftwarefdn.slack.com/archives/C0230LWBE2X/p1724060123942189

Hi @dgovil , This is considered a valid graph (i.e. unique naming within the "namespace" / graph level) so it seems it's more a problem with logic for code generation...

Hey @niklasharrysson , I looked at the code a bit and it's the same issue we've discussed before -- that of lack of appropriate `ShaderGraph` context when dealing with `nodegraph`s...

This looks like it's the same issue : node names created during shader generation are not scoped within graphs (graph is flattened), and also they do not have full pathing...

For this, I think it would be useful to have the traversal API have 2 extensions: 1. Still allow for incremental traversal using the existing iterator but allow the option...

- Can we still add this file, or something equivalent to the test suite are as part of this PR? There are timers in the test suite. - BTW: The...