Skip to content

Comments

HYDRA-2005 : Fix depth peeling with USD data#370

Merged
debloip-adsk merged 3 commits intodevfrom
debloip/HYDRA-2005/depth-peeling-usd
Jan 20, 2026
Merged

HYDRA-2005 : Fix depth peeling with USD data#370
debloip-adsk merged 3 commits intodevfrom
debloip/HYDRA-2005/depth-peeling-usd

Conversation

@debloip-adsk
Copy link
Collaborator

@debloip-adsk debloip-adsk commented Jan 14, 2026

Using depth peeling in MayaHydra was causing z-fighting artifacts with USD data due to it being "rendered" twice, once by MRenderOperations and once by MayaHydra's specific USD handling. To fix this, we specifically exclude MayaUsd from MRenderOperations, so they are only handled by MayaHydra's special USD handling.

I could not get the issue to repro in our test environment, even if I tried removing some of our test-specific options. I did find out that the artifacts are only present when MSAA is off, but that still doesn't make the bug work in testing. There is likely some other element needed to repro, possibly an idle event, that is needed for the issue to occur.

Depends on the following PR in maya-usd : Autodesk/maya-usd#4447

@debloip-adsk debloip-adsk requested a review from ppt-adsk January 14, 2026 20:06
@debloip-adsk debloip-adsk self-assigned this Jan 20, 2026
@debloip-adsk debloip-adsk requested review from ppt-adsk and removed request for ppt-adsk January 20, 2026 18:06
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.

Thanks for the nice documentation.

@debloip-adsk debloip-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Jan 20, 2026
@debloip-adsk debloip-adsk merged commit 6fb3004 into dev Jan 20, 2026
10 checks passed
@debloip-adsk debloip-adsk deleted the debloip/HYDRA-2005/depth-peeling-usd branch January 20, 2026 18: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.

2 participants