<div dir="ltr"><div>> Would like to know if you think there is any chances of getting more<br>> changes accepted. I don't want to waste time if that is not going to<br>> happen. I would like to move the subsample locations so they are at<br>> integers and 1/2<br><br></div>I think this is a good idea in principle, but the reality is that I probably won't have the time to review such patches.<br><div><div><br>Søren<br><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 6, 2016 at 3:22 PM, Bill Spitzak <span dir="ltr"><<a href="mailto:spitzak@gmail.com" target="_blank">spitzak@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks that looks great, glad to see this stuff is finally in there.<br>
<br>
I'm disappointed you did not accept the changes to the normalization.<br>
I believe I can prove that the residual error will always be zero,<br>
unless the filter is so large that it would not work anyway (due to<br>
the entries being smaller than the minimum fixed value). You can try<br>
the same test I did which led me to this conclusion, try putting a<br>
print statement in when the residual error is not zero and see if you<br>
can trigger it.<br>
<br>
Would like to know if you think there is any chances of getting more<br>
changes accepted. I don't want to waste time if that is not going to<br>
happen. I would like to move the subsample locations so they are at<br>
integers and 1/2, and some other errors, plus push the rejected<br>
patches, maybe with better attempts at explaining them.<br>
<br>
In a longer term I would like to do something to support varying-size<br>
filters, and to implement a true 2-pass transform.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On Sat, Sep 3, 2016 at 12:10 PM, Søren Sandmann<br>
<<a href="mailto:soren.sandmann@gmail.com">soren.sandmann@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> I have pushed these patches to master, except (a) I didn't like the changes<br>
> to the normalization patch, so I went with the one I posted back in April<br>
> (<a href="https://lists.freedesktop.org/archives/pixman/2016-April/004553.html" rel="noreferrer" target="_blank">https://lists.freedesktop.<wbr>org/archives/pixman/2016-<wbr>April/004553.html</a>), and<br>
> (b) I didn't push the patch adding the filtering dropdown to the demo/scale<br>
> program. [1]<br>
><br>
><br>
> Søren<br>
><br>
><br>
> [1] I accidentally did push it, but reverted it afterwards.<br>
><br>
> On Wed, Aug 31, 2016 at 1:03 AM, <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>> wrote:<br>
>><br>
>> If anybody is maintaining pixman now, these patches have been reviewed<br>
>> several times and should be pushed. They are primarily written by<br>
>> Søren based on versions I wrote. They fix several defects in the<br>
>> seperable filter generation.<br>
>><br>
>> ______________________________<wbr>_________________<br>
>> Pixman mailing list<br>
>> <a href="mailto:Pixman@lists.freedesktop.org">Pixman@lists.freedesktop.org</a><br>
>> <a href="https://lists.freedesktop.org/mailman/listinfo/pixman" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/pixman</a><br>
><br>
><br>
</div></div></blockquote></div><br></div>