<div dir="ltr">Thanks Laurent, I look forward to hearing their thoughts.<div><br></div><div>Jack</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jun 7, 2020 at 3:01 AM Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com">laurent.pinchart@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Jack,<br>
<br>
On Sat, Jun 06, 2020 at 04:58:00PM +0200, Jack Hogan wrote:<br>
> Hello,<br>
> <br>
> I am not a programmer but I was perusing the calculate_ccm module in ccm.ccp<br>
> and I noticed that the interpolation factor lambda is computed directly off<br>
> correlated color temperature values in deg K.  Normally this is done in mired<br>
> units, that is the interpolation factor is computed using one over each of the<br>
> cct's, see for instance the DNG SDK.  Is there a reason why it was not done<br>
> this way?<br>
> <br>
> Also the matrices in apply_saturation the coefficients look incorrect (from the<br>
> 601 standard instead of 709).<br>
> <br>
> Hope it helps.<br>
<br>
Thanks a lot for the feedback. I've added David and Naush in CC, as the<br>
authors of the code I'm sure they know better than I do :-)<br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div>