search

AlecThomson / structurefunction

MIT License
6 stars 3 forks source link

Add autocorrelation function #5

Open ErikOsinga opened 1 year ago

ErikOsinga commented 1 year ago

Hey Alec,

Very elegant code that's working very well! Given that the computation of pairs and the bookkeeping etc is already there, it would be nice to also add the computation of the autocorrelation function?

Kindly, Erik

AlecThomson commented 1 year ago

Hey @ErikOsinga!

Thanks for the suggestion! I'm curious how the autocorrelation is defined in the structure function / source pair context. What would be the functional form of the ACF in this case?

Also, if you already have it implemented somewhere I'm always happy to receive pull requests ;)