Skip to content
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

Very small bias in RR estimate for auto-correlation functions #1012

Open
johannesulf opened this issue Mar 25, 2021 · 0 comments
Open

Very small bias in RR estimate for auto-correlation functions #1012

johannesulf opened this issue Mar 25, 2021 · 0 comments
Assignees
Labels
Milestone

Comments

@johannesulf
Copy link
Contributor

The following is a very minor point. There's a nice notebook by Lehman Garrison showing that the naive analytic RR estimator, RR = N dV N / V, has a slight bias. Instead, it should be RR = (N - 1) dV N / V. Halotools seems to be using the naive RR estimator with the slight bias.

https://nbviewer.jupyter.org/gist/lgarrison/1efabe4430429996733a9d29397423d2

Should be trivial to correct in halotools. It's a very minor point because N is generally very, very large.

@aphearin aphearin self-assigned this Mar 25, 2021
@aphearin aphearin added the bug label Mar 25, 2021
@aphearin aphearin added this to the next-release milestone Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants