Skip to content

Conversation

@michaeldsmith
Copy link
Collaborator

@michaeldsmith
Copy link
Collaborator Author

@ThomasWilshaw let us know if you have any feedback on this PR, I think it addresses your comments on closed PR #191

@ThomasWilshaw
Copy link
Contributor

To reply to your comment in the previous PR, is there any reason not to copy the whole header? I guess the complexity comes if the CTL changes something that should be reflected in the header, like the chromaticity coordinates. Would a solution be to do what you are as default but add an option to duplicate the header? I'll confess I don't know what the best approach is.

@michaeldsmith
Copy link
Collaborator Author

Thanks - I'll go ahead and merge this PR and we can work a more general approach to EXR metadata in the future

@michaeldsmith michaeldsmith merged commit 1aa4da8 into ampas:master Oct 7, 2025
85 checks passed
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.

ctlrender crashes with test image t08.exr, data window min is (30,40) not (0,0)

2 participants