shithub: aubio

Download patch

ref: 3f27a98ed43e11b18e94150ae8ffa2f1ab9be946
parent: ca45e586a294d1e8825ef6d977f869061757fd3a
author: Paul Brossier <piem@piem.org>
date: Mon Dec 2 07:07:05 EST 2013

src/tempo/beattracking.c: improve comments, remove old ones

--- a/src/tempo/beattracking.c
+++ b/src/tempo/beattracking.c
@@ -300,11 +300,7 @@
   fvec_t *acfout = bt->acfout;
 
   if (gp) {
-    // doshiftfbank again only if context dependent model is in operation
-    //acfout = doshiftfbank(acf,gwv,timesig,laglen,acfout); 
-    //don't need acfout now, so can reuse vector
-    // gwv is, in first loop, definitely all zeros, but will have
-    // proper values when context dependent model is activated
+    // compute shift invariant comb filterbank
     fvec_zeros (acfout);
     for (i = 1; i < laglen - 1; i++) {
       for (a = 1; a <= bt->timesig; a++) {
@@ -313,12 +309,9 @@
         }
       }
     }
+    // since gp is set, gwv has been computed in previous checkstate
     fvec_weight (acfout, bt->gwv);
     gp = fvec_quadratic_peak_pos (acfout, fvec_max_elem (acfout));
-    /*
-       while(gp<32) gp =gp*2;
-       while(gp>64) gp = gp/2;
-     */
   } else {
     //still only using general model
     gp = 0;
--