Changeset 3f27a98

Timestamp:

Dec 2, 2013, 6:07:05 PM (11 years ago)

Author:

Paul Brossier <piem@piem.org>

Branches:

feature/autosink, feature/cnn, feature/cnn_org, feature/constantq, feature/crepe, feature/crepe_org, feature/pitchshift, feature/pydocstrings, feature/timestretch, fix/ffmpeg5, master, pitchshift, sampler, timestretch, yinfft+

Children:

c54b2c0

Parents:

ca45e58

Message:

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

File:

• src/tempo/beattracking.c (modified) (2 diffs)

src/tempo/beattracking.c

@@ -301 +301 @@
 
   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++) {
@@ -314 +310 @@
       }
     }
+    // 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