Skip to content

Autodoc - #39

Merged
eduardojsbarroso merged 1 commit into
mainfrom
autodoc
Oct 13, 2026
Merged

eduardojsbarroso merged 1 commit into
mainfrom
autodoc

Conversation

@jeanjacquemier

@jeanjacquemier jeanjacquemier commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Add the documentation with Sphinx and the corresponding mechanisms
- Add a logo to the README
- Add clp_covariance and clp_firecrown confirguration options
- Add an overview
- Add an installation doc:
- Add doc generation with CI
- Use github Pages withgithub actions to publish the docs
- add deployment on autodoc branch before merging
- Add some github badges
- Add LSST DESC software dependencie to doc

Comment thread .github/workflows/docs.yml Resolved

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The documentation contains invalid configuration guidance, lacks functional autodoc, and includes build and deployment defects.

Review effort: Balanced
Findings: 1 High severity · 6 Medium severity · 6 Low severity

Open (13)
What changed in this PR

Introduces a Sphinx documentation site, configuration guides, and automated validation/deployment.

Changes:

  • Adds overview, installation, covariance, and Firecrown documentation.
  • Adds Sphinx configuration and dependencies.
  • Adds documentation CI, GitHub Pages deployment, and README branding.
File Description
README.md Adds the CLPipe logo.
docs/​source/​overview.rst Adds the project overview.
docs/​source/​installation.rst Adds installation and testing instructions.
docs/​source/​clp_firecrown.rst Documents Firecrown configuration.
docs/​source/​clp_covariance.rst Documents covariance configuration.
docs/​requirements.txt Declares documentation dependencies.
docs/​Makefile Adds Sphinx build targets.
docs/​index.rst Defines documentation navigation.
docs/​conf.py Configures Sphinx and its theme.
.gitignore Ignores generated documentation.
.github/​workflows/​docs.yml Builds and deploys documentation.
.github/​workflows/​ci.yml Adds documentation validation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/Makefile Resolved
Comment thread .github/workflows/docs.yml Resolved
Comment thread docs/Makefile Resolved
Comment thread docs/conf.py Resolved
Comment thread docs/source/clp_covariance.rst Resolved
Comment thread docs/index.rst Resolved
Comment thread docs/source/clp_covariance.rst Outdated
Comment on lines +161 to +162
This is handled by:
extract_data_covariance()

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Comment thread docs/source/installation.rst Resolved
Comment thread docs/source/installation.rst Resolved
Comment thread docs/source/overview.rst Resolved
@coveralls

coveralls commented Oct 7, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 37614240398

Warning

No base build found for commit 3a6cd70 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 42.255%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 1020
Covered Lines: 431
Line Coverage: 42.25%
Coverage Strength: 0.85 hits per line

💛 - Coveralls

@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Comment thread .github/workflows/docs.yml Resolved
    - Add a logo to the README.
    - Add clp_covariance and clp_firecrown confirguration options.
    - Add an overview.
    - Add an installation doc.
    - Add doc generation (to test it)  with the CI at every Pull Request.
    - Use github Pages withgithub actions to publish the docs.
    - Trigger the documentation deployment when a branch is merged into the main branch.
    - Add some github badges.
    - Add LSST DESC software dependencie to doc.
    - Add sphinx copy button extension.
    - Add coveralls command.
    - Add API doc.
@eduardojsbarroso
eduardojsbarroso merged commit 93899ba into main Oct 13, 2026
6 checks passed
@eduardojsbarroso
eduardojsbarroso deleted the autodoc branch October 13, 2026 13:06
eduardojsbarroso added a commit that referenced this pull request Nov 3, 2026
Possible now that clp_firecrown imports ExactBinnedClusterRecipe when use_grid is false (#39).
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.

5 participants