Skip to content

Comments

HYDRA-2005 : Add pluginDisplayFilter for subscene override to fix mayaHydra depth peeling#4447

Merged
seando-adsk merged 3 commits intodevfrom
debloip/HYDRA-2005/fix-maya-hydra-depth-peeling
Jan 15, 2026
Merged

HYDRA-2005 : Add pluginDisplayFilter for subscene override to fix mayaHydra depth peeling#4447
seando-adsk merged 3 commits intodevfrom
debloip/HYDRA-2005/fix-maya-hydra-depth-peeling

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 allow MayaHydra to specifically exclude MayaUSD proxy shapes from MRenderOperations, export the display filter in the API and also register it for the VP2 render delegate.

Sister PR in MayaHydra : Autodesk/maya-hydra#370

ProxyRenderDelegate::drawDbClassification, kRegistrantId, ProxyRenderDelegate::Creator);
CHECK_MSTATUS(status);

status = plugin.registerDisplayFilter(
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Doing the same thing for the Vp2RenderDelegate as for the DrawOverride

Copy link
Collaborator

@pierrebai-adsk pierrebai-adsk left a comment

Choose a reason for hiding this comment

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

I don't understand these rendering things, but code looks good.

@debloip-adsk debloip-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Jan 15, 2026
@seando-adsk seando-adsk added the core Related to core library label Jan 15, 2026
@seando-adsk seando-adsk merged commit d780aff into dev Jan 15, 2026
11 checks passed
@seando-adsk seando-adsk deleted the debloip/HYDRA-2005/fix-maya-hydra-depth-peeling branch January 15, 2026 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Related to core library 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