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

[PR #1959] docs(instructors): fix broken lab links #34682

New Issue
Open
opened 2026-07-14 19:28:56 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-07-14 19:28:56 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1959
Author: @coyaSONG
Created: 7/14/2026
Status: 🔄 Open

Base: dev ← Head: iss1810-fix-instructor-lab-links

📝 Commits (1)

📊 Changes

3 files changed (+33 additions, -33 deletions)

View changed files

📝 instructors/course-map.qmd (+16 -16)
📝 instructors/foundations-syllabus.qmd (+16 -16)
📝 instructors/getting-started.qmd (+1 -1)

📄 Description

Summary

Update 33 broken Lab links across three Instructor guides from obsolete flat
.html endpoints to the directory routes generated by the Labs build.

Area

  • Book (textbook content, figures, exercises)
  • TinyTorch (modules, tests, milestones)
  • StaffML (interview questions, challenges)
  • Kits (hardware labs)
  • Infrastructure (CI/CD, scripts, config)
  • Instructors (course map, syllabus, getting-started guide)

Changes

  • Replace 33 /labs/vol1/lab_*.html links with canonical
    /labs/vol1/lab_*/ routes.
  • Keep link text and course content unchanged.
  • Cover all 16 Volume I labs referenced by the Instructor materials.

Testing

  • Rendered the book locally (quarto render)
  • Ran tests (pytest tests/)
  • Ran tito module test NN for affected module(s)
  • Manual verification (described below)
  • Ran pre-commit on all three changed files
  • Ran shared/scripts/check-internal-links.py on all three changed files
  • Confirmed all 16 unique replacement URLs return HTTP 200
  • Confirmed git diff --check passes

The old sample route /labs/vol1/lab_00_introduction.html returns HTTP 404,
while /labs/vol1/lab_00_introduction/ returns HTTP 200. Quarto is not
installed in my local environment, so the repository's Instructor validation
workflow will perform the full site render.

Related Issues

Related to #1810

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.

## 📋 Pull Request Information **Original PR:** https://github.com/harvard-edge/cs249r_book/pull/1959 **Author:** [@coyaSONG](https://github.com/coyaSONG) **Created:** 7/14/2026 **Status:** 🔄 Open **Base:** `dev` ← **Head:** `iss1810-fix-instructor-lab-links` --- ### 📝 Commits (1) - [`d580440`](https://github.com/harvard-edge/cs249r_book/commit/d5804405e5c3dcd6d8b7972536d0a40f0a544521) Fix instructor lab links (issue #1810) ### 📊 Changes **3 files changed** (+33 additions, -33 deletions) <details> <summary>View changed files</summary> 📝 `instructors/course-map.qmd` (+16 -16) 📝 `instructors/foundations-syllabus.qmd` (+16 -16) 📝 `instructors/getting-started.qmd` (+1 -1) </details> ### 📄 Description ## Summary Update 33 broken Lab links across three Instructor guides from obsolete flat `.html` endpoints to the directory routes generated by the Labs build. ## Area - [ ] Book (textbook content, figures, exercises) - [ ] TinyTorch (modules, tests, milestones) - [ ] StaffML (interview questions, challenges) - [ ] Kits (hardware labs) - [ ] Infrastructure (CI/CD, scripts, config) - [x] Instructors (course map, syllabus, getting-started guide) ## Changes - Replace 33 `/labs/vol1/lab_*.html` links with canonical `/labs/vol1/lab_*/` routes. - Keep link text and course content unchanged. - Cover all 16 Volume I labs referenced by the Instructor materials. ## Testing - [ ] Rendered the book locally (`quarto render`) - [ ] Ran tests (`pytest tests/`) - [ ] Ran `tito module test NN` for affected module(s) - [x] Manual verification (described below) - [x] Ran pre-commit on all three changed files - [x] Ran `shared/scripts/check-internal-links.py` on all three changed files - [x] Confirmed all 16 unique replacement URLs return HTTP 200 - [x] Confirmed `git diff --check` passes The old sample route `/labs/vol1/lab_00_introduction.html` returns HTTP 404, while `/labs/vol1/lab_00_introduction/` returns HTTP 200. Quarto is not installed in my local environment, so the repository's Instructor validation workflow will perform the full site render. ## Related Issues Related to #1810 --- *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 added the pull-request label 2026-07-14 19:28:56 -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#34682
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?