[PR #1331] [CLOSED] #1317 [Bug] Labs lab01 added prediction radios rendering

## 📋 Pull Request Information **Original PR:** https://github.com/harvard-edge/cs249r_book/pull/1331 **Author:** [@asgalon](https://github.com/asgalon) **Created:** 4/14/2026 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `feature/feature-1317` --- ### 📝 Commits (1) - [`a64ab4b`](https://github.com/harvard-edge/cs249r_book/commit/a64ab4b40c75c6f0ef7296ba20084d967f09eb34) #1317 [Bug] Labs lab01 added prediction radios rendering ### 📊 Changes **1 file changed** (+136 additions, -72 deletions) <details> <summary>View changed files</summary> 📝 `labs/vol1/lab_01_ml_intro.py` (+136 -72) </details> ### 📄 Description ## Summary Make the prediction choices visible to actually add the predictions and unlock the lower part ## Area  - [ ] Book (textbook content, figures, exercises) - [ ] TinyTorch (modules, tests, milestones) - [ ] StaffML (interview questions, challenges) - [ ] Kits (hardware labs) - [ ] Infrastructure (CI/CD, scripts, config) - [x] Labs - [ ] - [ ] ## Changes labs/vol1/lab_01_ml_intro.py: - added mo.vstack with prediction radiobuttons for each step. ## Testing  - [ ] Rendered the book locally (`quarto render`) - [ ] Ran tests (`pytest tests/`) - [ ] Ran `tito module test NN` for affected module(s) - [x] Manual verification (describe below) - [x] Ran 'make tests' - [x] Ran 'marimo run vol1/lab_01_ml_intro.py' Checked lab flow in browser manually ## Related Issues Related to #1317 TODO: Check consistency of numerical values in prediction / lower part (200x / 6x, 512MB, 7812MB) --- *By submitting this PR, you agree to release your contribution under the project's license.* --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror commented

2026-04-21 22:21:38 -05:00

Owner

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1331
Author: @asgalon
Created: 4/14/2026
Status: ❌ Closed

Base: dev ← Head: feature/feature-1317

📝 Commits (1)

a64ab4b #1317 [Bug] Labs lab01 added prediction radios rendering

📊 Changes

1 file changed (+136 additions, -72 deletions)

View changed files

📝 labs/vol1/lab_01_ml_intro.py (+136 -72)

📄 Description

Summary

Make the prediction choices visible to actually add the predictions and unlock the lower part

Area

Book (textbook content, figures, exercises)
TinyTorch (modules, tests, milestones)
StaffML (interview questions, challenges)
Kits (hardware labs)
Infrastructure (CI/CD, scripts, config)
Labs
- [ ]

Changes

labs/vol1/lab_01_ml_intro.py:

added mo.vstack with prediction radiobuttons for each step.

Testing

Rendered the book locally (quarto render)
Ran tests (pytest tests/)
Ran tito module test NN for affected module(s)
Manual verification (describe below)
Ran 'make tests'
Ran 'marimo run vol1/lab_01_ml_intro.py'

Checked lab flow in browser manually

Related to #1317

TODO: Check consistency of numerical values in prediction / lower part (200x / 6x, 512MB, 7812MB)

By submitting this PR, you agree to release your contribution under the project's license.

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

GiteaMirror added the pull-request label 2026-04-21 22:21:38 -05:00

GiteaMirror closed this issue

2026-04-21 22:21:40 -05:00

[PR #1331] [CLOSED] #1317 [Bug] Labs lab01 added prediction radios rendering #6478