ampas / idt-calculator

ACES Input Device Transform Calculator Apps
Other
10 stars 3 forks source link

The DCTL template still still includes a clamp at 1.0 #97

Open nick-shaw opened 1 month ago

nick-shaw commented 1 month ago

The _clipf() function clamps any values above 1.0:

    // Apply exposure and white balance factors
    const float Rraw = _clipf((b[0] * p_R * k) / min_b);
    const float Graw = _clipf((b[1] * p_G * k) / min_b);
    const float Braw = _clipf((b[2] * p_B * k) / min_b);

If I remember correctly an inappropriate clamp was removed from other templates in the P-2013-001 app.