github-starred/cs249r_book
2
0
Fork 0
mirror of https://github.com/harvard-edge/cs249r_book.git synced 2026-05-08 18:01:20 -05:00
Code Issues 81 Packages Projects Releases 24 Wiki Activity

[PR #963] [CLOSED] Chapter 15 #7048

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

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/963
Author: @hzeljko
Created: 9/11/2025
Status: ❌ Closed

Base: dev ← Head: chapter-15

📝 Commits (10+)

  • e3e2840 Merge pull request #958 from hzeljko/line_colors_in_code_listing
  • d70825a feat: add /improve Claude command and review system
  • 6509bc6 feat: streamline /improve command with real Task subagents
  • 2940463 feat: Pure Claude-native review system - no external code
  • 33d2948 feat: Apply high-consensus improvements to introduction Purpose section
  • c4fead3 feat: Add table preservation to review constraints
  • b803a7d feat: Optimize reviewer personas for ML Systems textbook
  • 1bb118e cleanup: Remove test review files
  • 125b039 feat: Add progressive review system commands and knowledge tracking
  • 23e6103 fix: Remove forward references from Introduction and ML Systems chapters

📊 Changes

74 files changed (+4320 additions, -2574 deletions)

View changed files

➕ .claude/AGENT_ARCHITECTURE_PLAN.md (+270 -0)
➕ .claude/COMMIT_RULES.md (+163 -0)
➕ .claude/KNOWLEDGE_MAP.md (+316 -0)
➕ .claude/PROGRESSIVE_KNOWLEDGE_TRACKER.md (+179 -0)
➕ .claude/REVIEW_HANDOFF_SPEC.md (+246 -0)
➕ .claude/WORKFLOW_EXAMPLE.md (+189 -0)
➕ .claude/agents/editor.md (+202 -0)
➕ .claude/agents/reviewer.md (+134 -0)
➕ .claude/agents/stylist.md (+162 -0)
➕ .claude/commands/improve.md (+138 -0)
➕ .claude/commands/review.md (+227 -0)
📝 .gitignore (+4 -1)
📝 cli/commands/build.py (+296 -10)
📝 cli/main.py (+34 -0)
📝 quarto/_extensions/mlsysbook-ext/custom-numbered-blocks/custom-numbered-blocks.lua (+5 -0)
📝 quarto/_extensions/mlsysbook-ext/custom-numbered-blocks/style/foldbox.css (+12 -0)
📝 quarto/_extensions/mlsysbook-ext/custom-numbered-blocks/style/foldbox.lua (+11 -20)
📝 quarto/_extensions/mlsysbook-ext/custom-numbered-blocks/style/foldbox.tex (+1 -1)
📝 quarto/_quarto.yml (+1 -1)
➕ quarto/assets/images/icons/callouts/icon_callout-definition.pdf (+0 -0)

...and 54 more files

📄 Description

Before submitting your Pull Request, please review and complete the items on this checklist.

  • The text has been proofread for grammar and spelling errors.
  • All images, figures, and tables render properly without any glitches.
  • All images have a source or they are properly linked to external sites.
  • The chapter's formatting is consistent with the rest of the book.
  • The chapter has been locally built and tested using Quarto.

🔄 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/963 **Author:** [@hzeljko](https://github.com/hzeljko) **Created:** 9/11/2025 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `chapter-15` --- ### 📝 Commits (10+) - [`e3e2840`](https://github.com/harvard-edge/cs249r_book/commit/e3e284076d42efda6073da60b16fe456bc69f3fd) Merge pull request #958 from hzeljko/line_colors_in_code_listing - [`d70825a`](https://github.com/harvard-edge/cs249r_book/commit/d70825a385accb9768d3d7f137c887b7eb588619) feat: add /improve Claude command and review system - [`6509bc6`](https://github.com/harvard-edge/cs249r_book/commit/6509bc6e1038bce2bd70900976f6afdd89194d73) feat: streamline /improve command with real Task subagents - [`2940463`](https://github.com/harvard-edge/cs249r_book/commit/2940463aef5ddf0e1d6e5ea805bc48bf57b0a7f3) feat: Pure Claude-native review system - no external code - [`33d2948`](https://github.com/harvard-edge/cs249r_book/commit/33d2948d102a0a997e0b6d6dc3079c6db246974a) feat: Apply high-consensus improvements to introduction Purpose section - [`c4fead3`](https://github.com/harvard-edge/cs249r_book/commit/c4fead33e22762732e3f79e7ee377c8860de17a3) feat: Add table preservation to review constraints - [`b803a7d`](https://github.com/harvard-edge/cs249r_book/commit/b803a7ddb37d0089b24f39530008751e49cafbac) feat: Optimize reviewer personas for ML Systems textbook - [`1bb118e`](https://github.com/harvard-edge/cs249r_book/commit/1bb118e9675f743b94d232646a4de3a2b323cae2) cleanup: Remove test review files - [`125b039`](https://github.com/harvard-edge/cs249r_book/commit/125b039fa40464c1f78a9d3abc63e3a8bcea57e1) feat: Add progressive review system commands and knowledge tracking - [`23e6103`](https://github.com/harvard-edge/cs249r_book/commit/23e6103c6bf76bddd8afb4d547946e83f7c1bc45) fix: Remove forward references from Introduction and ML Systems chapters ### 📊 Changes **74 files changed** (+4320 additions, -2574 deletions) <details> <summary>View changed files</summary> ➕ `.claude/AGENT_ARCHITECTURE_PLAN.md` (+270 -0) ➕ `.claude/COMMIT_RULES.md` (+163 -0) ➕ `.claude/KNOWLEDGE_MAP.md` (+316 -0) ➕ `.claude/PROGRESSIVE_KNOWLEDGE_TRACKER.md` (+179 -0) ➕ `.claude/REVIEW_HANDOFF_SPEC.md` (+246 -0) ➕ `.claude/WORKFLOW_EXAMPLE.md` (+189 -0) ➕ `.claude/agents/editor.md` (+202 -0) ➕ `.claude/agents/reviewer.md` (+134 -0) ➕ `.claude/agents/stylist.md` (+162 -0) ➕ `.claude/commands/improve.md` (+138 -0) ➕ `.claude/commands/review.md` (+227 -0) 📝 `.gitignore` (+4 -1) 📝 `cli/commands/build.py` (+296 -10) 📝 `cli/main.py` (+34 -0) 📝 `quarto/_extensions/mlsysbook-ext/custom-numbered-blocks/custom-numbered-blocks.lua` (+5 -0) 📝 `quarto/_extensions/mlsysbook-ext/custom-numbered-blocks/style/foldbox.css` (+12 -0) 📝 `quarto/_extensions/mlsysbook-ext/custom-numbered-blocks/style/foldbox.lua` (+11 -20) 📝 `quarto/_extensions/mlsysbook-ext/custom-numbered-blocks/style/foldbox.tex` (+1 -1) 📝 `quarto/_quarto.yml` (+1 -1) ➕ `quarto/assets/images/icons/callouts/icon_callout-definition.pdf` (+0 -0) _...and 54 more files_ </details> ### 📄 Description Before submitting your Pull Request, please review and complete the items on this checklist. - [x] The text has been proofread for grammar and spelling errors. - [x] All images, figures, and tables render properly without any glitches. - [x] All images have a source or they are properly linked to external sites. - [x] The chapter's formatting is consistent with the rest of the book. - [x] The chapter has been locally built and tested using Quarto. --- <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:13:49 -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#7048
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?