github-starred/cs249r_book
2
0
Fork 0
mirror of https://github.com/harvard-edge/cs249r_book.git synced 2026-07-18 00:44:29 -05:00
Code Issues 276 Packages Projects Releases 26 Wiki Activity

[PR #1679] [MERGED] fix: math notation and prose drift cleanup (31 sites, 11 files) #17366

New Issue
Closed
opened 2026-05-24 12:09:14 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-05-24 12:09:14 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1679
Author: @profvjreddi
Created: 5/5/2026
Status: ✅ Merged
Merged: 5/5/2026
Merged by: @profvjreddi

Base: dev ← Head: fix/math-adjacent-spans

📝 Commits (6)

  • caabb9c fix(edge_intelligence): collapse adjacent math spans in 224x224x3 dim
  • 9da65ed fix(math): wrap multi-letter math subscripts in \text{} (5 chapters)
  • 1448133 fix(hw_acceleration): merge math-anchored multiplier into single span
  • d048bdc fix(prose): lowercase concept terms in body prose (6 sites)
  • 7044d34 fix(prose): close up pre-/non- compound prefixes per CMS $7.89 (3 chapters)
  • 86fbe1e fix(security_privacy): add period to bare 'vs' in bullet list (1 site)

📊 Changes

15 files changed (+31 additions, -31 deletions)

View changed files

📝 book/quarto/contents/vol1/conclusion/conclusion.qmd (+1 -1)
📝 book/quarto/contents/vol1/data_engineering/data_engineering.qmd (+2 -2)
📝 book/quarto/contents/vol1/data_selection/data_selection.qmd (+4 -4)
📝 book/quarto/contents/vol1/hw_acceleration/hw_acceleration.qmd (+1 -1)
📝 book/quarto/contents/vol1/ml_systems/ml_systems.qmd (+3 -3)
📝 book/quarto/contents/vol1/nn_computation/nn_computation.qmd (+1 -1)
📝 book/quarto/contents/vol1/optimizations/model_compression.qmd (+1 -1)
📝 book/quarto/contents/vol2/collective_communication/collective_communication.qmd (+2 -2)
📝 book/quarto/contents/vol2/data_storage/data_storage.qmd (+10 -10)
📝 book/quarto/contents/vol2/distributed_training/distributed_training.qmd (+1 -1)
📝 book/quarto/contents/vol2/edge_intelligence/edge_intelligence.qmd (+1 -1)
📝 book/quarto/contents/vol2/network_fabrics/network_fabrics.qmd (+1 -1)
📝 book/quarto/contents/vol2/performance_engineering/performance_engineering.qmd (+1 -1)
📝 book/quarto/contents/vol2/security_privacy/security_privacy.qmd (+1 -1)
📝 book/quarto/contents/vol2/sustainable_ai/sustainable_ai.qmd (+1 -1)

📄 Description

Summary

Style-drift cleanup across 11 chapter files, found via systematic audit of all 33 chapters against book-prose.md §2 (math notation) and §10 (MIT Press editorial standard).

Six commits, each one logical change:

  1. caabb9c14 — vol2/edge_intelligence: broken triple-dim \$224 \\times 224\$\$\\times\$3 → single span
  2. 9da65edf5 — Math subscripts: 8 sites in 5 files (E_{op}, T_{io}, N_{GPUs\\_per\\_node}, etc.) wrapped in \\text{}
  3. 1448133cb — vol1/hw_acceleration:752: math-anchored multiplier \$10^2\$\$\\times\$ merged
  4. d048bdcb1 — Concept terms lowercase in body prose (Data Gravity, Transformer architecture): 6 sites
  5. 7044d34b8 — Pre-/non- prefix close-ups per CMS §7.89: 14 sites in 3 files
  6. 86fbe1e9a — vol2/security_privacy:1342: bare vs → vs.

Test plan

  • Pre-commit hooks all pass (already verified locally on each commit)
  • Book renders cleanly (PDF + HTML)
  • No index/cross-ref breakage from \\index{Pre-staging} → \\index{Prestaging} rename (single occurrence, verified)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/harvard-edge/cs249r_book/pull/1679 **Author:** [@profvjreddi](https://github.com/profvjreddi) **Created:** 5/5/2026 **Status:** ✅ Merged **Merged:** 5/5/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `fix/math-adjacent-spans` --- ### 📝 Commits (6) - [`caabb9c`](https://github.com/harvard-edge/cs249r_book/commit/caabb9c14e5a1043316b6acfd116fb0f23084123) fix(edge_intelligence): collapse adjacent math spans in 224x224x3 dim - [`9da65ed`](https://github.com/harvard-edge/cs249r_book/commit/9da65edf519ae5cfaef72418582b5fff854eb9fa) fix(math): wrap multi-letter math subscripts in \text{} (5 chapters) - [`1448133`](https://github.com/harvard-edge/cs249r_book/commit/1448133cb78c640fe9f8d21dee4911528e599644) fix(hw_acceleration): merge math-anchored multiplier into single span - [`d048bdc`](https://github.com/harvard-edge/cs249r_book/commit/d048bdcb1c706f08d5d8dc9ce29e3f9e79e93084) fix(prose): lowercase concept terms in body prose (6 sites) - [`7044d34`](https://github.com/harvard-edge/cs249r_book/commit/7044d34b870d0751fd535421f3e8865b42ebcdd2) fix(prose): close up pre-/non- compound prefixes per CMS $7.89 (3 chapters) - [`86fbe1e`](https://github.com/harvard-edge/cs249r_book/commit/86fbe1e9a5e43d8644eead416bb7dc39d141ecdb) fix(security_privacy): add period to bare 'vs' in bullet list (1 site) ### 📊 Changes **15 files changed** (+31 additions, -31 deletions) <details> <summary>View changed files</summary> 📝 `book/quarto/contents/vol1/conclusion/conclusion.qmd` (+1 -1) 📝 `book/quarto/contents/vol1/data_engineering/data_engineering.qmd` (+2 -2) 📝 `book/quarto/contents/vol1/data_selection/data_selection.qmd` (+4 -4) 📝 `book/quarto/contents/vol1/hw_acceleration/hw_acceleration.qmd` (+1 -1) 📝 `book/quarto/contents/vol1/ml_systems/ml_systems.qmd` (+3 -3) 📝 `book/quarto/contents/vol1/nn_computation/nn_computation.qmd` (+1 -1) 📝 `book/quarto/contents/vol1/optimizations/model_compression.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/collective_communication/collective_communication.qmd` (+2 -2) 📝 `book/quarto/contents/vol2/data_storage/data_storage.qmd` (+10 -10) 📝 `book/quarto/contents/vol2/distributed_training/distributed_training.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/edge_intelligence/edge_intelligence.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/network_fabrics/network_fabrics.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/performance_engineering/performance_engineering.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/security_privacy/security_privacy.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/sustainable_ai/sustainable_ai.qmd` (+1 -1) </details> ### 📄 Description ## Summary Style-drift cleanup across 11 chapter files, found via systematic audit of all 33 chapters against book-prose.md §2 (math notation) and §10 (MIT Press editorial standard). Six commits, each one logical change: 1. **caabb9c14** — `vol2/edge_intelligence`: broken triple-dim `\$224 \\times 224\$\$\\times\$3` → single span 2. **9da65edf5** — Math subscripts: 8 sites in 5 files (`E_{op}`, `T_{io}`, `N_{GPUs\\_per\\_node}`, etc.) wrapped in `\\text{}` 3. **1448133cb** — `vol1/hw_acceleration:752`: math-anchored multiplier `\$10^2\$\$\\times\$` merged 4. **d048bdcb1** — Concept terms lowercase in body prose (Data Gravity, Transformer architecture): 6 sites 5. **7044d34b8** — Pre-/non- prefix close-ups per CMS §7.89: 14 sites in 3 files 6. **86fbe1e9a** — `vol2/security_privacy:1342`: bare `vs` → `vs.` ## Test plan - [ ] Pre-commit hooks all pass (already verified locally on each commit) - [ ] Book renders cleanly (PDF + HTML) - [ ] No index/cross-ref breakage from `\\index{Pre-staging}` → `\\index{Prestaging}` rename (single occurrence, verified) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-24 12:09:14 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area: book
area: collabs
area: kits
area: labs
area: socratiq
area: staffml
area: tinytorch
area: tools
area: website
bug
dependencies
format: epub
format: pdf
javascript
link-health
link-rot
priority-high
pull-request
Mirrored from GitHub Pull Request
 staffml
type: bug
type: citation
type: code
type: errata
type: improvement
type: new
type: question
vault-sli
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#17366
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?