astropy / astropy/halotools

Very small bias in RR estimate for auto-correlation functions

Open
#1,012 0 comments 1 reaction 1 assignee Claimed by @aphearin View on GitHub
bug
Dominant language
Python
Stars
120
Forks
70
PR merge metrics
No merged PRs in 30d

Description

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.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.