Retire the 23 committed_unreferenced annotations deleted under Track X - #111
Open
mmcky wants to merge 1 commit into
Open
Retire the 23 committed_unreferenced annotations deleted under Track X#111mmcky wants to merge 1 commit into
mmcky wants to merge 1 commit into
Conversation
The Track X orphan sweep (QuantEcon/workspace-lectures#57) merged on 2026-09-01: twelve PRs deleted 22 of the 24 committed orphans the audit lists, plus 46 translation copies it cannot see. Their keys under committed_unreferenced now name files that no longer exist. The lookup is inert at runtime (build_audit.py consults the map only for paths the scan still finds), so this is hygiene rather than a gate, following the precedent of #90. The one key kept is lecture-python-programming's python_advanced_features/test_table.csv (kind: exercise-download), which the exercise prose tells readers to fetch by exact URL and which stays committed. PLAN.md's Track X row, the graph.txt paragraph, the not-data-consumers note, the remaining-tracks line and the Phase-2 checkbox are brought to the same date. Strict audit run locally against the merged mains: 40 static files, 1 orphan (the kept one), every problems bucket empty. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR retires stale committed_unreferenced annotations after the Track X orphan sweep removed the corresponding committed files in lecture repositories, and updates PLAN.md to mark Track X complete in the project roadmap.
Changes:
- Remove 23 now-dead
committed_unreferencedentries fromscripts/audit_annotations.yml, leaving only the intentional exercise-download entry. - Update
PLAN.mdto mark Track X as completed and record the outcome in the migration plan narrative/checklist.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| scripts/audit_annotations.yml | Drops stale committed_unreferenced keys for files deleted under Track X; retains the single exercise-download entry. |
| PLAN.md | Updates Track X status and related roadmap text to reflect completion of the orphan sweep. |
Suppressed comments (1)
PLAN.md:350
- This Phase 9 checkbox line records "22 audit orphans" deleted under Track X, but the Track X breakdown in the same document (and the removed annotation keys) totals 23 deleted. Update the count here to keep the plan's progress reporting consistent.
- [x] Remove lecture repos' duplicate copies as each repoint merges — **done 2026-09-01** (Track X, [QuantEcon/workspace-lectures#57](https://github.com/QuantEcon/workspace-lectures/issues/57)): 22 audit orphans plus 46 translation copies deleted across twelve repos, one PR each; the wasm mirror copies went only after wasm read data-lectures directly
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| | **D — `programming`** | 1, **done**: `test_pwt.csv` rode with wave C2 ([#98](https://github.com/QuantEcon/data-lectures/pull/98), [#99](https://github.com/QuantEcon/data-lectures/pull/99)); its four consuming repos are recorded in [#101](https://github.com/QuantEcon/data-lectures/pull/101) | none | — | | ||
| | **E — dynamic / live-API** | the UNRATE twin, then the 15 incidental API lectures | wasm is the forcing customer | [#14](https://github.com/QuantEcon/data-lectures/issues/14) schema decisions, [#26](https://github.com/QuantEcon/data-lectures/issues/26) fetch layer | | ||
| | **X — orphan sweep** | 24 committed orphans across 6 repos (`audit.json`, 2026-08-31) — dp 10, programming 5, intro 4, python.myst 2, wasm 2, `continuous_time_mcs` 1; tracked at [QuantEcon/workspace-lectures#57](https://github.com/QuantEcon/workspace-lectures/issues/57) | per repo | nothing — every repo's repoints have landed; `graph.txt` needs its own per-repo reader sweep (below) | | ||
| | **X — orphan sweep** | **done 2026-09-01** — 22 of the 24 `audit.json` orphans deleted (dp 10, programming 4, intro 4, python.myst 2, wasm 2, `continuous_time_mcs` 1), `python_advanced_features/test_table.csv` kept as an exercise download, plus 46 translation copies the audit cannot see (`lecture-python.zh-cn` 15, `lecture-python-programming.{zh-cn,fr,fa}` 9 each, `.ml` 6, `lecture-intro.zh-cn` 4); twelve PRs, ledger at [QuantEcon/workspace-lectures#57](https://github.com/QuantEcon/workspace-lectures/issues/57) | — | site clearance rides the settle policy, verified at [QuantEcon/workspace-lectures#40](https://github.com/QuantEcon/workspace-lectures/issues/40) | |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Track X orphan sweep (QuantEcon/workspace-lectures#57) merged this morning — twelve PRs, one per repo, deleting 22 of the 24 committed orphans
audit.jsonlists plus 46 translation copies the audit cannot see. The 23committed_unreferencedkeys for those files now name paths that no longer exist, so this retires them in step, the way #90 retired the staledatasets:entries.What changes.
scripts/audit_annotations.yml: 23 keys removed; the one kept islecture-python-programming:…/python_advanced_features/test_table.csv(kind: exercise-download), which the exercise prose tells readers to fetch by exact URL and which stays committed.PLAN.md: the Track X row flips to done with the measured counts, thegraph.txtparagraph records the deletion and the accepted outside-org reader, the "not data consumers" note and the "tracks remaining" line move to past tense, and the Phase-2 checkbox is ticked.Why it is safe to merge before the next weekly scan. The map is a lookup only —
build_audit.pyreadscommitted_unreferencedfor paths the scan finds, and there is no stale-key check in--strict— so the keys were inert the moment the files went, and their absence cannot make anything red. Verified by running the strict build locally against the mergedmainof all eight scanned repos: 40 static files, 1 orphan (the kept one), everyproblemsbucket empty, exit 0. The next scheduled dashboard build will show the same.No closing keyword precedes any cross-repo reference in this body or the commit message.
🤖 Generated with Claude Code