Skip to content

Added HdArnold v2 render settings tests. - #464

Merged
debloip-adsk merged 1 commit into
devfrom
tremblp/HYDRA-2345/hydra_v2_render_settings
Jul 9, 2026
Merged

Added HdArnold v2 render settings tests.#464
debloip-adsk merged 1 commit into
devfrom
tremblp/HYDRA-2345/hydra_v2_render_settings

Conversation

@ppt-adsk

@ppt-adsk ppt-adsk commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Run HdArnold tests with environment variables to use Hydra v2 render settings, in addition to the current set that runs with Hydra v1 render settings.

@ppt-adsk
ppt-adsk requested a review from lanierd-adsk July 3, 2026 20:42
@ppt-adsk ppt-adsk self-assigned this Jul 3, 2026
Comment thread cmake/test.cmake

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Add the capability to add a suffix to create a test with a different label from an existing scene. This is used in this branch to run tests with Hydra v1 render settings as well as Hydra v2 render settings.

return;
}

// Cameras can be invisible and still be renderable, so adapter creation

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Tests with Hydra v2 render settings were failing as Maya cameras were not being translated into the Hydra scene. For Hydra v1 render settings we translate using camera parameters and do not check the visibility status of cameras.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Make sure that InsertDag is not part of the public interface of this class.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Run through the HdArnold tests again, using Hydra v2 render settings in this loop.

Comment on lines +86 to +87
"MAYA_HYDRA_HD_ARNOLD_HYDRA_V2_RENDER_SETTINGS=1"
"USDIMAGINGGL_ENGINE_ENABLE_SCENE_INDEX=1"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Awkwardly, two environment variables are needed to enable Hydra v2 render settings in Arnold. This is being discussed with the Arnold team.

@ppt-adsk

ppt-adsk commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator Author

Mac 2027 build seems to have had machine problems, all other builds are fine.

@ppt-adsk

ppt-adsk commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator Author

Mac 2027 build had machine-related problems, all other builds fine.

@lanierd-adsk lanierd-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 adding these tests.

@ppt-adsk ppt-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Jul 7, 2026
@debloip-adsk
debloip-adsk merged commit 75435a8 into dev Jul 9, 2026
10 of 11 checks passed
@debloip-adsk
debloip-adsk deleted the tremblp/HYDRA-2345/hydra_v2_render_settings branch July 9, 2026 15:06
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