I've made some updates and I believe this patch if ready for further review. The recent changes include:
- Fixed issue saving the skinned mesh in its rest pose.
- Ensuring that disabled…
This is definitely something to consider. However, I'm thinking that perhaps such an optimization could be done in a new follow-up PR, to simplify debugging and testing initially. Would that be…
I would rather wait for a final decision for vgroup topic first. Renaming randomly another generic attribute is not trivial, this can have completely unexpected consequences too (if other…
@HooglyBoogly Since Blender has string attributes, would it make sense to also handle converting USD StringArrays? I can't remember if this was discussed in the original attribute import pull…
however, there is an on-going issue with regard to that new 'do not allow same names for vertex groups and generic attributes' behavior, and it is likely to be reverted for 4.0... so not so sure…
I added a utility function to rename the attributes and I believe this addresses the requested changes.