Fix missing origin annotations for MDL#2806
Fix missing origin annotations for MDL#2806jreichel-nvidia wants to merge 4 commits intoAcademySoftwareFoundation:mainfrom
Conversation
These are based on the node paths and were missing for the second input of a unit transform node and input sockets of node graphs.
|
On second thought I'm not that sure anymore about the change for the unit transform node. Do we want multiple nodes with the same path? I don't know where paths are used elsewhere and would leave that decision to others. If the decision is that multiple nodes with the same path is ok/the correct thing to do, then we might want to avoid for the MDL backend multiple parameters with the same origin annotations. These are used by integrations to map arguments for the MaterialX material to parameters of the generated MDL code. The 2nd input of the unit transform node does not have a direct counterpart in the original material. Mapping the value from that path would be logically wrong (and doesn't work anyway because of type mismatches). Instead of asking all integrations to add code for this special case it is probably better not to emit the origin annotation in this case (as before my change). |
|
@jreichel-nvidia This single-line change looks fine to me, assuming it addresses the issue you're encountering in MDL. Can you provide a quick description of the case that it addresses? |
These are based on the node paths and were missing for the second input of a unit transform node and input sockets of node graphs.