Sorry about that :(. Thanks for investigating the fix here.
No worries at all! Like I said, I should have caught it in the review myself, since it wasn't obvious how the path building should…
It looks like there are actually 2 separate, unrelated issues. The bug I mentioned above was preventing the UsdSkel animation from being imported at all in Blender 4.2. The fix I have will…
Thanks for pointing this out, @deadpin. It looks like there were two separate issues affecting 4.2, then. (Without the above fix, this UsdSkel animation exported from Blender wouldn't load at…
I believe the issue is a bug in how the joint paths are saved. I.e., forward-slash separators in joint paths are being saved as underscores. E.g., path "mixamorig1_Hips/mixamorig1_Spine" is…
I've confirmed that the UsdSkel export of this animation worked as expected in a build as of a couple of months ago, so I'm comparing the output to determine what might have changed.
Thanks for porting this @CharlesWardlaw ! I was thinking that the unit conversion should be made more generic to handle arbitrary unit scaling, not just centimeters. The initial USD branch…
@makowalski One part I'm not sure about is the "Convert World Material" option currently being under a "Lights" panel, matching the usd branch. It seems that with issue #122651 it might be…
Adding @CharlesWardlaw as a reviewer, as he originally implemented the panels feature in the USD branch.
@makowalski Yes that's a limitation of USD's MaterialX translation currently. Is there a place where you resolve that name? It doesn't seem to make it into the MaterialX graph itself, so it's…
I can only do a partial review (due to time constraints), but the changes to the existing files (e.g., usd_writer_material.cc
, etc.) look good to me. I tested exporting MaterialX to Houdini and…
Good to know the crash is fixed. Otherwise, the changes look good and work well with my tests. I will go ahead and approve this PR. Thank you for this work!
Not sure if it's related to the missing guards you just mentioned, but I get a crash when trying to export a simple animation with an animating transform. To reproduce, load the attached blend…