docs: migrate to Material for MkDocs, overhaul README and guides - #165
Open
MarcelRosier wants to merge 9 commits into
Open
docs: migrate to Material for MkDocs, overhaul README and guides#165MarcelRosier wants to merge 9 commits into
MarcelRosier wants to merge 9 commits into
Conversation
- Replace Sphinx with mkdocs-material, mkdocstrings, mkdocs-jupyter - Add snippet-based algorithm tables with single source of truth - Import tutorial notebook from BrainLesion/tutorials - Slim down README, move detailed usage to docs - Add custom logo, atlas documentation, backend guidance - Fix admonition sprawl and algorithm table access UX
- Add prominent Documentation card grid linking to quickstart, segmentation, inpainting, and missing MRI guides - Remove redundant Supported Tasks section and old Documentation bullet list - Update Docker/Singularity note to match docs wording - Add uv install commands alongside pip for consistency - Fix RTD badge URL (http -> https)
- Simplify code examples to use default algorithms - Add batch processing sections to segmentation, inpainting, and missing MRI guides - Restructure mkdocs.yml nav: group guides under Using BraTS, add Prerequisites section - Update Docker/Singularity wording for consistency - Add uv install commands throughout - Tighten guide descriptions and headings - Remove preprocessing warning from algorithms page - Update docs home page cards and citation styling
Contributor
There was a problem hiding this comment.
Pull request overview
This PR migrates the documentation stack from Sphinx/Furo to Material for MkDocs, restructures the docs into task-focused guides with reusable snippets and API reference pages, and significantly simplifies the repository README to funnel users into the new docs.
Changes:
- Replaced Sphinx-based docs with MkDocs Material, including mkdocstrings-powered API pages and mkdocs-jupyter notebook rendering.
- Reworked docs IA/navigation (Getting Started / Guides / Prerequisites / Reference / Citation) and introduced reusable algorithm table snippets.
- Overhauled README to a shorter “docs-first” entry point with quick links and a minimal example.
Reviewed changes
Copilot reviewed 43 out of 48 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Replaces long in-README task docs with a docs landing grid, simpler install/example, and shorter citation/contributing sections. |
| pyproject.toml | Swaps Sphinx doc dependencies for MkDocs Material + mkdocstrings + mkdocs-jupyter + revision-date plugin. |
| mkdocs.yml | Adds MkDocs Material configuration (theme, plugins, extensions) and new site navigation. |
| .readthedocs.yaml | Switches Read the Docs build from Sphinx to MkDocs using mkdocs.yml. |
| .pre-commit-config.yaml | Adjusts YAML hook configuration to avoid failing on mkdocs.yml tags. |
| .gitignore | Ignores MkDocs build output directory (site/). |
| docs/index.md | New MkDocs home page with quick install/example and navigation cards. |
| docs/assets/logo.svg | Adds logo used by MkDocs theme. |
| docs/getting-started/installation.md | New install page (pip + container runtime setup). |
| docs/getting-started/quickstart.md | New quickstart covering segmentation/inpainting/missing MRI with default constructors. |
| docs/guides/segmentation.md | New segmentation guide with examples and included algorithm-table snippets + batch section. |
| docs/guides/inpainting.md | New inpainting guide with example + batch section + algorithm-table snippet. |
| docs/guides/missing-mri.md | New missing-MRI guide with example + batch section + algorithm-table snippet. |
| docs/guides/preprocessing.md | New preprocessing guide describing atlas requirements and helper wrappers. |
| docs/guides/backends.md | New Docker/Singularity backend guide including GPU caveats. |
| docs/reference/algorithms.md | New reference page that assembles per-task algorithm-table snippets. |
| docs/reference/api/core.md | New mkdocstrings API page for core modules. |
| docs/reference/api/constants.md | New mkdocstrings API page for constants. |
| docs/reference/api/exceptions.md | New mkdocstrings API page for exceptions. |
| docs/reference/api/logging.md | New mkdocstrings API page for logging. |
| docs/snippets/algorithm-tables/adult-glioma-pre-post.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/adult-glioma-pre-treatment.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/africa.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/meningioma.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/meningioma-rt.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/metastases.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/pediatric.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/goat.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/inpainting.md | New reusable algorithm table snippet. |
| docs/snippets/algorithm-tables/missing-mri.md | New reusable algorithm table snippet. |
| docs/tutorials/utils.py | Adds tutorial visualization helpers for MRI modalities and outputs. |
| docs/citation.md | Adds dedicated citation page (narrative + BibTeX + CITATION.cff pointer). |
| docs/Makefile | Removes Sphinx build Makefile. |
| docs/make.bat | Removes Sphinx build batch script. |
| docs/source/conf.py | Removes Sphinx configuration. |
| docs/source/index.rst | Removes Sphinx index/toctree entry point. |
| docs/source/readme.rst | Removes Sphinx README include page. |
| docs/source/preprocess_readme.py | Removes README preprocessing script for Sphinx/MyST admonitions. |
| docs/source/core/core.rst | Removes Sphinx core toctree. |
| docs/source/core/segmentation_algorithms.rst | Removes Sphinx automodule page. |
| docs/source/core/inpainting_algorithms.rst | Removes Sphinx automodule page. |
| docs/source/core/missing_mri_algorithms.rst | Removes Sphinx automodule page. |
| docs/source/core/brats_algorithm.rst | Removes Sphinx automodule page. |
| docs/source/utils/utils.rst | Removes Sphinx utils page. |
| docs/source/utils/logging.rst | Removes Sphinx logging page. |
Suppressed comments (1)
docs/getting-started/installation.md:9
- Add a small note inside the install snippet so readers immediately see that the preprocessing extra is Python >= 3.10 only.
pip install brats
pip install brats[preprocessing]
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Add comment that brats[preprocessing] requires Python >= 3.10 - Fix BibTeX author field: et al. -> and others for proper citation rendering - Clarify Python version requirement in installation.md (3.9+ base, 3.10+ preprocessing)
MarcelRosier
force-pushed
the
migrate/mkdocs-material
branch
from
August 9, 2026 13:26
3c6990b to
03145ef
Compare
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.
Summary
Migrate docs from Sphinx/Furo to Material for MkDocs, overhaul README, and restructure guides to be more approachable for new users.
What changed
Docs migration (Sphinx → Material for MkDocs)
README overhaul
Why
Test locally
Open http://localhost:8000