search

TomasLenc / acf_tools

Utilities for working with autocorrelation to capture periodicity in a signal.
GNU General Public License v3.0
2 stars 0 forks source link

set acf normalization true by default (lag 0 = 1) #18

Closed TomasLenc closed 7 months ago

TomasLenc commented 7 months ago

We will set the normalization option to be true by default. This means that the output acf values will be equivalent to taking Pearson's correlations with the input signal x and it's time-shifted versions.

The only thing we must do to make this happen is to normalize the ACF by the value at lag 0, making the lag0 value equal to 1.

TomasLenc commented 7 months ago

I am bypassing review here cause it's a minor change and all tests run allright.