From f481d283d6541176af68a2153b6cf28d6e4d20d9 Mon Sep 17 00:00:00 2001
From: Flamefire <alex@grundis.de>
Date: Fri, 25 Apr 2014 16:12:12 +0200
Subject: [PATCH] Improve test cases

---
 PitchTracker/TestSounds.rar                   | Bin 840954 -> 840957 bytes
 .../Record/PitchTracker/CPitchTrackerTest.cs  |  23 +++++++++++++-----
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/PitchTracker/TestSounds.rar b/PitchTracker/TestSounds.rar
index 7dd70d57103cc09683f6ca47483abd7539590d5e..3e956da1c6095c9fc0ef4c508bb94293a22fe292 100644
GIT binary patch
delta 406
zcmeyh()jO6;|U^0Cb=aCCKxf)F)}c4F)=VOY4<-)no>E_Mb?;?L4kpRq1ZVmv9vfW
zKUuG&qJ-gtAD`w#x43#{cCY$|LRVI9(=hM0HT_fn{!vYE{>CH~%c}2m(0jq@hDOdU
zD!W*JSbbQNp>^Wtvt`CoTwFr2JTWpRht~WPEm0Act*>inkU5m&=r~j9!t-acCJudN
zCtZ_RBO~kTTIIcRd=}3$blCjm!m+F+%FKWE967Y+^n-}d8Ehg9;ifa{#e^ds_)JU@
za9)>fu;#Ymg$<&U)e08PbM(G&{!A?|BggRssj5_eGllqPrcGiZC&jkT3_hcv;GVa^
z(QMJyy?@(HR&lob@}FIj@LH#Gt&fe^hu>mMIr}A*4jH9g>|oZ|+-|n=*1B7F8|&(4
z+~7RAkiGfM(q9ZG#pXHlAN!mnB%oAh(J&|WN1f#iM`!-yKEHXnET24D*tmFN>W8#S
w6T_&a{~wwUGPWON1Yss1W(HywAZ7((HXvpPVh$kY1Y#~A=H7mgk;n2f0QHHdGynhq

delta 403
zcmeyn()ia(;|U^06ZV%Jm|(<E&B(yO%*4RJv^a64SW$A1i>xs(g8~BsL$PyCVrg+$
zezIOkMG3<O=~W^V-Qwz5xxMPw>{!&c=4=4#{{08f78reDG1<ix*V;7QaXKTDaE50I
z_lMGl5f?%yem?7yW+Eb@vP)uzMP_T{AFUT2YF7XLGBQ}U+-Pbxa$R`-j8#UX%-cy_
zN4Pd^`tz5|Zpw|B?lwt{=`R=dT=j5c|5@4D8ae&IhSdf<8Vu{Q4Rq8t)*qOubV8v$
z=2~Lpw&aBg+G;)zU2U4CFPuN)mlTs=-vN^^r{?82>_3yuqN6!UH`{3W3<rn4y9rIX
zuG!VU`7%R<`DV$__BePg;&arD3f+g_bXacI8M(HkoLwxy7MRYL8@x4k>u#n${|z??
zcRTU3-}L^%I7!#0O}_Wp5fz0$g^cnRyFdIXHfU;-@0<BeN~HMYQD<g%rPB}2e9B1L
ua`gX$<|B;lM;Jkv35c12m<5PgftU@5*@2h?h&h3n3y8V5A7SLNybJ(xZmZq^

diff --git a/Vocaluxe/Lib/Sound/Record/PitchTracker/CPitchTrackerTest.cs b/Vocaluxe/Lib/Sound/Record/PitchTracker/CPitchTrackerTest.cs
index dc08cec7..7ba4966b 100644
--- a/Vocaluxe/Lib/Sound/Record/PitchTracker/CPitchTrackerTest.cs
+++ b/Vocaluxe/Lib/Sound/Record/PitchTracker/CPitchTrackerTest.cs
@@ -57,6 +57,9 @@ namespace Vocaluxe.Lib.Sound.Record.PitchTracker
             CBase.Log.LogDebug(msg);
             _TestCount += _CurTestCount;
             _CurTestCount = 0;
+            //Do a reset first as we actually have an impossible situation (drop by multiple octaves)
+            byte[] data = new byte[4096 * 2];
+            _Process(data);
         }
         #endregion
 
@@ -94,7 +97,7 @@ namespace Vocaluxe.Lib.Sound.Record.PitchTracker
                 msg += " Errors=" + (_TestCount - _PassedCount[i]);
                 msg += " Passed=" + _PassedCount[i];
                 msg += " Total=" + _TestCount;
-                msg += " Speed=" + _SamplesPerSec[i] + "samples/s (=" + (_SamplesPerSec[i] / 44100) + "rec.s/s)";
+                msg += " Speed=" + _SamplesPerSec[i] / 1000 + "kSamples/s (=" + (_SamplesPerSec[i] / 44100) + "rec.s/s)";
                 CBase.Log.LogDebug(msg);
             }
         }
@@ -157,10 +160,14 @@ namespace Vocaluxe.Lib.Sound.Record.PitchTracker
             bool[] valids = new bool[_Analyzers.Count];
             for (int distort = 0; distort < 10; distort++)
             {
+                //Do a reset first as we actually have an impossible situation (drop by multiple octaves)
+                byte[] data = new byte[sampleCt * 2];
+                _Process(data);
                 for (int tone = toneFrom; tone <= toneTo; tone++)
                 {
-                    byte[] data;
                     _GetSineWave(_BaseToneFreq * Math.Pow(_HalftoneBase, tone), 44100, sampleCt, ref angle, out data);
+                    if (tone == 46 && distort == 4)
+                        data = new byte[data.Length];
                     _Distort(data, tone, distort);
 
                     for (int i = 0; i < sampleCt / batchCt; i++)
@@ -181,10 +188,14 @@ namespace Vocaluxe.Lib.Sound.Record.PitchTracker
                         }
                         if (ok)
                             continue;
-                        string msg = "Note " + _ToneToNote(tone) + " detected as ";
+                        string msg = "Note " + _ToneToNote(tone) + "(" + distort + ") at buffer " + (i + 1) + "/" + (sampleCt / batchCt) + " detected as ";
                         for (int j = 0; j < valids.Length; j++)
                             msg += _ToneToNote(_Tones[j]) + (valids[j] ? "" : "(!)") + "; ";
                         CBase.Log.LogDebug(msg);
+                        /*CWavFile w = new CWavFile();
+                        w.Create(tone + "-" + distort + ".wav", 1, 44100, 16);
+                        w.Write16BitSamples(data);
+                        w.Close();*/
                     }
                 }
             }
@@ -314,14 +325,14 @@ namespace Vocaluxe.Lib.Sound.Record.PitchTracker
 
         private static bool _IsNoteValid(int note, int time, IList<STimedNote> tones)
         {
-            const int lastNoteMaxTimeDiff = 1024 * 1000 / 44100; // old note is valid for 1024 more samples
+            const int lastNoteMaxTimeDiff = 1536 * 1000 / 44100; // old note is valid for 1536 more samples
             for (int i = 0; i < tones.Count; i++)
             {
                 if (tones[i].Time > time)
                     break;
                 if (i + 1 == tones.Count || time <= tones[i + 1].Time + lastNoteMaxTimeDiff)
                 {
-                    if (tones[i].Note == note)
+                    if (tones[i].Note == note || tones[i].Note < 0)
                         return true;
                 }
             }
@@ -371,7 +382,7 @@ namespace Vocaluxe.Lib.Sound.Record.PitchTracker
                     if (error)
                     {
                         string msg = "Note " + _ToneToNote(curNote) + " at " + time + "ms detected as ";
-                        for (int i = 0; i < 4; i++)
+                        for (int i = 0; i < _Tones.Length; i++)
                             msg += _ToneToNote(_Tones[i]) + (valids[i] ? "" : "(!)") + "; ";
                         CBase.Log.LogDebug(msg);
                     }