Skip to content

Issue/687/cosmo pub calls [version:1.16.8]#694

Merged
m-aguena merged 6 commits intomainfrom
issue/687/cosmo_pub_calls
Dec 3, 2025
Merged

Issue/687/cosmo pub calls [version:1.16.8]#694
m-aguena merged 6 commits intomainfrom
issue/687/cosmo_pub_calls

Conversation

@m-aguena
Copy link
Member

@m-aguena m-aguena commented Dec 1, 2025

Description

Makes the cosmology have the same validate_input as the theory object.

Main changes

  • Make validate_input an attibute of Modeling

Checklist

Besides passing all CI checks and coverage is at 100%, make sure you also checked the following items
(check details in CONTRIBUTING).

For developers

  • Notebooks: notebooks related to this PR have been updated and all notebooks can run correctly.
  • Build the documentation: All documentation builds correctly.

For reviewers

  • Notebooks: notebooks related to this PR have been updated and all notebooks can run correctly.
  • Build the documentation: All documentation builds correctly.

For developers (part 2)

After the PR has been approved by two reviewers:

  • Update the code version in clmm/__ini__.py.
  • Keep only relevant points in the squash and merge commit message.

@coveralls
Copy link

coveralls commented Dec 1, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling 5225d29 on issue/687/cosmo_pub_calls
into 2b18cee on main.

@hsinfan1996 hsinfan1996 marked this pull request as ready for review December 1, 2025 09:24
Copy link
Collaborator

@eduardojsbarroso eduardojsbarroso left a comment

Choose a reason for hiding this comment

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

It seems everything is good, besides the documentation line

@github-actions github-actions bot changed the title Issue/687/cosmo pub calls Issue/687/cosmo pub calls [version:1.16.8] Dec 3, 2025
@m-aguena m-aguena self-assigned this Dec 3, 2025
@m-aguena m-aguena merged commit 86faa10 into main Dec 3, 2025
5 checks passed
@m-aguena m-aguena deleted the issue/687/cosmo_pub_calls branch December 12, 2025 16:01
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

Successfully merging this pull request may close these issues.

3 participants