halotools icon indicating copy to clipboard operation
halotools copied to clipboard

autocorrelation pair count normalization

Open aphearin opened this issue 5 years ago • 0 comments

As pointed out by @Christopher-Bradshaw in #962 (and also https://github.com/manodeep/Corrfunc/pull/201), the angular_tpcf function appears to have an incorrect factor of Nr x Nr instead of Nr x (Nr-1). This has a negligible impact on all practical cases of interest, but the error is needless and should still be fixed. The main question concerns whether other mock_observables functions are impacted as well, or whether this is isolated to angular_tpcf.

aphearin avatar Jan 07 '20 20:01 aphearin