<div dir="ltr">On Sun, Mar 6, 2016 at 8:06 PM,  <span dir="ltr"><<a href="mailto:spitzak@gmail.com" target="_blank">spitzak@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Bill Spitzak <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>><br>
<br>
The IMPULSE special-cases did not sample the center of the of the region. This<br>
caused it to sample the filters outside their range, and produce assymetric<br>
filters and other errors. Fixing this required changing the arguments to<br>
integral() so the correct point could be determined.<br></blockquote><div><br></div><div>I don't understand what is wrong and why this patch fixes it. Which region precisely did not have its center sampled? When IMPULSE filters are involved the width of the integral is 0 so there isn't really any "region" to sample.<br><br>Can you give a concrete example where the previous code produced asymmetric filters? Also, what "other errors" was produced? I think these examples should be added to the commit log.<br></div><div><br><br>Søren<br></div><br></div></div></div>