aces-dev icon indicating copy to clipboard operation
aces-dev copied to clipboard

Clamp in ACEScsc.ACES-to-ACEScg.ctl

Open scottdyer opened this issue 3 years ago • 1 comments

https://github.com/ampas/aces-dev/blob/518c27f577e99cdecfddf2ebcfaa53444b1f9343/transforms/ctl/csc/ACEScg/ACEScsc.Academy.ACES_to_ACEScg.ctl#L32

This clamp seems erroneous. None of the other color space conversion transforms contain a similar clamp and this seems wrong as it breaks reversibility - any valid ACES (AP0) values outside of the AP1 triangle would be lost if converted to ACEScg and back.

Is there a reason why this clamp is here? Would removing this break anything for CG artists?

scottdyer avatar Sep 04 '20 01:09 scottdyer

Sure feel like it should not be here!

KelSolaar avatar Sep 04 '20 01:09 KelSolaar