Skip to content

Major overhaul of documentation#347

Merged
teutoburg merged 8 commits into
dev_masterfrom
docs/rtd-overhaul
May 18, 2026
Merged

Major overhaul of documentation#347
teutoburg merged 8 commits into
dev_masterfrom
docs/rtd-overhaul

Conversation

@astronomyk
Copy link
Copy Markdown
Contributor

@astronomyk astronomyk commented May 15, 2026

Summary

Brings the IRDB documentation up to the standard of SimCADO's docs, addressing feedback from the MICADO science team ahead of their first serious ScopeSim session. Follows the canonical A* Vienna docs template (scopesim-targets) and the approach outlined by @teutoburg.

  • sphinx-book-theme + myst-nb replace the default alabaster theme and nbsphinx. Config rewritten from scratch based on the scopesim-targets baseline — drops ~200 lines of commented-out dead code.
  • Shared docs/ScopeSim_guide.md factors out the boilerplate that was copy-pasted across all three instrument READMEs (prerequisites, pip install, working directory setup, running notebooks, references, contact). Each README now {include}s it — one place to maintain.
  • MICADO / METIS / MOSAIC readme.rstreadme.md — converted to MyST markdown, instrument-specific content kept, shared content included. MOSAIC: fixes stale reference to a non-existent Introduction_to_Scopesim_for_MOSAIC.ipynb.
  • index.rstindex.md — replaces the plain RST table with a sphinx_design card gallery (logo + description + link for each documented instrument).

Notebooks

Notebooks remain as .ipynb — no conversion in this PR, per the agreed scope.

Test plan

  • ReadTheDocs preview build succeeds on this branch
  • Landing page shows card grid with MICADO / METIS / MOSAIC logos and links
  • Each instrument README shows the full guide (prerequisites → contact) via the shared include
  • MICADO validation table renders with 6 rows, coloured status, clickable PDF links
  • MOSAIC README no longer references the non-existent intro notebook

Open issues

To be tackled separately, but just so it's noted somewhere:

  • The markdownlinkchecker now tripped on all relative links in the .md files, even though they work just fine in the built documentation. Disabled for now to avoid failures all the time. Should be investigated how to deal with relative links.
  • The one MOSAIC notebook does not show any images. Not sure why.

@astronomyk astronomyk requested review from a team and oczoske as code owners May 15, 2026 21:34
@teutoburg teutoburg added the documentation Improvements or additions to documentation label May 15, 2026
@teutoburg teutoburg changed the title docs: overhaul RTD docs — sphinx-book-theme, shared guide, card landing page Major overhaul of documentation May 15, 2026
Replace nbsphinx + default theme with myst-nb + sphinx-book-theme,
following the scopesim-targets canonical A* Vienna docs template.
Add sphinx-design (for card gallery) and sphinx-copybutton. Strip
commented-out dead code and LaTeX/man/Texinfo output boilerplate.
Convert MICADO, METIS, and MOSAIC readme.rst files to readme.md.
Factor out common installation boilerplate (prerequisites, pip install,
working directory setup, notebook workflow, references, contacts) into
docs/ScopeSim_guide.md, which each README includes via MyST {include}.

MOSAIC: fix stale reference to non-existent
Introduction_to_Scopesim_for_MOSAIC.ipynb (replaced with MOSAIC_demo.ipynb).
Replace the plain RST table index with a sphinx_design card grid showing
the three documented ELT instruments (MICADO, METIS, MOSAIC), each with
its logo, a one-line description, and a direct link to its getting-started
guide. Retain a full instrument packages table below the cards.
@teutoburg teutoburg moved this from 🏗 In progress to 👀 Awaiting Review in ScopeSim-development May 16, 2026
Copy link
Copy Markdown
Contributor

@teutoburg teutoburg left a comment

Choose a reason for hiding this comment

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

Yeah, as discussed on Slack, this is now fixed, so go. Ignore the one comment.

Comment thread conf.py
Copy link
Copy Markdown
Contributor

@oczoske oczoske left a comment

Choose a reason for hiding this comment

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

Try to get a complete list of METIS notebooks as indicated.
There's a question about MOSAIC notebook not showing any images - has that been resolved? There are a few static images which will have to be in the right place for RTD to pick them up.

Comment thread METIS/docs/README.md
Comment thread METIS/docs/README.md
@github-project-automation github-project-automation Bot moved this from 👀 Awaiting Review to 🏗 In progress in ScopeSim-development May 16, 2026
@teutoburg teutoburg force-pushed the docs/rtd-overhaul branch from 3938f4a to c5fd670 Compare May 16, 2026 20:16
@teutoburg
Copy link
Copy Markdown
Contributor

I now replaced the tables in METIS/README with toctrees. In the case of the demo notebooks dynamic, the other ones explicit, to keep the intro / science case split. There's a point to be made if this split should be reflected in the folder structure of the notebooks, but that's a discussion for later.

The toctree is currently a bit broken because some notebooks violate that rule that each notebook must have exactly one top-level heading (#). But I' prefer to fix that in a separate PR (very soon).

Is this acceptable to you @oczoske and @astronomyk or do you feel strong attachment to the previous tables?

@teutoburg teutoburg merged commit 84f8b77 into dev_master May 18, 2026
13 checks passed
@github-project-automation github-project-automation Bot moved this from 🏗 In progress to ✅ Done in ScopeSim-development May 18, 2026
@teutoburg teutoburg deleted the docs/rtd-overhaul branch May 18, 2026 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants