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-114593 preserve load state of prims #2141

Merged
merged 4 commits into from
Feb 25, 2022

Conversation

pierrebai-adsk
Copy link
Collaborator

Added code to preserve the loaded or unloaded state of a prim when grouped, reparented or renamed. Note that we don't remove the old rules when duplicating since the source prim still exists.

  • Refactor the existing load-state preservation code in reusable functions.
  • Use them in the duplicate, rename, and reparenting commands.
  • Add unit tests for grouping and renaming, which also covers parenting since grouping reparents.

Added code to preserve the loaded or unloaded state of a prim when grouped, reparented or renamed. Note that we don't remove the old rules when duplicating since the source prim still exists.

- Refactor the existing load-state preservation code in reusable functions.
- Use them in the duplicate, rename, and reparenting commands.
ppt-adsk
ppt-adsk previously approved these changes Feb 24, 2022
Copy link
Collaborator

@ppt-adsk ppt-adsk left a comment

Choose a reason for hiding this comment

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

Great job, just minor typos and grammar.


// Retrieve the effective rule for the source path.
//
// The reason we retrieve the effetive rule is that even
Copy link
Collaborator

Choose a reason for hiding this comment

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

"effetive" --> "effective".

//
// Note: the UsdStageLoadRules has a Minimize function that simplifies
// rules, but we don't want to change rules the user might have set.
// The user may expect those rules to exists for some future purpose
Copy link
Collaborator

Choose a reason for hiding this comment

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

"to exists" --> "to exist".

@pierrebai-adsk
Copy link
Collaborator Author

Only the known flaky test TestALUSDMayaPython_LayerManager failed in the interactive linux instance.

@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Feb 24, 2022
@seando-adsk seando-adsk merged commit 81fa56c into dev Feb 25, 2022
@seando-adsk seando-adsk deleted the t_bailp/MAYA-114593/preserve-load-state branch February 25, 2022 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants