Fwd: Convolution

Boudewijn Rempt boud at valdyas.org
Wed May 25 22:05:16 CEST 2005


On Wednesday 25 May 2005 22:03, Bart Coppens wrote:

> So I had almost the same fix for it. Only, I divide by kernel[0].factor
> instead of kernelSum, and I still add the offset to it. Also, I kept the
> int sums[depth], instead of float. I don't see the use in making it float,
> since we only add QUANTUM*int to it. (I did set it with memset(&sums, 0,
> depth * sizeof(int));, though, instead of just depth).

Sounds a lot better -- I guess I'd prefer your code, so if you could check it 
in (or mail it -- I'm still futzing around a bit), I'd be very happy.
-- 
Boudewijn Rempt 
http://www.valdyas.org/fading/index.cgi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kimageshop/attachments/20050525/4f5453c5/attachment-0001.pgp


More information about the kimageshop mailing list