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

Nuke: LoadClip colorspace override - OP-6591#6215

Merged
jakubjezek001 merged 7 commits intodevelopfrom
bugfix/OP-6591_LoadClip-colorspace-override
Mar 18, 2024
Merged

Nuke: LoadClip colorspace override - OP-6591#6215
jakubjezek001 merged 7 commits intodevelopfrom
bugfix/OP-6591_LoadClip-colorspace-override

Conversation

@tokejepsen
Copy link
Copy Markdown
Contributor

@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
Copy Markdown
Contributor Author

@jakubjezek001 ready for testing now.

@tokejepsen tokejepsen marked this pull request as ready for review March 7, 2024 11:49
Copy link
Copy Markdown
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
@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