[PR #1553] [MERGED] fix(slides): repair the 6 unique broken tinyMLx/courseware URLs #8261

New Issue

GiteaMirror · 2026-04-27T17:37:13-05:00

GiteaMirror commented

2026-04-27 17:37:13 -05:00

📋 Pull Request Information

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

Base: dev ← Head: fix/slides-broken-tinymlx-links

📝 Commits (1)

17c233d fix(slides): repair the 6 unique broken tinyMLx/courseware URLs

📊 Changes

4 files changed (+6 additions, -6 deletions)

View changed files

📝 slides/tinyml-applications.qmd (+1 -1)
📝 slides/tinyml-deploying.qmd (+1 -1)
📝 slides/tinyml-fundamentals.qmd (+1 -1)
📝 slides/tinyml-mlops.qmd (+3 -3)

📄 Description

Follow-up to #1424.

Summary

Link-rot tracker reported 405 broken URLs on the Slides site, dominated by tinyMLx/courseware/raw/master/edX/... references. After de-duplicating against the upstream repo's actual contents (via the GitHub Contents API), only 6 unique URLs were actually broken — the other ~325 occurrences in slides/*.qmd were references to those same 6 dead targets, repeated across the per-course deck inventory tables.

This PR fixes those 6.

Two categories of fix

Category A — file exists with extended title; URL needs encoding (2 fixes)

Was	Now
`readings/5-11-1.pdf`	`readings/5-11-1%20Responsible%20AI%20Intro.pdf`
`readings/5-11-3.pdf`	`readings/5-11-3%20Sustainable%20AI.pdf`

The upstream files contain the title in the filename. The bare-number form in our source matched the upstream README — which is itself broken. The URL-encoded extended form returns 200; verified manually.

Category B — file genuinely missing in upstream repo (4 fixes)

Path	Topic
`other/2-1-14.pdf`	"Assignment Solution"
`readings/4-7-11.pdf`	"Deploying a Multi-Tenant Application"
`slides/3-1-6.pdf`	"What You'll Learn in This Course"
`slides/5-6-X.pdf`	"Model Conversion" (the `X` is a literal placeholder in the upstream README too)

No correct URL exists; the upstream README.md references these as broken links itself. Replaced the entire <a href> cell content with an em-dash (—), preserving the table layout and the topic name in the adjacent cell, but removing the broken link.

Files touched

slides/tinyml-mlops.qmd — 2 URL fixes + 1 em-dash
slides/tinyml-applications.qmd — 1 em-dash
slides/tinyml-deploying.qmd — 1 em-dash
slides/tinyml-fundamentals.qmd — 1 em-dash

Test plan

Verified URL-encoded paths return 200 for the 2 Category A fixes
Confirmed the 4 Category B paths are also broken in the upstream README (so no recoverable URL)
No working tinyMLx/courseware URL was modified — only the 6 broken ones touched
Next nightly link-rot run should report Slides clean of tinyMLx/courseware failures (down from 405 → 0 for that pattern)

_{🔄 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/1553 **Author:** [@profvjreddi](https://github.com/profvjreddi) **Created:** 4/26/2026 **Status:** ✅ Merged **Merged:** 4/26/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `fix/slides-broken-tinymlx-links` --- ### 📝 Commits (1) - [`17c233d`](https://github.com/harvard-edge/cs249r_book/commit/17c233d71bd880b93ec9b763ee45cc9e42832518) fix(slides): repair the 6 unique broken tinyMLx/courseware URLs ### 📊 Changes **4 files changed** (+6 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `slides/tinyml-applications.qmd` (+1 -1) 📝 `slides/tinyml-deploying.qmd` (+1 -1) 📝 `slides/tinyml-fundamentals.qmd` (+1 -1) 📝 `slides/tinyml-mlops.qmd` (+3 -3) </details> ### 📄 Description Follow-up to #1424. ## Summary Link-rot tracker reported 405 broken URLs on the Slides site, dominated by `tinyMLx/courseware/raw/master/edX/...` references. After de-duplicating against the upstream repo's actual contents (via the GitHub Contents API), **only 6 unique URLs were actually broken** — the other ~325 occurrences in `slides/*.qmd` were references to those same 6 dead targets, repeated across the per-course deck inventory tables. This PR fixes those 6. ## Two categories of fix ### Category A — file exists with extended title; URL needs encoding (2 fixes) | Was | Now | |---|---| | `readings/5-11-1.pdf` | `readings/5-11-1%20Responsible%20AI%20Intro.pdf` | | `readings/5-11-3.pdf` | `readings/5-11-3%20Sustainable%20AI.pdf` | The upstream files contain the title in the filename. The bare-number form in our source matched the upstream README — which is itself broken. The URL-encoded extended form returns 200; verified manually. ### Category B — file genuinely missing in upstream repo (4 fixes) | Path | Topic | |---|---| | `other/2-1-14.pdf` | "Assignment Solution" | | `readings/4-7-11.pdf` | "Deploying a Multi-Tenant Application" | | `slides/3-1-6.pdf` | "What You'll Learn in This Course" | | `slides/5-6-X.pdf` | "Model Conversion" (the `X` is a literal placeholder in the upstream README too) | No correct URL exists; the upstream `README.md` references these as broken links itself. Replaced the entire `<a href>` cell content with an em-dash (`—`), preserving the table layout and the topic name in the adjacent cell, but removing the broken link. ## Files touched - `slides/tinyml-mlops.qmd` — 2 URL fixes + 1 em-dash - `slides/tinyml-applications.qmd` — 1 em-dash - `slides/tinyml-deploying.qmd` — 1 em-dash - `slides/tinyml-fundamentals.qmd` — 1 em-dash ## Test plan - [x] Verified URL-encoded paths return 200 for the 2 Category A fixes - [x] Confirmed the 4 Category B paths are also broken in the upstream README (so no recoverable URL) - [x] No working `tinyMLx/courseware` URL was modified — only the 6 broken ones touched - [ ] Next nightly link-rot run should report Slides clean of `tinyMLx/courseware` failures (down from 405 → 0 for that pattern) --- <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-27 17:37:13 -05:00

GiteaMirror closed this issue

2026-04-27 17:37:15 -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#8261