Matthias, Andi,<br><br>It will be better to use bugzilla and post new entry with patch than mailing list. Look here :<br><br><a href="https://bugs.kde.org/buglist.cgi?query_format=advanced&short_desc_type=allwordssubstr&short_desc=&product=digikamimageplugins&component=Lens+Distortion">https://bugs.kde.org/buglist.cgi?query_format=advanced&short_desc_type=allwordssubstr&short_desc=&product=digikamimageplugins&component=Lens+Distortion</a><br>
<br>Gilles<br><br><div class="gmail_quote">2009/3/10 Matthias Welwarsky <span dir="ltr"><<a href="mailto:matze@welwarsky.de">matze@welwarsky.de</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Tuesday 10 March 2009 09:14:43 Andi Clemens wrote:<br>
> Ok so I'll wait for another update...<br>
<br>
</div>I think I've fixed it, but no time to test. Will post an update tonight.<br>
<div><div></div><div class="h5"><br>
><br>
> Andi<br>
><br>
> On Tuesday 10 March 2009 08:56:48 Matthias Welwarsky wrote:<br>
> > On Monday 09 March 2009 11:30:27 Andi Clemens wrote:<br>
> > > I will take a look it the patch now...<br>
> ><br>
> > I've taken another look into it, seems its not complete yet. Geometry and<br>
> > distortion correction also need subpixel access, and I've found that it<br>
> > doesn't work any more if I check "16 bits color depth" in the "Raw<br>
> > Conversion" settings.<br>
> ><br>
> > > Andi<br>
> > ><br>
> > > On Sunday 08 March 2009 19:51:12 Gilles Caulier wrote:<br>
> > > > Matthias,<br>
> > > ><br>
> > > > Thanks a lots. this patch solve a task from my TODO list.<br>
> > > ><br>
> > > > This patch is a good candidate to go to 0.11 development branch and,<br>
> > > > if it doesn't break i18n, in 0.10 (trunk) before finale release.<br>
> > > ><br>
> > > > Marcel, Andi, can you check this patch for me and guide Matthias on<br>
> > > > the right way. I'm very busy here to prepare my evening classes<br>
> > > > presentation for next Wednesday. To test, just use these pictures :<br>
> > > ><br>
> > > > <a href="http://digikam3rdparty.free.fr/TEST_IMAGES/LENSFUN/" target="_blank">http://digikam3rdparty.free.fr/TEST_IMAGES/LENSFUN/</a><br>
> > > ><br>
> > > > Thanks in advance.<br>
> > > ><br>
> > > > Gilles Caulier<br>
> > > ><br>
> > > > 2009/3/8 Matthias Welwarsky <<a href="mailto:matze@welwarsky.de">matze@welwarsky.de</a>><br>
> > > ><br>
> > > > > Hello,<br>
> > > > ><br>
> > > > > the attached patch fixes CA correction in the lens auto-correction<br>
> > > > > tool. The fix<br>
> > > > > consists of two parts:<br>
> > > > > 1.) set Alpha component of corrected pixel to "fully opaque",<br>
> > > > > otherwise corrected image is not visible (trivial)<br>
> > > > > 2.) Implement sub-pixel interpolation in class DImg, necessary for<br>
> > > > > CA and CI<br>
> > > > > correction with lensfun.<br>
> > > > > There are two new pixel color accessor functions:<br>
> > > > > getSubPixelColor() and getSubPixelColorFast(). The first one uses<br>
> > > > > Lanczos interpolation to compute subpixel colors, the second one<br>
> > > > > bilinear interpolation. Lanczos yields better<br>
> > > > > results, but is a lot slower than bilinear. Before using<br>
> > > > > getSubPixelColor(),<br>
> > > > > prepareSubPixelAccess() needs to be called to precompute the<br>
> > > > > Lanczos kernel.<br>
> > > > ><br>
> > > > > Lanczos implementation was adapted from ufraw, Bilinear was<br>
> > > > > implemented according to Wikipedia description.<br>
> > > > ><br>
> > > > > I would be happy to hear comments on how to optimize the<br>
> > > > > computation, it's probably not optimal yet.<br>
> > > > ><br>
> > > > > regards,<br>
> > > > >         matthias<br>
> > > > ><br>
> > > > > _______________________________________________<br>
> > > > > Digikam-devel mailing list<br>
> > > > > <a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
> > > > > <a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
> > ><br>
> > > _______________________________________________<br>
> > > Digikam-devel mailing list<br>
> > > <a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
> > > <a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
> ><br>
> > _______________________________________________<br>
> > Digikam-devel mailing list<br>
> > <a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
> > <a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
><br>
> _______________________________________________<br>
> Digikam-devel mailing list<br>
> <a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
> <a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
<br>
_______________________________________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
</div></div></blockquote></div><br>