AcademySoftwareFoundation / OpenColorIO

A color management framework for visual effects and animation.
https://opencolorio.org
BSD 3-Clause "New" or "Revised" License
1.74k stars 431 forks source link

Enhance ociochecklut to print the output after each step in a multi-transform LUT #1910

Closed doug-walker closed 5 months ago

doug-walker commented 8 months ago

Some LUT file formats, such as the Common LUT Format (CLF), may contain a series of operators rather than only a single LUT. It would be helpful for debugging color issues if ociochecklut command-line tool would print the output from each operator in the stack rather than only the final result.

This would be a nice task for a new contributor.

pennelee commented 7 months ago

Hi @doug-walker, I would like to take a look at this issue. Is there somewhere I can get a sample LUT, like the one you mentioned? Thank you!

doug-walker commented 7 months ago

@pennelee , thanks for offering to work on this. There are a lot of CLFs in the test data that could be used, for example:

OpenColorIO/tests/data/files/clf/multiple_ops.clf

pennelee commented 7 months ago

Hi @doug-walker , I just wanted to check to see if my thinking is in the right direction. I think I need to create a GroupTransform from the FileTransform's processor. Then access each transform from the GroupTransform one by one, but do I need to create a new Processor for each transform to apply to the input pixel or is there a different way to access each transform individually? Thank you!

doug-walker commented 7 months ago

Hi @pennelee, yes, you need to create a new Processor from each element of the GroupTransform.

pennelee commented 6 months ago

Hi @doug-walker , @carolalynn22 , I have submitted https://github.com/AcademySoftwareFoundation/OpenColorIO/pull/1925 for this issue, let me know if you have any feedback. Thanks!