-
Notifications
You must be signed in to change notification settings - Fork 22
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
add cuts in quasar redshift #977
Conversation
py/picca/tests/test_3_cor.py
Outdated
@@ -553,6 +553,477 @@ def test_xwick(self): | |||
|
|||
return | |||
|
|||
def test_cf_angl_2(self): |
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.
rename this functions with _zcuts
instead
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.
we should rename the test files too
It does not make any sense, I cannot even reproduce the error in my computer. We did not change any line in the code rather than the names of functions and files. How is it possible that picca_xcf.py is failing now? |
Just renaming the test makes it work. I have no idea how that is possible. |
Thank you very much Ignasi for finding the issue. |
This allows for limiting the quasar redshift at the correlation level.
In the context of EDR, we decided to perform this instead of doing it at the delta extraction level.