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

Add CosmoPower P(k,z) #85

Closed
wants to merge 4 commits into from
Closed

Add CosmoPower P(k,z) #85

wants to merge 4 commits into from

Conversation

itrharrison
Copy link
Collaborator

Merging will close #84

Some things to think about so far:

  • Will build this out because I want to use it in a Likelihood (CrossCorrelationLikelihood) which needs a Pk (delta_tot, delta_tot) grid. It would be better to use the full machinery inhereted from BoltzmannBase.
  • CosmoPower networks do not contain metadata about valid parameter ranges. It would be good to think about a way of packaging them with it and then using them here to prevent sampling outside those ranges.
  • I don't know how to manage that different params are supported for Pk than tt, te, ee.

@itrharrison itrharrison added enhancement New feature or request theory Related to cobaya Theories labels Nov 4, 2022
@itrharrison itrharrison self-assigned this Nov 4, 2022
@itrharrison itrharrison linked an issue Nov 4, 2022 that may be closed by this pull request
@itrharrison
Copy link
Collaborator Author

Superceded by #88

@itrharrison itrharrison deleted the dev-84-cosmopower-pkz branch March 3, 2023 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request theory Related to cobaya Theories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add CosmoPower P(k, z) emulation
1 participant