That's a good point, and I've added a comment to explain this. xform is a matrix for doubles (GfMatrix4d), but Blender expects a matrix of floats, so I explicitly convert to a GfMatrix4f…
I refactored the function to return an optional tuple, as you recommended. I think this is an elegant solution. Thanks for the suggestion!
I added test cases for skeleton and blend shape import. These require the attached usd_blend_shape_test.usda and arm.usda files to be copied to the tests/usd/ directory.
I believe at…
@dr.sybren Thank you so much for the comments and testing!
As an intermediary solution, there's also the idea of having bones in pose/object mode just displayed as spheres (#106230), which…
Would be good to add one or two test cases though to our unittests suit for USD ?
Yes, I will be adding the test cases. Thanks!
@mont29 FYI, I just merged the latest from main into this pull request, including the fix to the USDZ import crash on linux.
e2a5846d56
Fix (unreported) improper handling of camera's baground image IDs in foreach_id callback.
I feel like the time is ripe to see what happens when I try to incorporate some of that code in Blender following the mechanism laid out in your Alembic importer.
@devin-bayly You shouldn't…