-
Notifications
You must be signed in to change notification settings - Fork 10
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
Collect required data for local and farm render jobs to have a 'colorspace' value specified #75
Conversation
…space' value specified
As far as I know, this PR's topic brings many topics into the table:
anyways, Here's a snippet of my transcode plugin settings. Here's Mantra local render result on my end |
Here's an error that I faced by accident. I assume it's not related to this PR.
also,
|
This I think you're improving now with: ynput/ayon-core#853
Yes, however - non-critical for now because it seems all default to using Houdini's
As far as I'm aware - this is already done for farm instances, yet not for "local render instances" I assume? Separate issue and should basically be part of cleanup and unifying the farm and local render behavior I'd say.
Any viewer capable of viewing with OCIO colorspaces should suffice. Recent DJV 2 versions should work, MRV2, OpenRV but also compositing software like Nuke or Fusion. What's important is that when you're viewing it - is that you use the same OCIO config as Houdini uses. (And of course set the colorspace values accordingly in the viewer as needed). So, viewing in a viewer a transcoded image (e.g. with display/view applied) should look 1:1 the same as in Houdini without additional color management (it's basically the sRGB view to it) and even a simple photo viewer should display it (if it's e.g. a JPEG) The raw render output image from Houdini should look 1:1 the same as in Houdini when you view it in other software using the same OCIO config as Houdini uses, and with the same display & view transform set. |
@moonyuet uh... that's so odd! I can't think of a reason the code changes here introduce that issue. Nor can I reproduce this? @MustafaJafar has my comment allowed you maybe to test this better / more in-depth to confirm whether things work as they should? |
@BigRoy I tested with the built addon instead of dev mode, it works normally. I used the setting below |
Thanks for testing - was that a Houdini "local render" or "farm render". |
I tested in local render first(I mean the comment above). I will test the farm render soon. |
I faced the exact same colorspace issue in farm rendering with the one in local rendering, which the color-picking mode means correct colorspace when loading in Nuke. |
Thanks @moonyuet So in the first log it:
Using display view with display "ACES" and view "DCDM". That view seems like an odd choice maybe? :) Did you specify that in settings, or? In the second log it:
Correctly assuming your image is in @moonyuet any idea where the If that view were
|
It is an odd choice, but just want to test to make sure the setting works as expected.
It is coming from the setting. I set the display and view as the screenshot showed below. |
Could you try with:
(It should actually also work with keeping Display+View empty, then it should take it from the OCIO default from Houdini; but let's try it explicitly first).
No.
@MustafaJafar is working on improving the settings UI for that here: ynput/ayon-core#853 |
So if all looks good based on this test by @moonyuet (thanks so much!) that means this may be approved and is ready for merge @MustafaJafar ? |
it looks good for me(I just forgot the approve button) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I have a better idea now.
Here's a test from Houdini (Local Render).
My OIIO Transcode settings (I'm currently on my PR ynput/ayon-core#853)
DEBUG: Conversion command: E:\Ynput\ayon-third-party\client\ayon_third_party\downloads\oiio\oiiotool.exe --nosoftwareattrib --colorconfig E:\Ynput\ayon-ocio\client\ayon_ocio\configs\OpenColorIOConfigs\aces_1.2\config.ocio -i:ch=R,G,B,A //storage/work/ayon_projects/Robo/Assets/Character/roboangry/work/cfx/pyblish/renders/mantra_ropTest\mantra_ropTest.1001-1002#.beauty.exr --ch R=R,G=G,B=B,A=A --subimage 0 --colorconvert ACES - ACEScg Output - sRGB -o C:\Users\MUSTAF~1\AppData\Local\Temp\op_transcoding_zn78lunc\mantra_ropTest.1001-1002#.beauty.jpeg
Render View in Houdini | Loaded Render in Nuke | Loaded Review in Nuke | Loaded Review in DJV | Uploaded Review in AYON |
---|---|---|---|---|
I like how they are matched.
About the product naming:
I think it happens because I kept the same values I used for testing #67
Changelog Description
This makes it so that for all render products generated from Houdini renderers and their render instances the colorspace is set to that of the
scene_linear
OCIO role in the currently set OCIO config.By doing so, the transcoding plug-in is capable of using this as the source colorspace when needing to transcode from a source colorspace to a destination colorspace.
Additional info
Without this it'd mean that the transcoding would basically get ignored because it would not know about the source colorspace of the output image. It would get ignored here.
The logic involved with this definitely shows we have some API refactoring to do in
ayon_core
to make this much easier to access and do. For now I've tried to add in comments in the code here what instance data is being set and why for the farm instances. Butayon_core.pipeline.farm.pyblish_functions
can definitely use some cleanup/restructuring to simplify things.Testing notes:
Global color management should be enabled in core! (However, it may be good to do also do at least a test run with one of the renderers with global color management disabled to ensure we're not breaking that.)
ayon+settings://core/publish/ExtractOIIOTranscode
profiles. Note that if you have no matching profiles it would of course still not transcode.