github-starred/cs249r_book
2
0
Fork 0
mirror of https://github.com/harvard-edge/cs249r_book.git synced 2026-05-06 17:49:07 -05:00
Code Issues 85 Packages Projects Releases 24 Wiki Activity

[PR #1513] [MERGED] Batch merge: PRs #1504, #1507, #1510, #1511 + chapter-opener font #7379

New Issue
Closed
opened 2026-04-24 17:27:27 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-04-24 17:27:27 -05:00
Owner
Copy Link

📋 Pull Request Information

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

Base: dev ← Head: chore/pr-merge-2026-04-24

📝 Commits (10+)

  • c19109f fix(tinytorch): LayerNorm gamma/beta missing requires_grad=True
  • 754f056 fix(lab04): remove answer spoilers from prediction prompts in lab_04_data_engr
  • 8863719 Refine Chapter 2 PDF layout
  • f0882af Refine Chapter 3 PDF layout
  • ae7b63c style(footnotes): normalize 7 footnote bold heads to Title Case
  • 22d4ab6 fix(book-pdf): use \rmfamily so chapter-opener labels match body Palatino
  • 22d6731 Merge branch 'worktree-fix+chapter-opener-font' into dev
  • 85c411a Merge origin/dev into chore/pr-merge-2026-04-24
  • 471a735 Merge pull request #1504 from Shashank-Tripathi-07/fix/layernorm-requires-grad
  • 6a417ba Merge pull request #1507 from harvard-edge/iss1417-fix-prediction-spoiler-in-lab04

📊 Changes

14 files changed (+160 additions, -146 deletions)

View changed files

📝 book/quarto/contents/vol1/backmatter/appendix_dam.qmd (+1 -1)
📝 book/quarto/contents/vol1/data_selection/data_selection.qmd (+1 -1)
📝 book/quarto/contents/vol1/frontmatter/about.qmd (+3 -3)
📝 book/quarto/contents/vol1/introduction/introduction.qmd (+2 -2)
📝 book/quarto/contents/vol1/ml_systems/ml_systems.qmd (+74 -63)
📝 book/quarto/contents/vol1/ml_workflow/ml_workflow.qmd (+33 -35)
📝 book/quarto/contents/vol1/nn_computation/nn_computation.qmd (+1 -1)
📝 book/quarto/contents/vol2/compute_infrastructure/compute_infrastructure.qmd (+1 -1)
📝 book/quarto/contents/vol2/fault_tolerance/fault_tolerance.qmd (+1 -1)
📝 book/quarto/contents/vol2/frontmatter/about.qmd (+3 -3)
📝 book/quarto/tex/header-includes.tex (+33 -20)
📝 labs/vol1/lab_04_data_engr.py (+5 -9)
📝 tinytorch/src/13_transformers/13_transformers.py (+2 -2)
📝 tinytorch/tests/13_transformers/test_transformer_gradient_flow.py (+0 -4)

📄 Description

Summary

Batch-merge of four reviewed PRs plus unpushed local book/PDF work onto dev.

Included PRs

  • #1504 — fix(tinytorch): LayerNorm gamma/beta missing requires_grad=True (Shashank-Tripathi-07)
  • #1507 — fix(lab04): remove answer spoilers from prediction prompts in lab_04_data_engr
  • #1510 — Refine Chapter 2 PDF layout (hzeljko)
  • #1511 — Refine Chapter 3 PDF layout (hzeljko)

Local commits folded in

  • fix(book-pdf): use \\rmfamily so chapter-opener labels match body Palatino
  • style(footnotes): normalize 7 footnote bold heads to Title Case
  • Earlier merge worktree-fix+chapter-opener-font

Deliberately skipped

  • #1394 SocratiQ — 76,514+/1,381- diff, CodeQL security failures, no human review yet.

Test plan

  • CI passes on this branch
  • Spell-check warnings (pre-existing, carried from source PRs) reviewed
  • Visual PDF check on Ch2 + Ch3 after merge

🔄 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/1513 **Author:** [@profvjreddi](https://github.com/profvjreddi) **Created:** 4/24/2026 **Status:** ✅ Merged **Merged:** 4/24/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `chore/pr-merge-2026-04-24` --- ### 📝 Commits (10+) - [`c19109f`](https://github.com/harvard-edge/cs249r_book/commit/c19109f9a22af6506b9f9387d5a198b6c6b96456) fix(tinytorch): LayerNorm gamma/beta missing requires_grad=True - [`754f056`](https://github.com/harvard-edge/cs249r_book/commit/754f056fab5e6d22f51a33a1c17eb27ddd9e82ab) fix(lab04): remove answer spoilers from prediction prompts in lab_04_data_engr - [`8863719`](https://github.com/harvard-edge/cs249r_book/commit/88637194f461aebaa9724151c4bb7759e111a8ea) Refine Chapter 2 PDF layout - [`f0882af`](https://github.com/harvard-edge/cs249r_book/commit/f0882af0e5e20319b66067971ef53263bfe2e11f) Refine Chapter 3 PDF layout - [`ae7b63c`](https://github.com/harvard-edge/cs249r_book/commit/ae7b63cb85deb977f192d61038b4062d7a54b16c) style(footnotes): normalize 7 footnote bold heads to Title Case - [`22d4ab6`](https://github.com/harvard-edge/cs249r_book/commit/22d4ab6371267557fa4b8d90f68414cbc33b8293) fix(book-pdf): use \rmfamily so chapter-opener labels match body Palatino - [`22d6731`](https://github.com/harvard-edge/cs249r_book/commit/22d6731f33f4a6db88d54f3a8412829c88f86574) Merge branch 'worktree-fix+chapter-opener-font' into dev - [`85c411a`](https://github.com/harvard-edge/cs249r_book/commit/85c411ab2fa989127057210127c695a76fc9c8e2) Merge origin/dev into chore/pr-merge-2026-04-24 - [`471a735`](https://github.com/harvard-edge/cs249r_book/commit/471a7351815a0f94cd870a8638e739a1886002be) Merge pull request #1504 from Shashank-Tripathi-07/fix/layernorm-requires-grad - [`6a417ba`](https://github.com/harvard-edge/cs249r_book/commit/6a417ba1a8503d46cd82ac447509e3acd4ed6837) Merge pull request #1507 from harvard-edge/iss1417-fix-prediction-spoiler-in-lab04 ### 📊 Changes **14 files changed** (+160 additions, -146 deletions) <details> <summary>View changed files</summary> 📝 `book/quarto/contents/vol1/backmatter/appendix_dam.qmd` (+1 -1) 📝 `book/quarto/contents/vol1/data_selection/data_selection.qmd` (+1 -1) 📝 `book/quarto/contents/vol1/frontmatter/about.qmd` (+3 -3) 📝 `book/quarto/contents/vol1/introduction/introduction.qmd` (+2 -2) 📝 `book/quarto/contents/vol1/ml_systems/ml_systems.qmd` (+74 -63) 📝 `book/quarto/contents/vol1/ml_workflow/ml_workflow.qmd` (+33 -35) 📝 `book/quarto/contents/vol1/nn_computation/nn_computation.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/compute_infrastructure/compute_infrastructure.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/fault_tolerance/fault_tolerance.qmd` (+1 -1) 📝 `book/quarto/contents/vol2/frontmatter/about.qmd` (+3 -3) 📝 `book/quarto/tex/header-includes.tex` (+33 -20) 📝 `labs/vol1/lab_04_data_engr.py` (+5 -9) 📝 `tinytorch/src/13_transformers/13_transformers.py` (+2 -2) 📝 `tinytorch/tests/13_transformers/test_transformer_gradient_flow.py` (+0 -4) </details> ### 📄 Description ## Summary Batch-merge of four reviewed PRs plus unpushed local book/PDF work onto `dev`. ## Included PRs - #1504 — `fix(tinytorch): LayerNorm gamma/beta missing requires_grad=True` (Shashank-Tripathi-07) - #1507 — `fix(lab04): remove answer spoilers from prediction prompts in lab_04_data_engr` - #1510 — `Refine Chapter 2 PDF layout` (hzeljko) - #1511 — `Refine Chapter 3 PDF layout` (hzeljko) ## Local commits folded in - `fix(book-pdf): use \\rmfamily so chapter-opener labels match body Palatino` - `style(footnotes): normalize 7 footnote bold heads to Title Case` - Earlier merge `worktree-fix+chapter-opener-font` ## Deliberately skipped - #1394 SocratiQ — 76,514+/1,381- diff, CodeQL security failures, no human review yet. ## Test plan - [ ] CI passes on this branch - [ ] Spell-check warnings (pre-existing, carried from source PRs) reviewed - [ ] Visual PDF check on Ch2 + Ch3 after merge --- <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-04-24 17:27:27 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area: book
area: collabs
area: kits
area: socratiq
area: tinytorch
area: tools
area: website
bug
dependencies
format: epub
format: pdf
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#7379
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?