endolith / waveform-analysis

Functions and scripts for analyzing waveforms, primarily audio. This is currently somewhat disorganized and unfinished.
MIT License
255 stars 82 forks source link

Update frequency_estimator.py #14

Closed safgen closed 7 years ago

safgen commented 7 years ago

fftconvolve returns a 2n-1 sized array so len(corr)/2 results in a non-integer.

endolith commented 7 years ago

I've been cleaning this up actually, but haven't pushed it yet. It's corr = corr[len(corr)//2:] in my version

endolith commented 7 years ago

Fixed in https://github.com/endolith/waveform_analysis/commit/cbdd1a71030e50d2739560696fce9bbd8c33ed0d#diff-6a6aa10edcf7f50a89223f3cdc0dbaf7R79