Separate checkpoint type execution paths
Open
enhancement
- Dominant language
- Python
- Stars
- 17.9k
- Forks
- 4.5k
- Avg merge
- 4d 3h
- Merged PRs (30d)
- 272
Description
Checkpointing code is pretty unreadable because of a lot of branching in low-level functions for different checkpoint types (local, global, dist ckpt types). We should move the branching to the higher stack level, i.e. have a separate function/path for local checkpoint, global checkpoint, etc. For logic that is definitely common across paths, use helpers.
To be clear, this is not a feature request. More of a readability refactor.
Contributor guide
Assessment
This issue has not been assessed yet.