-
Notifications
You must be signed in to change notification settings - Fork 69
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
Allowing for cross-correlations in HOD power spectrum. #1224
Conversation
The code is fine, but there's a problem with conda. I'm trying to fix it, give me a few min (hopefully). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot @anicola . Just one small comment.
The checks are only not passing because coveralls is down for maintenance, but we can ignore that, so don't worry.
pyccl/tests/test_profiles.py
Outdated
p2.fourier_2pt(COSMO, 1., 1E13, 1., pgood, prof2=pgood_b) | ||
# doesn't raise for two different HOD profiles | ||
pgood_b.update_parameters(log10M0_0=10.) | ||
p2.fourier_2pt(COSMO, 1., 1E13, 1., pgood, prof2=pgood_b) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we check that this gives the same result as the standard profile2pt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for going through the code and the comments David. Should be fixed now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Allowing for cross-correlations in HOD power spectrum. (#1224)
This PR allows for cross-correlations with different HOD profiles. In the case in which the profiles are different, the code reverts to the usual Fourier-space two-point moment as implemented in Profile2pt. This holds under the assumption that the two tracers are disjoint.