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

MAYA-114188 save the target layer #2884

Merged
merged 3 commits into from
Feb 21, 2023

Conversation

pierrebai-adsk
Copy link
Collaborator

  • When saving a Maya scene, the current target layer of USD stage get saved in an attribute on the proxy shape.
  • When loading a scene, when creating the USD stage, set the target layer to the one that was saved.
  • The target layer is persisted as a text attribute containing the USD layer identifier.
  • Listen to edit target change to save the attribute when the target is set to avoid losing it if the proxy shape gets recomputed.
  • Add a unit test,

- When saving a Maya scene, the current target layer of USD stage get saved in an attribute on the proxy shape.
- When loading a scene, when creating the USD stage, set the target layer to the one that was saved.
- The target layer is persisted as a text attribute containing the USD layer identifier.
- Add a unit test,.
- Listen to edit target change to save the attribute when the target is set to avoid losing it if the proxy shape gets recomputed.
@pierrebai-adsk pierrebai-adsk added enhancement New feature or request adsk Related to Autodesk plugin labels Feb 16, 2023
@pierrebai-adsk pierrebai-adsk added the schema Related to core library schemas label Feb 16, 2023
@pierrebai-adsk
Copy link
Collaborator Author

Only failure is testUsdExportPackage detecting undeletable files in temp. Why are we even checking the content of temp in that test? It keeps randomly failing due to that.

Copy link
Collaborator

@vlasovi vlasovi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Feb 17, 2023
@seando-adsk seando-adsk merged commit 58a8469 into dev Feb 21, 2023
@seando-adsk seando-adsk deleted the bailp/MAYA-114188/persist-edit-target branch February 21, 2023 20:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adsk Related to Autodesk plugin enhancement New feature or request ready-for-merge Development process is finished, PR is ready for merge schema Related to core library schemas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants