annotate dsp/signalconditioning/Framer.cpp @ 73:dcb555b90924

* Key detector: when returning key strengths, use the peak value of the three underlying chromagram correlations (from 36-bin chromagram) corresponding to each key, instead of the mean. Rationale: This is the same method as used when returning the key value, and it's nice to have the same results in both returned value and plot. The peak performed better than the sum with a simple test set of triads, so it seems reasonable to change the plot to match the key output rather than the other way around. * FFT: kiss_fftr returns only the non-conjugate bins, synthesise the rest rather than leaving them (perhaps dangerously) undefined. Fixes an uninitialised data error in chromagram that could cause garbage results from key detector. * Constant Q: remove precalculated values again, I reckon they're not proving such a good tradeoff.
author cannam
date Fri, 05 Jun 2009 15:12:39 +0000
parents d7116e3183f8
children e5907ae6de17
rev   line source
cannam@0 1 /* -*- c-basic-offset: 4 indent-tabs-mode: nil -*- vi:set ts=8 sts=4 sw=4: */
cannam@0 2
cannam@0 3 /*
cannam@0 4 QM DSP Library
cannam@0 5
cannam@0 6 Centre for Digital Music, Queen Mary, University of London.
cannam@0 7 This file copyright 2005-2006 Christian Landone.
cannam@0 8 All rights reserved.
cannam@0 9 */
cannam@0 10
cannam@0 11 #include "Framer.h"
cannam@0 12 #include <math.h>
cannam@0 13
cannam@0 14 //////////////////////////////////////////////////////////////////////
cannam@0 15 // Construction/Destruction
cannam@0 16 //////////////////////////////////////////////////////////////////////
cannam@0 17
cannam@0 18 Framer::Framer()
cannam@0 19 {
cannam@0 20 m_dataFrame = NULL;
cannam@0 21 m_strideFrame = NULL;
cannam@0 22 }
cannam@0 23
cannam@0 24 Framer::~Framer()
cannam@0 25 {
cannam@0 26 if( m_dataFrame != NULL )
cannam@0 27 delete [] m_dataFrame;
cannam@0 28
cannam@0 29 if( m_strideFrame != NULL )
cannam@0 30 delete [] m_strideFrame;
cannam@0 31 }
cannam@0 32
cannam@0 33 void Framer::configure( unsigned int frameLength, unsigned int hop )
cannam@0 34 {
cannam@0 35 m_frameLength = frameLength;
cannam@0 36 m_stepSize = hop;
cannam@0 37
cannam@0 38 resetCounters();
cannam@0 39
cannam@0 40 if( m_dataFrame != NULL )
cannam@0 41 {
cannam@0 42 delete [] m_dataFrame;
cannam@0 43 m_dataFrame = NULL;
cannam@0 44 }
cannam@0 45 m_dataFrame = new double[ m_frameLength ];
cannam@0 46
cannam@0 47 if( m_strideFrame != NULL )
cannam@0 48 {
cannam@0 49 delete [] m_strideFrame;
cannam@0 50 m_strideFrame = NULL;
cannam@0 51 }
cannam@0 52 m_strideFrame = new double[ m_stepSize ];
cannam@0 53 }
cannam@0 54
cannam@0 55 void Framer::getFrame(double *dst)
cannam@0 56 {
cannam@0 57
cannam@0 58 if( (m_ulSrcIndex + ( m_frameLength) ) < m_ulSampleLen )
cannam@0 59 {
cannam@0 60 for( unsigned int u = 0; u < m_frameLength; u++)
cannam@0 61 {
cannam@0 62 dst[ u ] = m_srcBuffer[ m_ulSrcIndex++ ];
cannam@0 63 }
cannam@0 64 m_ulSrcIndex -= ( m_frameLength - m_stepSize );
cannam@0 65 }
cannam@0 66 else
cannam@0 67 {
cannam@0 68 unsigned int rem = (m_ulSampleLen - m_ulSrcIndex );
cannam@0 69 unsigned int zero = m_frameLength - rem;
cannam@0 70
cannam@0 71 for( unsigned int u = 0; u < rem; u++ )
cannam@0 72 {
cannam@0 73 dst[ u ] = m_srcBuffer[ m_ulSrcIndex++ ];
cannam@0 74 }
cannam@0 75
cannam@0 76 for( unsigned int u = 0; u < zero; u++ )
cannam@0 77 {
cannam@0 78 dst[ rem + u ] = 0;
cannam@0 79 }
cannam@0 80
cannam@0 81 m_ulSrcIndex -= (( rem - m_stepSize ) );
cannam@0 82 }
cannam@0 83
cannam@0 84 m_framesRead++;
cannam@0 85 }
cannam@0 86
cannam@0 87 void Framer::resetCounters()
cannam@0 88 {
cannam@0 89 m_framesRead = 0;
cannam@0 90 m_ulSrcIndex = 0;
cannam@0 91 }
cannam@0 92
cannam@0 93 unsigned int Framer::getMaxNoFrames()
cannam@0 94 {
cannam@0 95 return m_maxFrames;
cannam@0 96 }
cannam@0 97
cannam@0 98 void Framer::setSource(double *src, unsigned int length)
cannam@0 99 {
cannam@0 100 m_srcBuffer = src;
cannam@0 101 m_ulSampleLen = length;
cannam@0 102
cannam@0 103 m_maxFrames = (unsigned int)ceil( (double)m_ulSampleLen/(double)m_stepSize ) ;
cannam@0 104 }