fix: widthframer no longer accumulates autocorrelation data into NaNs

hello-adam · hello-adam · commit 362f27b7f1e2 · 2020-03-11T14:52:47.000-04:00
diff --git a/src/hobbits-plugins/analyzers/WidthFramer/widthframer.cpp b/src/hobbits-plugins/analyzers/WidthFramer/widthframer.cpp
@@ -233,10 +233,13 @@ QVector<QPointF> WidthFramer::autocorrelate(QSharedPointer<const BitArray> bits)
 
     // prepare and run first FFT
     for (int i = 0; i < N; i++) {
+        m_fft_in[i][0] = 0;
+        m_fft_in[i][1] = 0;
         if (i < bits->sizeInBits()) {
             m_fft_in[i][0] = bits->at(i) ? 1 : -1;
-            m_fft_in[i][1] = 0;
         }
+        m_fft_out[i][0] = 0;
+        m_fft_out[i][1] = 0;
     }
     fftw_execute(m_fft_plan1);
 

Original file line number	Diff line number	Diff line change
`@@ -233,10 +233,13 @@ QVector<QPointF> WidthFramer::autocorrelate(QSharedPointer<const BitArray> bits)`
`233`	`233`
`234`	`234`	`// prepare and run first FFT`
`235`	`235`	`for (int i = 0; i < N; i++) {`
	`236`	`+ m_fft_in[i][0] = 0;`
	`237`	`+ m_fft_in[i][1] = 0;`
`236`	`238`	`if (i < bits->sizeInBits()) {`
`237`	`239`	`m_fft_in[i][0] = bits->at(i) ? 1 : -1;`
`238`		`- m_fft_in[i][1] = 0;`
`239`	`240`	`}`
	`241`	`+ m_fft_out[i][0] = 0;`
	`242`	`+ m_fft_out[i][1] = 0;`
`240`	`243`	`}`
`241`	`244`	`fftw_execute(m_fft_plan1);`
`242`	`245`