Commit 3a42c02
committed
fix(ci): unblock pre-commit after cascade of latent regressions
Every hook now passes on `pre-commit run --all-files` (exit 0 after one
auto-fix pass + one verification pass — the standard pre-commit contract).
Unblocks book-validate-dev, which has been red on various hooks since the
mlsysim.core import failure finally cleared.
Fixes applied (source-traced, not suppressed):
1. codespell: 'OT' in mlperf-edu/reference/cloud/micro_lstm.py is the
column name for Oil Temperature in the ETTh1 dataset (Zhou et al.,
AAAI 2021), not a typo for 'to/of/or/not/it'. Added 'ot' to
.codespell-ignore-words.txt (case-insensitive, covers OT).
2. bib-lint §5 bibliography hygiene: 12 entries in
mlperf-edu/paper/refs.bib missing required publisher/journal per the
canonical mapping in book-prose-merged.md §5. Added canonical
publishers (MLSys → mlsys.org, ICLR → OpenReview.net, CVPR → IEEE,
NAACL → ACL, etc.); promoted krizhevsky2009cifar from @Article to
@TechReport with institution = University of Toronto. banbury2021mlperf
uses Curran Associates Inc. (pre-2022 NeurIPS rule); flagged
banbury2024wakevision for author review since the booktitle says CVPR
but web verification suggests it is still an arXiv preprint.
3. Over-eager 'vs.' style sweep corrupted anchor IDs: 77 instances of
-vs.- inside {#sec-...}, {#tbl-...}, {#fig-...} definitions and their
@-references across 27 QMDs. Anchor IDs must be literal strings without
periods per the repo's own section-ID naming rule; stripped the period
from all anchor tokens while preserving 'vs.' in visible prose.
4. 4 broken SVG filename references from the same sweep (pam4-vs.-nrz,
traditional-vs.-ml-fleet, tco-build-vs.-buy, centralized-vs.-decentralized)
— filenames on disk use vs- (no period) so refs restored to match.
5. Malformed XML declaration in bathtub-curve.svg:
'<?xml version="utf-8"?>' → '<?xml version="1.0" encoding="utf-8"?>'.
6. 21 quad-asterisks (****term****) in training.qmd collapsed to **term**.
7. bibtex-tidy auto-reformatted mlperf-edu/paper/refs.bib (alphabetical
order + consistent indentation + wrapped author lists) and pipe-table
prettifier realigned columns across ~20 QMDs. These are all cosmetic
formatter output — no content changes.
Verified: pre-commit run --all-files run #1 modified files (exit 1),
run #2 exit 0 with 61 Passed / 0 Failed.1 parent 853dc90 commit 3a42c02
45 files changed
Lines changed: 868 additions & 797 deletions
File tree
- book/quarto/contents
- vol1
- backmatter
- benchmarking
- conclusion
- data_engineering
- data_selection
- frameworks
- frontmatter
- hw_acceleration
- introduction
- ml_ops
- ml_systems
- ml_workflow
- model_serving
- nn_architectures
- nn_computation
- optimizations
- parts
- responsible_engr
- training
- vol2
- backmatter
- collective_communication
- compute_infrastructure
- distributed_training
- edge_intelligence
- fault_tolerance
- images/svg
- fleet_orchestration
- frontmatter
- inference
- introduction
- network_fabrics
- ops_scale
- responsible_ai
- robust_ai
- security_privacy
- sustainable_ai
- mlperf-edu/paper
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
72 | 72 | | |
73 | 73 | | |
74 | 74 | | |
| 75 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
227 | 227 | | |
228 | 228 | | |
229 | 229 | | |
230 | | - | |
| 230 | + | |
231 | 231 | | |
232 | 232 | | |
233 | 233 | | |
| |||
319 | 319 | | |
320 | 320 | | |
321 | 321 | | |
322 | | - | |
| 322 | + | |
323 | 323 | | |
324 | 324 | | |
325 | 325 | | |
| |||
347 | 347 | | |
348 | 348 | | |
349 | 349 | | |
350 | | - | |
351 | | - | |
352 | | - | |
353 | | - | |
354 | | - | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
| 354 | + | |
355 | 355 | | |
356 | 356 | | |
357 | 357 | | |
358 | 358 | | |
359 | 359 | | |
360 | | - | |
| 360 | + | |
361 | 361 | | |
362 | 362 | | |
363 | 363 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
32 | | - | |
| 32 | + | |
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| |||
107 | 107 | | |
108 | 108 | | |
109 | 109 | | |
110 | | - | |
| 110 | + | |
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
| |||
163 | 163 | | |
164 | 164 | | |
165 | 165 | | |
166 | | - | |
167 | | - | |
| 166 | + | |
| 167 | + | |
168 | 168 | | |
169 | | - | |
| 169 | + | |
170 | 170 | | |
171 | 171 | | |
172 | 172 | | |
173 | 173 | | |
174 | | - | |
| 174 | + | |
175 | 175 | | |
176 | 176 | | |
177 | 177 | | |
178 | 178 | | |
179 | 179 | | |
180 | | - | |
| 180 | + | |
181 | 181 | | |
182 | | - | |
| 182 | + | |
183 | 183 | | |
184 | 184 | | |
185 | 185 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
221 | 221 | | |
222 | 222 | | |
223 | 223 | | |
224 | | - | |
225 | | - | |
226 | | - | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
227 | 227 | | |
228 | 228 | | |
229 | 229 | | |
| |||
240 | 240 | | |
241 | 241 | | |
242 | 242 | | |
243 | | - | |
| 243 | + | |
244 | 244 | | |
245 | 245 | | |
246 | 246 | | |
| |||
924 | 924 | | |
925 | 925 | | |
926 | 926 | | |
927 | | - | |
928 | | - | |
929 | | - | |
| 927 | + | |
| 928 | + | |
| 929 | + | |
930 | 930 | | |
931 | 931 | | |
932 | 932 | | |
| |||
942 | 942 | | |
943 | 943 | | |
944 | 944 | | |
945 | | - | |
| 945 | + | |
946 | 946 | | |
947 | 947 | | |
948 | 948 | | |
| |||
0 commit comments