[PR #1171] [MERGED] TinyTorch: progressive disclosure + Windows install cleanup #1153

Closed
opened 2026-03-22 16:02:13 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1171
Author: @profvjreddi
Created: 2/13/2026
Status: Merged
Merged: 2/13/2026
Merged by: @profvjreddi

Base: devHead: feature/tinytorch-core


📝 Commits (5)

  • 7b43dc5 Merge dev into feature/tinytorch-core
  • 0630674 fix(module16): correct sparsity percentage bugs in compression module
  • 3947b2d fix(tinytorch): enforce progressive disclosure across 9 modules
  • a26ee9f Merge dev into feature/tinytorch-core (includes PR #1169 Windows fix)
  • 173f28f fix(tinytorch): clean up Windows install fix comments from PR #1169

📊 Changes

12 files changed (+179 additions, -239 deletions)

View changed files

📝 tinytorch/site/extra/install.sh (+3 -1)
📝 tinytorch/src/01_tensor/01_tensor.py (+60 -61)
📝 tinytorch/src/02_activations/02_activations.py (+19 -19)
📝 tinytorch/src/05_dataloader/05_dataloader.py (+29 -45)
📝 tinytorch/src/06_autograd/06_autograd.py (+10 -10)
📝 tinytorch/src/07_optimizers/07_optimizers.py (+7 -7)
📝 tinytorch/src/10_tokenization/10_tokenization.py (+10 -54)
📝 tinytorch/src/13_transformers/13_transformers.py (+4 -5)
📝 tinytorch/src/14_profiling/14_profiling.py (+6 -6)
📝 tinytorch/src/16_compression/16_compression.py (+26 -25)
📝 tinytorch/src/17_acceleration/17_acceleration.py (+1 -1)
📝 tinytorch/tito/commands/setup.py (+4 -5)

📄 Description

Summary

  • Enforce progressive disclosure across 9 TinyTorch modules (tensor, activations, layers, losses, autograd, optimizers, training, tokenization, attention, transformers, acceleration, benchmarking, capstone)
  • Fix sparsity percentage bugs in compression module (module 16)
  • Clean up Windows install fix comments from PR #1169 (contributor attribution + proper explanations)

Context

  • Modules updated to follow consistent pedagogical scaffolding
  • PR #1169 (Windows Git Bash install fix by @adil-mubashir-ch) was merged into dev; this branch includes a follow-up patch that cleans up comment typos and moves is_windows to its usage site

Test plan

  • Verify tito setup completes on macOS/Linux
  • Verify module imports work: python3 -c "import tinytorch"
  • Spot-check progressive disclosure in updated modules

🔄 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/1171 **Author:** [@profvjreddi](https://github.com/profvjreddi) **Created:** 2/13/2026 **Status:** ✅ Merged **Merged:** 2/13/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `feature/tinytorch-core` --- ### 📝 Commits (5) - [`7b43dc5`](https://github.com/harvard-edge/cs249r_book/commit/7b43dc5ff5b73c2ca4d943edbb0a72d620b3bdc5) Merge dev into feature/tinytorch-core - [`0630674`](https://github.com/harvard-edge/cs249r_book/commit/0630674a71783cb37e0c11c723a11528b87740e5) fix(module16): correct sparsity percentage bugs in compression module - [`3947b2d`](https://github.com/harvard-edge/cs249r_book/commit/3947b2defa9468033e2bd1f952af9e2ef94b7147) fix(tinytorch): enforce progressive disclosure across 9 modules - [`a26ee9f`](https://github.com/harvard-edge/cs249r_book/commit/a26ee9fff16e5c7f7cfa10e48d4997b6bfa4d64d) Merge dev into feature/tinytorch-core (includes PR #1169 Windows fix) - [`173f28f`](https://github.com/harvard-edge/cs249r_book/commit/173f28f88d81f1608efeaec6e931b626df1805a9) fix(tinytorch): clean up Windows install fix comments from PR #1169 ### 📊 Changes **12 files changed** (+179 additions, -239 deletions) <details> <summary>View changed files</summary> 📝 `tinytorch/site/extra/install.sh` (+3 -1) 📝 `tinytorch/src/01_tensor/01_tensor.py` (+60 -61) 📝 `tinytorch/src/02_activations/02_activations.py` (+19 -19) 📝 `tinytorch/src/05_dataloader/05_dataloader.py` (+29 -45) 📝 `tinytorch/src/06_autograd/06_autograd.py` (+10 -10) 📝 `tinytorch/src/07_optimizers/07_optimizers.py` (+7 -7) 📝 `tinytorch/src/10_tokenization/10_tokenization.py` (+10 -54) 📝 `tinytorch/src/13_transformers/13_transformers.py` (+4 -5) 📝 `tinytorch/src/14_profiling/14_profiling.py` (+6 -6) 📝 `tinytorch/src/16_compression/16_compression.py` (+26 -25) 📝 `tinytorch/src/17_acceleration/17_acceleration.py` (+1 -1) 📝 `tinytorch/tito/commands/setup.py` (+4 -5) </details> ### 📄 Description ## Summary - Enforce progressive disclosure across 9 TinyTorch modules (tensor, activations, layers, losses, autograd, optimizers, training, tokenization, attention, transformers, acceleration, benchmarking, capstone) - Fix sparsity percentage bugs in compression module (module 16) - Clean up Windows install fix comments from PR #1169 (contributor attribution + proper explanations) ## Context - Modules updated to follow consistent pedagogical scaffolding - PR #1169 (Windows Git Bash install fix by @adil-mubashir-ch) was merged into dev; this branch includes a follow-up patch that cleans up comment typos and moves `is_windows` to its usage site ## Test plan - [ ] Verify `tito setup` completes on macOS/Linux - [ ] Verify module imports work: `python3 -c "import tinytorch"` - [ ] Spot-check progressive disclosure in updated modules --- <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-03-22 16:02:13 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#1153