Skip to content

Comments

If not present initialize HdStormRendererPlugin__domeLightTexturesMax…#238

Merged
lilike-adsk merged 1 commit intodevfrom
tremblp/HYDRA-1227/storm_ibl_sample_setting_default
Jan 20, 2025
Merged

If not present initialize HdStormRendererPlugin__domeLightTexturesMax…#238
lilike-adsk merged 1 commit intodevfrom
tremblp/HYDRA-1227/storm_ibl_sample_setting_default

Conversation

@ppt-adsk
Copy link
Collaborator

…Resolution optionVar to 1024.

@ppt-adsk ppt-adsk requested a review from lilike-adsk January 17, 2025 19:45
@ppt-adsk ppt-adsk self-assigned this Jan 17, 2025
// solution with a Hydra preferences category in the Maya
// preferences UI is preferable, but at time of writing is not in
// scope. PPT, 17-Jan-2025.
MGlobal::executeCommand("if (!`optionVar -exists HdStormRendererPlugin__domeLightTexturesMaxResolution`) { optionVar -iv HdStormRendererPlugin__domeLightTexturesMaxResolution 1024; }");
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

A temporary solution until a more general set of Maya Hydra preferences, along with a category in the Maya preferences UI, is implemented.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Sounds good at this moment.

@ppt-adsk ppt-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Jan 20, 2025
@lilike-adsk lilike-adsk merged commit 1068d23 into dev Jan 20, 2025
10 checks passed
@lilike-adsk lilike-adsk deleted the tremblp/HYDRA-1227/storm_ibl_sample_setting_default branch January 20, 2025 14:56
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.

2 participants