<div dir="ltr">Thanks for the comments.<div><br></div><div>Indeed, looking at the matrices in apply_saturation, I actually have no recollection where those numbers come from. Actually I suspect it makes no difference mathematically (unless saturation were changed by different amounts on the U and V axes), but I agree it would make sense to use the "official" numbers there.</div><div><br></div><div>As regards the interpolation, the behaviour is intended to match the way the AWB works (so that the two algorithms "agree" about exactly where on the CT curve they are, rather than caring particularly about the reported value). That said, the AWB might do better to work in mireds too, and then this would be changed to match. It would make no difference to the actual white point chosen (in terms of red/blue gains), but the colour temperature number reported in between the calibrated locations might be less approximate. I'll put it on the list of things to look at!</div><div><br></div><div>Thanks again and best regards</div><div><br></div><div>David</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 8 Jun 2020 at 07:35, Jack Hogan <<a href="mailto:jack.hogan77@gmail.com">jack.hogan77@gmail.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"><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" target="_blank">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>
</blockquote></div>