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

HYDRA-1368 - Apply default material on geomSubset #250

Conversation

cossetr-adsk
Copy link
Collaborator

No description provided.

@cossetr-adsk cossetr-adsk self-assigned this Feb 13, 2025
cmds.modelEditor(panel, edit=True, useDefaultMaterial=True)
cmds.refresh()
self.assertSnapshotClose("defaultMaterialGeomSubsetUsdPrims.png", self.IMAGE_DIFF_FAIL_THRESHOLD, self.IMAGE_DIFF_FAIL_PERCENT)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This test works in all relevant USD versions, even if the fix only applies to 24.03+

@cossetr-adsk cossetr-adsk marked this pull request as ready for review February 14, 2025 23:19
@lanierd-adsk lanierd-adsk added ready-for-merge Development process is finished, PR is ready for merge and removed ready-for-merge Development process is finished, PR is ready for merge labels Feb 17, 2025
@lanierd-adsk
Copy link
Collaborator

@cossetr-adsk : I will let you add the "ready for merge" label if that's fine for you and you don't want to modify anything.

@cossetr-adsk cossetr-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Feb 18, 2025
@debloip-adsk debloip-adsk merged commit ca70b31 into Autodesk:dev Feb 18, 2025
10 checks passed
@cossetr-adsk cossetr-adsk deleted the cossetr/dev/HYDRA-1368_default_material_geomSubset branch February 18, 2025 16:22
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.

3 participants