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

EMSUSD-594 material dup ten times #3329

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

pierrebai-adsk
Copy link
Collaborator

Duplicate failed to preserve material connections when the new name was not immediately near the old name when sorted by name. This happens when duplicating a 9 suffix into a 10 suffix, because the one wirh a 1 suffix sits in-between. In general, due to the way the naming can skip missing values, we cannot and should not assume anything about the relative names.

  • The search optimization failed when the name suffix changed length numerically. (9 -> 10)
  • Fix the merge prim call to also merge children since the SdfCopySpec function copies children.
  • Add a unit test.

@pierrebai-adsk pierrebai-adsk added bug Something isn't working adsk Related to Autodesk plugin labels Sep 15, 2023
@pierrebai-adsk pierrebai-adsk force-pushed the bailp/EMSUSD-594/material-dup-ten-times branch from 8202ee7 to 071d40b Compare September 15, 2023 18:26
- The search optimization failed when the name suffix changed length numerically. (9 -> 10)
- fix the merge prim call to also merge children since the SdfCopySpec function copies children.
- Add a unit test for material connections in duplicate command.
@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Sep 18, 2023
@seando-adsk seando-adsk added ufe-usd Related to UFE-USD plugin in Maya-Usd and removed bug Something isn't working adsk Related to Autodesk plugin labels Sep 20, 2023
@seando-adsk seando-adsk merged commit 8223656 into dev Sep 20, 2023
11 checks passed
@seando-adsk seando-adsk deleted the bailp/EMSUSD-594/material-dup-ten-times branch September 20, 2023 13:10
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 ufe-usd Related to UFE-USD plugin in Maya-Usd
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants