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

redo the planck likelihood without requiring Planck data #54

Open
qyx268 opened this issue Apr 20, 2021 · 1 comment
Open

redo the planck likelihood without requiring Planck data #54

qyx268 opened this issue Apr 20, 2021 · 1 comment
Assignees
Labels
API Breaking A change that breaks backwards compatibility status: requires research An issue that requires further research work

Comments

@qyx268
Copy link
Collaborator

qyx268 commented Apr 20, 2021

Currently, to use the Planck PS likelihoods, there are external data and code (e.g. clik) needed. A new python package has been published to do this. Perhaps, it's a good idea to walk away from the Planck code and to use this planck-low-py, in particular when PR #39 is still not merged...
paper: https://arxiv.org/abs/2104.05715
code: https://github.com/heatherprince/planck-low-py

@qyx268 qyx268 self-assigned this Apr 20, 2021
@qyx268 qyx268 added API Breaking A change that breaks backwards compatibility status: requires research An issue that requires further research work labels Apr 20, 2021
@steven-murray
Copy link
Member

I'm all for making things more python-native, but I'm a bit worried that that code is not yet ready for production. Is there perhaps a way we could help bring it up to production level?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Breaking A change that breaks backwards compatibility status: requires research An issue that requires further research work
Projects
None yet
Development

No branches or pull requests

2 participants