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

Settings: Host imageio use AYON settings #53

Merged
merged 3 commits into from
Feb 14, 2024

Conversation

iLLiCiTiT
Copy link
Member

Changelog Description

Imageio settings in hosts does not require conversion.

Additional info

The changes should be affecting only settings that are used in colorspace.py. Modified colorspace.py to use AYON data structure. I'm not sure if there is a host which is using directly imageio settings that are handled by colorspace.py? I couldn't find any.

Testing notes:

I need help with testing notes as I don't know what it in fact affects. Probably validate if imageio is used correctly during publishing?

@ynbot ynbot added type: enhancement Improvement of existing functionality or minor addition size/XS labels Feb 13, 2024
Copy link
Member

@moonyuet moonyuet left a comment

Choose a reason for hiding this comment

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

It doesn't work at Max when enabled imageio and global ocio config(not sure if it is related to this particular PR). The script editor shows me an error when I use the Set Colorspace in Max
image
Maybe it is somehow related to the ocio env hook isn't installed correctly in Max.
My core settings in imageio:
image

My Max settings in imageio:
image

@jakubjezek001
Copy link
Member

It doesn't work at Max when enabled imageio and global ocio config(not sure if it is related to this particular PR). The script editor shows me an error when I use the Set Colorspace in Max

Have a look if you are having activated ayon_ocio in your bundle.

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.

works perfect in nuke! ;)

Copy link
Member

@moonyuet moonyuet left a comment

Choose a reason for hiding this comment

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

Works now... Thanks @jakubjezek001 for help!
image

@iLLiCiTiT iLLiCiTiT merged commit 43a67fe into develop Feb 14, 2024
9 checks passed
@iLLiCiTiT iLLiCiTiT deleted the enhancement/OP-8196_Imageio-use-AYON-settings branch February 14, 2024 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS type: enhancement Improvement of existing functionality or minor addition
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants