-
Notifications
You must be signed in to change notification settings - Fork 26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Autocorrelation #70
base: master
Are you sure you want to change the base?
Autocorrelation #70
Conversation
replaced the autocorrelation calculation with a proper FFT based one
Hello @wdpozzo! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
The tests are still failing for me. Especially the HalfGaussian but also others some of the time. |
mmh... I am not sure why, to be honest. Will try and rewrite things again. I am confused by the fact that the ACL comes out so small (the ACF looks sane, though) and that the logZ calculation fails. Did you spot anything wrong? |
it seems to be working now. slicing a deque does not work as we thought. also, i extended the nmcmc to 5 times the ACL and all tests on my machine pass. added a warning if nmcmc > maxmcmc