Hello
I see theres also
- int octaveRatio = pow( 2, octaveDifference );
+ const double octaveRatio = pow( 2, octaveDifference );
in the patch. So the bug was probably due to lost precision when converting
floating point to int.
Alvar
On Mon, Jul 11, 2011 at 3:03 PM, Chris Cannam
<[email protected]>wrote:
> On 28 June 2011 12:00, Dr Nicholas J Bailey
> <[email protected]> wrote:
> > The attached patch fixes a silly bug in the pitch-tracker which causes it
> to
> > have a target frequency of 0Hz whenever the target note is in the octave
> below
> > the reference octave (usually the one with A440 in it).
>
> I've only just got around to looking at this (I see that Michael has
> committed it for you) and I'm intrigued -- all it does is add three
> consts, right? How does that fix anything? What am I missing?
>
>
> Chris
>
>
> ------------------------------------------------------------------------------
> All of the data generated in your IT infrastructure is seriously valuable.
> Why? It contains a definitive record of application performance, security
> threats, fraudulent activity, and more. Splunk takes this data and makes
> sense of it. IT sense. And common sense.
> http://p.sf.net/sfu/splunk-d2d-c2
> _______________________________________________
> Rosegarden-devel mailing list
> [email protected] - use the link below to unsubscribe
> https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
>
------------------------------------------------------------------------------
All of the data generated in your IT infrastructure is seriously valuable.
Why? It contains a definitive record of application performance, security
threats, fraudulent activity, and more. Splunk takes this data and makes
sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-d2d-c2
_______________________________________________
Rosegarden-devel mailing list
[email protected] - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
