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-104544 Handle changing the USD stage #474

Merged

Conversation

williamkrick
Copy link
Contributor

The cached usdStage in the proxyRenderDelegate can change. If that happens delete everything cached in the ProxyRenderDelegate and clear the subscene container so VP2 draws correctly.

…ete everything and clear the subscene container.
@williamkrick williamkrick requested review from huidong-chen and kxl-adsk and removed request for huidong-chen April 29, 2020 19:27
@williamkrick
Copy link
Contributor Author

The changes I've done aren't perfect. In some cases when I load a new usd stage some of the old stage keeps on drawing in the viewport. Looking at the documentation for MPxSubSceneOverride it seems like clearing the MSubSceneContainer should be enough to prevent anything from the old stage from drawing, so I'm guessing there is still a bug in Maya which is causing that problem.

I'll wait until @HdC-adsk makes some progress on the drawing issues he's looking at before tackling this additional issue, in case they are duplicates.

@kxl-adsk kxl-adsk added the vp2renderdelegate Related to VP2RenderDelegate label Apr 29, 2020
@williamkrick williamkrick added the proxy Related to base proxy shape label Apr 30, 2020
lib/mayaUsd/nodes/proxyShapeBase.cpp Show resolved Hide resolved
lib/mayaUsd/nodes/proxyShapeBase.cpp Outdated Show resolved Hide resolved
lib/mayaUsd/nodes/proxyShapeBase.cpp Show resolved Hide resolved
lib/mayaUsd/render/vp2RenderDelegate/proxyRenderDelegate.h Outdated Show resolved Hide resolved
@williamkrick williamkrick requested a review from kxl-adsk May 1, 2020 19:11
Copy link
Contributor

@pilarmolinalopez pilarmolinalopez left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks!

@kxl-adsk kxl-adsk merged commit 1576883 into dev May 4, 2020
@kxl-adsk kxl-adsk deleted the krickw/MAYA-104544/update_the_cached_usdStage_if_it_changes branch May 4, 2020 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proxy Related to base proxy shape vp2renderdelegate Related to VP2RenderDelegate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants