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

Adsk Contrib - Add preliminary support for ACES Metadata Files (AMF) #1604

Merged
merged 2 commits into from
Apr 5, 2022

Conversation

doug-walker
Copy link
Collaborator

Adds a prototype Python tool named pyocioamf that converts an ACES Metadata File (AMF) into the OCIO native transform format CTF. It uses a prototype ACES Reference config file that is serving as a database of ACES Transform IDs for interpreting the AMF file. An example AMF file was included as well, since they are not very widely available currently.

This is intended as a preliminary first step to explore how to best support AMF in OCIO.

Signed-off-by: Doug Walker doug.walker@autodesk.com

Signed-off-by: Doug Walker <doug.walker@autodesk.com>
@hodoulp
Copy link
Member

hodoulp commented Mar 11, 2022

This pull request is still waiting for some review/approval. Thanks.

@hodoulp
Copy link
Member

hodoulp commented Mar 29, 2022

Friendly reminder that the pull request can now be merged this week as it's now more than 2 weeks old.

@remia
Copy link
Collaborator

remia commented Apr 5, 2022

Just confirming, I tested on my side on it worked as expected when I checked the result against standard ACES pipeline (removing temporarily the LMTs in the AMF). Thanks for the work @doug-walker, for a prototype it will already be really helpful!

Copy link
Member

@hodoulp hodoulp left a comment

Choose a reason for hiding this comment

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

As the PR is already more than two-weeks old, I approve it.

@hodoulp hodoulp merged commit a92993f into main Apr 5, 2022
doug-walker added a commit that referenced this pull request Apr 5, 2022
@doug-walker doug-walker added this to the OCIO 2.2.0 milestone Apr 6, 2022
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

Successfully merging this pull request may close these issues.

3 participants