Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Maya: Safer handling of expected render output names #1496

Merged

Conversation

create-issue-branch[bot]
Copy link

@create-issue-branch create-issue-branch bot commented May 7, 2021

Feature

This adds better handling of render settings in Maya.

First part concerns Creator:

If render instance is created, based on selected renderer sane default settings are applied to render settings - animation is turned on, exr is set as default format and resolution is set from asset. For vray _ is used as default render element separator.

Todo: this should be extended to be more universal. For example by loading json data from Settings so TD can add/override per projects those renderer settings.

Second part is Validator that must validate all OpenPype required render settings. That could be also extended by json data from Settings, so custom renderer settings can be validated.

Json data should include attribute name and expected value:

{
    "renderer": "vray",
    "attributes": {
         "someNodeType.someAttribute": "value"
    }
}

someNodeType will be used to get actual node (cmds.ls(type="someNodeType")) as render setting nodes can have different names.

closes #1159

@antirotor antirotor self-assigned this May 7, 2021
@antirotor antirotor added type: enhancement Enhancements to existing functionality host: Maya labels May 7, 2021
@antirotor
Copy link
Member

this now include changes in #1504 as it depends on it

@antirotor antirotor requested a review from mkolar May 12, 2021 16:40
@antirotor antirotor marked this pull request as ready for review May 12, 2021 16:40
@mkolar mkolar merged commit d303c23 into develop May 17, 2021
@mkolar mkolar deleted the feature/1159-maya-safer-handling-of-expected-render-output-names branch May 17, 2021 11:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Maya type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maya: Safer handling of expected render output names
2 participants