PixarAnimationStudios / OpenUSD

Universal Scene Description
http://www.openusd.org
Other
6.11k stars 1.21k forks source link

Set Excuded Paths to UsdImagingGLHdEngine during Runtime #462

Open zhoub opened 6 years ago

zhoub commented 6 years ago

Description of Issue

It would be great to has API to change the excluded paths in runtime when using UsdImagingGLHdEngine, now only has the chance at construction.

Steps to Reproduce

https://github.com/PixarAnimationStudios/USD/blob/dev/pxr/usdImaging/lib/usdImagingGL/hdEngine.cpp

System Information (OS, Hardware)

All

Package Versions

v0.8.4

Build Flags

All

jtran56 commented 6 years ago

Filed as internal issue #159506.

murphyeoin commented 6 years ago

A fix for this would also be very useful for us at Animal Logic - we've had to work around this in the past. Thanks!

c64kernal commented 6 years ago

Thanks guys! We're wondering what the benefit of that would be over rebuilding the UsdImagingGLHdEngine (as far as we can tell it would be close to the same cost)? Any insight here would be great in terms what you're trying to achieve with this API.

murphyeoin commented 6 years ago

Hi George, we had a few use cases, for example in AL_USDMaya we want to be able to "hide" objects from Hydra that we have converted (temporarily) to maya geometry to work on before pushing back to USD. I think we made the assumption that rebuilding/creating a new object would be slower than modifying the state of an existing one - but i'll admit we never tested that. So apologies.. next time we'll get some empirical evidence first.. Thanks, Eoin

c64kernal commented 6 years ago

@murphyeoin got it! That makes a lot of sense. @tcauchois totally called it -- he suggested that maybe visibility control is more of what you're after -- that would certainly be a faster code path.