Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adapt to refactoring in core USD of registry handling of prim specs versus prim definitions #738

Merged

Conversation

mattyjams
Copy link
Contributor

These changes were made by the core USD team to allow some refactoring in the schema registry. They make use of API added in core USD version 20.05 so older APIs can be removed for later versions of USD. Support for older versions of USD is maintained as well.

The changes here are required to build against core USD at commit PixarAnimationStudios/OpenUSD@0e3171a or later.

@kxl-adsk kxl-adsk added the build Related to building maya-usd repository label Aug 25, 2020
@kxl-adsk kxl-adsk self-requested a review August 25, 2020 13:00
…ange where we will replace the prim spec in SchemaAdaptor with a pointer to a UsdPrimDefinition.

1. Moving more of the code in GetSchemaByName() and ApplySchemaName() into the
   version separated #ifdef blocks so that it can be changed in vesion > 2002
   block only.
2. SchemaAdaptor will also store the schema name token as that will not be able
   to be retrieved directly from a UsdPrimDefinition.
3. _GetMayaAttrNameOrAlias() takes a name instead of an attribute spec.
4. Added a static accessor, _GetAttributeSpec(), to reduce the number of places
   that will require USD version #ifdefs when the SchemaAdaptor is able to hold
   a UsdPrimDefinition.

(Internal change: 2092234)
…ptor

This involves converting the SchemaAdaptor to store a UsdPrimDefinition pointer
instead of an SdfPrimSpecHandle while maintaining backwards compatibility with
older versions of USD that don't have UsdPrimDefinition.

GetSchemaPrimSpec() was removed in core USD commit
PixarAnimationStudios/OpenUSD@0e3171a

(Internal change: 2092250)
This identifies core USD commit 5d1771d (post 20.08 release) as the most recent
supported version of USD:

PixarAnimationStudios/OpenUSD@5d1771d
@mattyjams mattyjams force-pushed the pr/prim_spec_vs_prim_definition_refactoring branch from ab96615 to fdc9e67 Compare August 25, 2020 16:44
@mattyjams
Copy link
Contributor Author

Rebased to resolve merge conflict in doc/build.md following PR #734.

@kxl-adsk kxl-adsk merged commit 4fa022a into Autodesk:dev Aug 27, 2020
@mattyjams mattyjams deleted the pr/prim_spec_vs_prim_definition_refactoring branch August 27, 2020 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to building maya-usd repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants