Repository navigation
Autodoc - #39
Autodoc#39jeanjacquemier wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
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
Open (13)
Keep checked-in documentation out of make clean · New Restrict production publishing to the main branch · New Use O instead of zero for Make options · New Enable autodoc extensions and add API documentation · New Remove unsupported cosmo configuration from CLPCovariance · New Document the correct default HMF model · New Replace copied CLPCovariance options with Firecrown options · New Use descriptive alt text for the logo · New Correct the spelling of publicly · New Reference merge_data_covariance for mixed covariance blocks · New Remove Markdown fences from the reStructuredText literal block · New Remove the literal Markdown fence from the test command · New Fix line wrapping and opening article in overview · New
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.
| This is handled by: | ||
| extract_data_covariance() |
There was a problem hiding this comment.
7f1ba7f to
e83b2b5
Compare
e83b2b5 to
00ea99c
Compare
00ea99c to
bcec083
Compare
Coverage Report for CI Build 37614240398Warning No base build found for commit Coverage: 42.255%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
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 ☂️ |
2c4550e to
3f1524d
Compare
3f1524d to
306b860
Compare
306b860 to
746e84b
Compare
7adb453 to
d9d6132
Compare
- 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
- Add sphinx copy button extension
- Add coveralls command
- Add API doc
44a28a4 to
5424be0
Compare



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