The nested example also happens to hit unfortunate collection naming collisions. The outer instances use "Instance0" as well as the inner instances etc. which leads to "Instance0.001". But the…
I'm attaching instanced_teapots.usda as an example of instancing in a Z-up scene, for comparison.
Thanks for the comments! In reply to the first issue:
It looks like the
convert_to_z_upprocessing isn't being triggered for the original geometry because they're not root xforms. So…
These changes look very good to me and work nicely in my tests.
When setting the scale and bias attribute values here and below, I'm not sure if it's necessary to construct a VtValue and I believe it should be safe to set the GvVec4f values directly, i.e., by calling scale_attr.Set(scale) and bias_attr.Set(bias). But I don't feel strongly about making this change.
As above, should the function be declared static?
Should this function be declared static, as it's local to the file?
