Changeset b4eadf0

Timestamp:

Feb 1, 2017, 10:53:54 PM (7 years ago)

Author:

Paul Brossier <piem@piem.org>

Branches:

yinfft+

Children:

a7b14a6

Parents:

8f0db97

git-author:

Eduard Müller <mueller.eduard@googlemail.com> (01/13/17 16:33:37)

git-committer:

Paul Brossier <piem@piem.org> (02/01/17 22:53:54)

Message:

new yinfft f0 "candidate selection" strategy

Signed-off-by: Paul Brossier <​piem@piem.org>

File:

• src/pitch/pitchyinfft.c (modified) (5 diffs)

src/pitch/pitchyinfft.c

@@ -38 +38 @@
   smpl_t tol;         /**< Yin tolerance */
   smpl_t confidence;  /**< confidence */
-  uint_t short_period; /** shortest period under which to check for octave error */
+  uint_t samplerate;  /**< samplerate we got initialized with */
 };
 
@@ -70 +70 @@
   p->win = new_aubio_window ("hanningz", bufsize);
   p->weight = new_fvec (bufsize / 2 + 1);
+  p->samplerate = samplerate;
   for (i = 0; i < p->weight->length; i++) {
     freq = (smpl_t) i / (smpl_t) bufsize *(smpl_t) samplerate;
@@ -94 +95 @@
     //p->weight->data[i] = SQRT(DB2LIN(p->weight->data[i]));
   }
-  // check for octave errors above 1300 Hz
-  p->short_period = (uint_t)ROUND(samplerate / 1300.);
+
+  // disable weighting
+  fvec_set_all (p->weight, 1.0);
+
   return p;
 
@@ -113 +116 @@
   fvec_t *yin = p->yinfft;
   smpl_t tmp = 0., sum = 0.;
+
   // window the input
   fvec_weighted_copy(input, p->win, p->winput);
@@ -146 +150 @@
     }
   }
-  // find best candidates
-  tau = fvec_min_elem (yin);
-  if (yin->data[tau] < p->tol) {
-    // no interpolation, directly return the period as an integer
-    //output->data[0] = tau;
-    //return;
-
-    // 3 point quadratic interpolation
-    //return fvec_quadratic_peak_pos (yin,tau,1);
-    /* additional check for (unlikely) octave doubling in higher frequencies */
-    if (tau > p->short_period) {
-      output->data[0] = fvec_quadratic_peak_pos (yin, tau);
-    } else {
-      /* should compare the minimum value of each interpolated peaks */
-      halfperiod = FLOOR (tau / 2 + .5);
-      if (yin->data[halfperiod] < p->tol)
-        output->data[0] = fvec_quadratic_peak_pos (yin, halfperiod);
-      else
-        output->data[0] = fvec_quadratic_peak_pos (yin, tau);
-    }
-  } else {
+
+  // calc min available confidence first
+  tmp = fvec_min(yin);
+  if (tmp > p->tol) {
+    // give up - got no confident candidate at all 
     output->data[0] = 0.;
-  }
+    return;
+  }
+
+  // choose lowest confident candidate first, to avoid choosing harmonics
+  tau = 0;
+  for (l = 1; l < yin->length; l++) {
+    // is this candidate "roughly" as good as the lowest one?
+    if (ABS (yin->data[l] - tmp) < 0.1) {
+      tau = l;
+      break;
+    }
+  }
+  // find local min around current pick to sharpen the results
+  const uint_t LOCAL_NOTE_SEEK_RANGE = 1;
+  const smpl_t note = aubio_bintomidi (tau, p->samplerate, p->fftout->length);
+  const uint_t startbin = MAX (0, (uint_t)aubio_miditobin (note - LOCAL_NOTE_SEEK_RANGE,
+    p->samplerate, p->fftout->length));
+  const uint_t endbin = MIN (yin->length, (uint_t)(aubio_miditobin (note + LOCAL_NOTE_SEEK_RANGE,
+    p->samplerate, p->fftout->length) + 0.5));
+  tmp = yin->data[tau];
+  for (l = startbin; l < endbin; l++) {
+    if (yin->data[l] < tmp ) {
+      tmp = yin->data[l];
+      tau = l;
+    }
+  }
+  output->data[0] = fvec_quadratic_peak_pos(yin, tau);
 }