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

MAINT: Documentation for power spectrum #559

Merged

Conversation

Fox-Davidson
Copy link
Collaborator

@Fox-Davidson Fox-Davidson commented Jul 1, 2022

Description

Read the Docs not working with new CAMB function deletions, need to remove reference to the function, changed to class
This will close #557

Checklist

  • Follow the Contributor Guidelines
  • Write unit tests
  • Write documentation strings
  • Assign someone from your working team to review this pull request
  • Assign someone from the infrastructure team to review this pull request

@Fox-Davidson Fox-Davidson added bug Something isn't working documentation Improvements or additions to documentation module: power spectrum ps-hack labels Jul 1, 2022
@Fox-Davidson Fox-Davidson self-assigned this Jul 1, 2022
@@ -9,6 +9,10 @@


class CAMB(TabulatedPowerSpectrum):
r'''CAMB linear matter power spectrum.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
r'''CAMB linear matter power spectrum.
'''CAMB linear matter power spectrum.

Copy link
Member

@Lucia-Fonseca Lucia-Fonseca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Lucia-Fonseca Lucia-Fonseca merged commit cfe5935 into skypyproject:module/power_spectrum Jul 1, 2022
@Fox-Davidson Fox-Davidson deleted the camb_doc branch August 3, 2022 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation module: power spectrum ps-hack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants