Skip to content
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

Include P1D measurements from DESI EDR #36

Open
andreufont opened this issue Jul 14, 2023 · 6 comments
Open

Include P1D measurements from DESI EDR #36

andreufont opened this issue Jul 14, 2023 · 6 comments
Assignees

Comments

@andreufont
Copy link
Contributor

From Karacayli et al. (2023) and from Ravoux et al. (2023)

The sysetmatics matrix might need some work, since it is not diagonal

@andreufont
Copy link
Contributor Author

Ravoux et al. (2023) was added in #44 , but we still need to include the results from Karacayli et al. (2023)

@andreufont
Copy link
Contributor Author

@p-slash - Would you be able to add to cup1d the files needed to read and use your EDR P1D measurement?

@andreufont
Copy link
Contributor Author

@jchavesmontero

@p-slash
Copy link
Contributor

p-slash commented Nov 27, 2023

I started to implement this. There is a catch though, not every redshift bin has the same k values. I will first push my fix for high resolution data

@p-slash
Copy link
Contributor

p-slash commented Nov 27, 2023

I see this was already raised in issue #14

@p-slash
Copy link
Contributor

p-slash commented Nov 27, 2023

My solution to this was to have a large covariance for those values and they are set to zero, so masking is possible somewhere down the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants