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

Nuke: LoadClip colorspace override - OP-6591 #6215

Merged
merged 7 commits into from
Mar 18, 2024

Conversation

tokejepsen
Copy link
Member

@tokejepsen tokejepsen commented Feb 15, 2024

Changelog Description

Setting the colorspace from the representation data was not supported.

Testing notes:

  1. Setup colormanagement for Nuke.
  2. Publish write instance with modified colorspace (color_picking for example).
  3. Load publish write instance and validate the correct colorspace is set.
  4. Add global or host overrided File rules with some other colorspace on your project
  5. Load again and notice that the colorspace was overriden (priority on filerules should override colorspaceData)

@tokejepsen tokejepsen added sponsored Client endorsed or requested port to AYON labels Feb 15, 2024
@ynbot ynbot added type: bug Something isn't working size/S Denotes a PR changes 100-499 lines, ignoring general files host: Nuke labels Feb 15, 2024
@tokejepsen tokejepsen marked this pull request as ready for review February 19, 2024 11:53
@tokejepsen
Copy link
Member Author

@jakubjezek001 ready for testing now.

@tokejepsen tokejepsen marked this pull request as ready for review March 7, 2024 11:49
Copy link
Member

@jakubjezek001 jakubjezek001 left a comment

Choose a reason for hiding this comment

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

tested and worked well

@jakubjezek001 jakubjezek001 merged commit ef33105 into develop Mar 18, 2024
3 checks passed
@ynbot ynbot added this to the next-patch milestone Mar 18, 2024
@jakubjezek001 jakubjezek001 deleted the bugfix/OP-6591_LoadClip-colorspace-override branch March 18, 2024 08:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Nuke port to AYON size/S Denotes a PR changes 100-499 lines, ignoring general files sponsored Client endorsed or requested type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants