astropy / astropy/halotools

autocorrelation pair count normalization

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

Description

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`.

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.