Build comprehensive Read the Docs documentation - #37
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR builds out a comprehensive Read the Docs / Sphinx documentation set for atavide_lite, adding a full navigation structure and a multi-page user + contributor guide, along with the RTD build configuration and pinned doc build dependencies.
Changes:
- Add Read the Docs configuration plus Sphinx config (
docs/conf.py) and pinned documentation requirements. - Introduce a multi-page documentation set covering onboarding, configuration, pipeline stages, outputs, troubleshooting, utilities, methods, cluster porting, and contributing.
- Align docs narrative with the project’s stated design rationale (stage isolation, HPC portability, and explicit operational contracts).
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.readthedocs.yaml |
Configures RTD build OS/Python and points RTD at the Sphinx config + requirements. |
docs/conf.py |
Defines Sphinx project configuration (theme, MyST, nitpicky mode, static path). |
docs/requirements.txt |
Pins Sphinx/MyST/theme dependencies for doc builds. |
docs/index.md |
Adds landing page and toctree navigation structure for the doc set. |
docs/overview.md |
Provides high-level motivation/scope for the pipeline and its usage model. |
docs/design.md |
Documents design rationale and trade-offs (explicit stages, restartability, portability). |
docs/installation.md |
Step-by-step installation and environment/database prerequisites. |
docs/profiles.md |
Guides users in selecting the appropriate cluster/profile by read type and scheduler model. |
docs/quickstart.md |
Walkthrough for a first small run including job submission/dependencies. |
docs/configuration.md |
Documents the analysis directory contract and DEFINITIONS.sh variables. |
docs/pipeline.md |
Explains pipeline stages, dependencies, and validation expectations. |
docs/outputs.md |
Describes stable output concepts, logs, and reproducibility records. |
docs/troubleshooting.md |
Adds a practical diagnostic checklist for common HPC/pipeline failures. |
docs/utilities.md |
Indexes repo utilities/components and explains when/how to use them. |
docs/methods.md |
Updates/expands scientific methods and reproducible reporting guidance. |
docs/contributing.md |
Adds contribution workflow and guidance (including safe use of agentic AI). |
docs/cluster-support.md |
Adds a structured cluster-porting guide/checklist and validation expectations. |
docs/_static/.gitkeep |
Ensures _static/ exists for Sphinx static assets. |
Suppressed comments (1)
.readthedocs.yaml:21
- The file ends with whitespace-only lines. Trailing whitespace can fail whitespace checks and makes diffs noisier; remove the blank/whitespace-only lines at the end of the config.
install:
- requirements: docs/requirements.txt
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Why
The branch previously contained a Read the Docs configuration pointing to a missing
docs/conf.pyand only one documentation page. New users and cluster contributors did not have a coherent route from installation through a complete run, nor an explanation of the design trade-offs and profile-porting requirements.User impact
Users now have task-oriented onboarding and reference documentation for paired and single/long-read workflows. Contributors have explicit validation, security, scheduler-policy, and pull-request guidance. The existing
README.mdandjoss:paper/paper.mdwere used as read-only sources and were not modified.Validation
sphinx-build -W --keep-going -b html docs /tmp/atavide-rtd-htmlsucceeded with Sphinx 7.4.7/MyST 3.0.1 (local Python-compatible validation of the Sphinx 8/MyST 4 deployment configuration).readthedocs.yamlparsed and its Sphinx/requirements paths were verifiedpython -m unittest discover -s tests -vpassed all 3 testsgit diff --checkpassedREADME.md,paper/paper.md, and unrelated data