Lightning-AI / Lightning-AI/lightning-thunder

Test docs as part of CI

Open
#1,311 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Python
Stars
1.5k
Forks
121
PR merge metrics
No merged PRs in 30d

Description

🚀 Feature

Test every code snippet in the project documentation to keep it always up to date with code changes.

Motivation

Documentation can become easily obsolete, especially in fast moving projects as Thunder.

For example, the code in this section of documentation is obsolete and generates an error when executed:

Traceback (most recent call last):
  File "<redacted>/example.py", line 5, in <module>
    from thunder.tests.litgpt_model import Config
ModuleNotFoundError: No module named 'thunder.tests'

The reason is that the Config object was moved to another project. Changing the import from from thunder.tests.litgpt_model import Config to from litgpt import Config fixes the issue.

Pitch

CI should execute code snippets in documentation and catch errors as above.

Alternatives
Additional context

The issue was discussed with @tfogal et al who requested to report it here.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the documentation section linked in the issue and reproduce its outdated import error. Then identify how the project's documentation snippets are organized and where CI configuration lives. Done means the documentation code snippets are executed in CI and failures such as the missing thunder.tests module are reported.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
ci-cd, documentation
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.