Lightning-AI / Lightning-AI/lightning-thunder
Test docs as part of CI
Nobody has claimed this yet.
- 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
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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