Skip to content

Conversation

@erikaharrison-adsk
Copy link
Contributor

Description of Change(s)

  • This change will make TaskControllerSceneIndex be compatible with all storm based renderDelegate.
    • By using dynamic_cast() for testing instead of TfToken testing, this is the same behavior as original HdxTaskController.

Link to proposal (if applicable)

  • N/A

Fixes Issue(s)

  • N/A

Checklist

This testing is similar to original TaskController storm testing, instead of
using pluginName, using a dynamic_cast for testing if SceneIndex is serving a
HdStRenderDelegate. This will help renderDelegate derived from Storm.

Signed-off-by: Xichen Zhou <[email protected]>
@tpalmergit15
Copy link

Filed as internal issue #USD-11155

(This is an automated message. See here for more information.)

@JustAlejandro
Copy link
Contributor

We ended up re-implementing this change, so in the next release we'll include an isForStorm argument in HdxTaskControllerSceneIndex's constructor parameters rather than the current rendererPluginName argument. The parameter can then be set by calling a HdxIsStorm helper. This is intended to be temporary since we don't want scene indices to configure themselves based on render delegate, but is necessary for now.

@pixar-oss pixar-oss closed this in 833d130 Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants