I have updated the equalizer module again with an improved calculation of equalizer coefficients. First, I replaced the subtraction of the two cosines with a product of two sines using the sum-to-product formula. This significantly reduces the roundoff error due to floating point cancellation. Second, I made the root value twice the root value in equ-xmms so that the resulting modified formulas for alpha, beta, and gamma lead to more efficient calculations.<br>
<br><div class="gmail_quote">On Sat, Mar 10, 2012 at 4:36 PM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sat, Mar 10, 2012 at 03:34:19PM -0600, Ron Wright wrote :<br>
> - band_texts[i] = new QLabel( band_frequencies[i] + "\n00.0dB" );<br>
> + band_texts[i] = new QLabel( iso_band_frequencies[i] + "\n0.0dB" );<br>
<br>
Why removing the extra 0 ?<br>
<br></blockquote><div><br>The values do not show the extra zero when initialized, so that would be why I removed the extra zero.<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> + CONNECT( THEMIM->getIM(), playingStatusChanged( int ), this,<br>
> + playingChanged( int ) );<br>
<br>
Connecting for all status changes for some UI band update seems overkill<br>
to me.<br>
Do you really need that?<br>
<br></blockquote><div><br>Does this imply that I should not keep the labels consistent with the initialized center frequencies in the module? Or is there a better way to keep them consistent?<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Best regards,<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - <a href="tel:%2B33%20672%20704%20734" value="+33672704734">+33 672 704 734</a><br>
Sent from my Electronic Device<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>