Delete committed dataset orphans (Track X) - #304
Conversation
These files are committed but read by nothing in this repository: every lecture that uses these datasets reads them from QuantEcon/data-lectures (verified per file on 2026-09-01), and the remaining files are written by the lectures themselves with %%file / %%writefile before being read. Translations sit outside the data-lectures audit's scanned set, so this is the translation half of the orphan sweep tracked at QuantEcon/workspace-lectures#57. An authenticated content sweep of 407 repositories (283 QuantEcon org repos plus the forks of the holding repos, by basename, positive and negative controls in the same pass) on 2026-09-01 found no reader of these paths. Mirrors the English repo's deletion in the same set: the two World Bank files were superseded when the GDP-per-capita lecture moved to data-lectures, and fig_3.ods is the source-format twin of fig_3.xlsx, referenced by nothing. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
short_path regenerates graph.txt with %%file before reading it, so the committed bytes are never consumed. The 2026-09-01 organisation sweep found no reader of this repo's copy. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
A local run of short_path.md writes lectures/graph.txt into the working tree; ignoring it prevents the deleted copy from being re-committed by a later git add -A. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Removes previously committed “orphan” dataset artifacts from the translated lecture repo, aligning with the referenced orphan sweep and ensuring regenerated lecture-generated files aren’t re-committed.
Changes:
- Delete
lectures/graph.txt(now generated byshort_path.mdvia%%file). - Delete two World Bank-derived dataset CSVs under
lectures/datasets/. - Add
lectures/graph.txtto.gitignore.
Reviewed changes
Copilot reviewed 2 out of 5 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| lectures/graph.txt | Removes committed graph data that is regenerated by a lecture. |
| lectures/datasets/Metadata_Country_API_NY.GDP.PCAP.CD_DS2_en_csv_v2_4770417.csv | Removes orphaned World Bank metadata CSV from the repo. |
| lectures/datasets/GDP_per_capita_world_bank.csv | Removes orphaned World Bank GDP-per-capita dataset CSV from the repo. |
| .gitignore | Ignores regenerated lectures/graph.txt to avoid re-commits. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
✅ Deploy Preview for playful-platypus-17d3bb ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Validation (2026-09-01, preview |
Deletes the same four paths as
lecture-python-intro's PR of the same title — the two World Bank files andfig_3.odsunderlectures/datasets/, and the shadowedlectures/graph.txt(separate commit) — and addslectures/graph.txtto.gitignore.Translations sit outside the data-lectures audit's scanned set, so this is the translation half of the orphan sweep: the same files the English repo deletes in its PR of the same title, verified here file by file on 2026-09-01 — every lecture that uses these datasets reads them from
QuantEcon/data-lectures, and the rest are written by the lectures themselves with%%file/%%writefilebefore being read. The translation sync is.md-only, so nothing will re-add them.short_path.mdwritesgraph.txtwith%%file; nothing in this edition mentions the other three.Published surface: none — all four are 404 on
quantecon.github.io/lecture-intro.zh-cn/today (intro.html200 control);datasets/and the lectures root sit outsidehtml_static_path. The deletion is complete on merge; no rows for QuantEcon/workspace-lectures#40.Deletion-time reader sweep (2026-09-01). Authenticated tarball-and-grep pass over 407 repositories: all 283 QuantEcon org repos (70 private, 26 archived;
numfocus,quantecon-book-dpandtest-cliare empty and unfetchable) plus 130 forks of the six holding repos (2 returned 404 and are recorded as not-swept rather than clean). Matched by basename, not URL, with each hit's three preceding lines folded in because URLs get split across source lines. Controls fired in the same pass: a never-existing basename returned 0 hits and 0 copies, and both known readers were found — the Englishpython_advanced_features.mdexercise link totest_table.csv, anddevopseng99/project.lecture-wasm's runtime fetch of intro'sgraph.txt. Result: no repository reads any path this PR deletes. Every other co-occurrence of these basenames is a%%fileembed, prose, a.gitignoreline, or review text; the pre-MySTmaster/source/…reads in old course repos have been dead since the MyST migration and are unaffected.Part of the Track X orphan sweep, QuantEcon/workspace-lectures#57 (one PR per holding repo, identical titles). Source list: https://quantecon.github.io/data-lectures/audit.json
orphans, 2026-08-31 run.🤖 Generated with Claude Code