Feature/eng 3362/fix color management prefs disable#373
Open
rhdutta wants to merge 3 commits intoynput:developfrom
Open
Feature/eng 3362/fix color management prefs disable#373rhdutta wants to merge 3 commits intoynput:developfrom
rhdutta wants to merge 3 commits intoynput:developfrom
Conversation
BigRoy
reviewed
Dec 9, 2025
Contributor
BigRoy
left a comment
There was a problem hiding this comment.
I see the reasoning - but I'm wondering if it was indeed the intend to "fully disable it".
Usually disabling OCIO management from AYON meant that it's 'unmanaged' by AYON and not that it was force disabled. @jakubjezek001 @antirotor thoughts? I'm fine with having a way to force disable OCIO in Maya - I just wonder what the right 'settings' would be like?
E.g. globally having host management enabled but workfile flag disabled I think makes more sense to be the thing that sounds like "manage it, but disable it explicitly". But now this PR makes it seems like if host management is disabled then force disable it always?
| ) | ||
| return | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changelog Description
Add support for
activate_host_color_managementsetting in Maya'sset_colorspace()function. When this setting isfalse, Maya's color management is now explicitly disabled.Additional review information
Previously, only the
workfile.enabledsetting was checked, which would simply skip color management configuration but not disable it, leaving Maya with its default (enabled) state. This allows studios to explicitly disable color management via AYON settings rather than relying on Maya's default preferences.Testing notes:
activate_host_color_management: false,workfile.enabled: falseactivate_host_color_management: true,workfile.enabled: falseactivate_host_color_management: true,workfile.enabled: true