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

Discover "aces-dev". #3

Closed
KelSolaar opened this issue Jun 17, 2020 · 3 comments
Closed

Discover "aces-dev". #3

KelSolaar opened this issue Jun 17, 2020 · 3 comments

Comments

@KelSolaar
Copy link
Contributor

Hi,

Instead of polluting #1, I'm creating a new thread to discuss the discovery repo I have been poking at lately: https://github.com/colour-science/discover-aces-dev

It is an experiment at this stage, but the code walks through https://github.com/ampas/aces-dev/tree/master/transforms/ctl, discovers and classifies all the relevant CTL transforms and transform pairs.

My naive hope is that we might eventually be able to use that process to automatically build the core ACES config that matches aces-dev without too much code.

A direct benefit is that it enforces things to be clean on the CTL side, and while implementing the discovery code, a few things surfaced that I hope we will be able to resolve in a timely fashion.

image

Note that I cheated for the LMTs, they should not be visible because their source and target are AP0. I also culled most of the Alexa transforms, the ridiculous number of them was blowing up the graph.

Cheers,

Thomas

@KelSolaar KelSolaar changed the title Discover ACESdev Discover "aces-dev". Jun 17, 2020
@michdolan
Copy link
Contributor

I think the challenge with this will be generating the OCIO v2 transforms from the underlying CTL algorithms (wherever possible) instead of generating LUTs through ctlrender. How do we determine which transforms must be generated from CTL, and which can be created natively with OCIO? I suspect we'll end up writing functions to standardize common components of ACES with OCIO which would be a bit more manually tuned. Up for discussion though.

At the very least, this discovery code serves to outline the scope of the config, and even to validate its complete representation in CI, which could be useful for detecting when ACES has changed and something is missing.

@KelSolaar
Copy link
Contributor Author

How do we determine which transforms must be generated from CTL, and which can be created natively with OCIO?

My assumption is that any single of the edges in the graph above has a corresponding OCIO transform, if so, then creating the config is really a matter of mapping them together.

@KelSolaar
Copy link
Contributor Author

I think we can close that one now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants