Skip to content

HYDRA-2484 : Register Hydra-based renderers with -cap/cpv flags - #488

Merged
lilike-adsk merged 2 commits into
devfrom
yoonb1/HYDRA-2484/register-hd-renderers-cap-flags
Aug 18, 2026
Merged

HYDRA-2484 : Register Hydra-based renderers with -cap/cpv flags#488
lilike-adsk merged 2 commits into
devfrom
yoonb1/HYDRA-2484/register-hd-renderers-cap-flags

Conversation

@benyoon-adsk

Copy link
Copy Markdown
Collaborator

We need to describe that a renderer registered with the renderer command is Hydra-based. This will allow clients to filter on Hydra-based renderers only, as this is a requirement for USD Render Setup.

This is done with the -capability / -capabilityValue flags (e.g. renderer -capability isHydra -capabilityValue true HdArnoldRendererPlugin) and USD render setup can query renderers with renderer -q -capability isHydra HdArnoldRendererPlugin.

@benyoon-adsk

Copy link
Copy Markdown
Collaborator Author

Need to wait for Maya rundev yaml file update to run preflight.

@ppt-adsk ppt-adsk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Needs a test. Probably do something like
test/lib/cmdLineRender/registeredMayaRenderers.py
Doesn't need an interactive Maya.

<< " -showBatchRenderProcedure " << dq << "batchRender -showImage true" << dq
<< " -renderSequenceProcedure mayaRenderSequence"
<< " -supportColorManagement true"
// Need these to list Hydra-based renderers in USD Render Setup.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// Need these to list Hydra-based renderers in USD Render Setup.
// Need these to list Hydra-based renderers.

Comment on lines +211 to +212
<< " -capability isHydra"
<< " -capabilityValue true"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Style comment, up to you if you like this or not, I would have this in a single line, since it's a single concept. If you'd rather leave it as two lines that's fine as well.

@ppt-adsk ppt-adsk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the test!

@benyoon-adsk benyoon-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Aug 14, 2026
@lilike-adsk
lilike-adsk merged commit 226bbdf into dev Aug 18, 2026
12 of 13 checks passed
@lilike-adsk
lilike-adsk deleted the yoonb1/HYDRA-2484/register-hd-renderers-cap-flags branch August 18, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants