Delete committed dataset orphans (Track X) - #1048
Conversation
These files are committed but read by nothing: the audit at https://quantecon.github.io/data-lectures/audit.json (2026-08-31 run) lists them under orphans, and an authenticated content sweep of 407 repositories (283 QuantEcon org repos plus the forks of the six holding repos, by basename, positive and negative controls in the same pass) on 2026-09-01 found no reader of these paths. Tracker: QuantEcon/workspace-lectures#57. finite_markov writes web_graph_data.txt with %%file before reading it, so neither the _static copy nor the stray copy at the lectures/ root is ever consumed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
A local run of finite_markov.md writes lectures/web_graph_data.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 two committed orphan copies of web_graph_data.txt (one at lectures/ root and one under _static/lecture_specific/finite_markov/) and prevents the root copy from being re-committed, relying on lectures/finite_markov.md generating the file at build time via %%file.
Changes:
- Delete
lectures/web_graph_data.txt. - Delete
lectures/_static/lecture_specific/finite_markov/web_graph_data.txt. - Add
lectures/web_graph_data.txtto.gitignorewith a clarifying comment.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
lectures/web_graph_data.txt |
Removes the committed root copy of the generated graph data file. |
lectures/_static/lecture_specific/finite_markov/web_graph_data.txt |
Removes the committed _static copy that is no longer needed. |
.gitignore |
Ignores the regenerated lectures/web_graph_data.txt to avoid re-committing it. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
📖 Netlify Preview Ready!Preview URL: https://pr-1048--sunny-cactus-210e3e.netlify.app Commit: Build Info
|
|
Validation (2026-09-01, preview |
Deletes both committed copies of
web_graph_data.txt— the_static/lecture_specific/finite_markov/copy and the stray one at thelectures/root — and adds the root path to.gitignore.finite_markov.mdwrites the file with%%filebefore reading it, so neither copy is ever consumed.Published surface and settle. The
_staticcopy serves at 200 (296 B) on python.quantecon.org today; the root copy is not served. This repo has no_build/htmlprune yet (QuantEcon/workspace-lectures#41 rollout), so the file clears on the settle policy: the next Monday cleancache.ymlrebuild followed by the nextpublish*tag (last publishpublish-2026aug14). Nothing reads it in the meantime.Rows for QuantEcon/workspace-lectures#40 (published URL, measured today with a 200 page control and a 404 never-existed control on the same host; expect 404 after the settle):
/_static/lecture_specific/finite_markov/web_graph_data.txtlecture-python.zh-cnholds both copies too and deletes them in the PR of the same title, together with its copies of the Track B files this repo deleted in #1035 and #1037.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