Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: ensure defaultArnoldRenderOptions are loaded when using mtoa #62

Merged
merged 2 commits into from
Jan 5, 2024

Conversation

ttblanchard
Copy link
Contributor

What was the problem/requirement? (What/Why)

The Maya Submitter intermittently throws an exception upon opening. This only happens when the mtoa plugin is loaded.

What was the solution? (How)

When the submitter detects Arnold as the renderer, it will try to access Arnold properties, however we are never loading the Arnold options.

Now before we try to access those properties we ensure the mtoa createOptions has been called.

What is the impact of this change?

The Submitter no longer throws an exception when the mtoa plugin is loaded.

How was this change tested?

Installed this version of the plugin on my local Maya 2023 install, then:

  • Open Maya
  • Ensure mtoa.mll is loaded
  • Open and save an empty Maya scene.
  • Open the Deadline Cloud submitter

Previously these steps would trigger an exception.

Did you run the "Job Bundle Output Tests"? If not, why not? If so, paste the test results here.

No, this change only effects plugin loading.

Was this change documented?

No, it's only fixing a regression.

Is this a breaking change?

No.

@ttblanchard ttblanchard requested a review from a team as a code owner December 21, 2023 20:49
@ttblanchard ttblanchard force-pushed the blantrev/fixmayamtoasubmitter branch from e5483f2 to 75e807d Compare December 21, 2023 20:56
@ttblanchard ttblanchard merged commit 8b0ffa8 into mainline Jan 5, 2024
5 checks passed
@epmog epmog deleted the blantrev/fixmayamtoasubmitter branch January 5, 2024 21:33
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.

3 participants