Skip to content

Commit 23166ff

Browse files
blevinpixar-oss
authored andcommitted
HdPrman: Move the dependency-declaring scene index a bit later in the order.
Specifically, we want it to run subsequently to the portal light resolving index, since that transfers light filters from a dome light onto portal lights using that dome. We want to pick up that indirect dependency from portal -> dome -> light filter. (Internal change: 2360292)
1 parent 3510694 commit 23166ff

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

third_party/renderman-26/plugin/hdPrman/dependencySceneIndexPlugin.cpp

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,12 @@ TF_REGISTRY_FUNCTION(TfType)
6060
TF_REGISTRY_FUNCTION(HdSceneIndexPlugin)
6161
{
6262
// This scene index should be added *before*
63-
// HdPrman_DependencyForwardingSceneIndexPlugin (which currently uses 1000).
63+
// HdPrman_DependencyForwardingSceneIndexPlugin (which currently uses 1000),
64+
// but subsequent to any scene indexes that generate data sources which
65+
// imply dependencies for this scene index to add. An example is
66+
// HdPrman_PortalLightResolvingSceneIndexPlugin.
6467
const HdSceneIndexPluginRegistry::InsertionPhase insertionPhase
65-
= 100;
68+
= 900;
6669

6770
for(const auto& rendererDisplayName : HdPrman_GetPluginDisplayNames()) {
6871
HdSceneIndexPluginRegistry::GetInstance().RegisterSceneIndexForRenderer(

0 commit comments

Comments
 (0)