Skip to content

Delete committed dataset orphans (Track X) - #1048

Merged
mmcky merged 2 commits into
mainfrom
track-x-orphans
Sep 1, 2026
Merged

Delete committed dataset orphans (Track X)#1048
mmcky merged 2 commits into
mainfrom
track-x-orphans

Conversation

@mmcky

@mmcky mmcky commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Deletes both committed copies of web_graph_data.txt — the _static/lecture_specific/finite_markov/ copy and the stray one at the lectures/ root — and adds the root path to .gitignore. finite_markov.md writes the file with %%file before reading it, so neither copy is ever consumed.

Published surface and settle. The _static copy serves at 200 (296 B) on python.quantecon.org today; the root copy is not served. This repo has no _build/html prune yet (QuantEcon/workspace-lectures#41 rollout), so the file clears on the settle policy: the next Monday clean cache.yml rebuild followed by the next publish* tag (last publish publish-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):

path today
/_static/lecture_specific/finite_markov/web_graph_data.txt 200, 296 B

lecture-python.zh-cn holds 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-dp and test-cli are 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 English python_advanced_features.md exercise link to test_table.csv, and devopseng99/project.lecture-wasm's runtime fetch of intro's graph.txt. Result: no repository reads any path this PR deletes. Every other co-occurrence of these basenames is a %%file embed, prose, a .gitignore line, or review text; the pre-MyST master/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

mmcky and others added 2 commits September 1, 2026 10:04
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>
Copilot AI lite review requested due to automatic review settings September 1, 2026 00:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.txt to .gitignore with 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.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

📖 Netlify Preview Ready!

Preview URL: https://pr-1048--sunny-cactus-210e3e.netlify.app

Commit: 9d2aef6


Build Info

@mmcky

mmcky commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Validation (2026-09-01, preview pr-1048--sunny-cactus): root web_graph_data.txt → 404; finite_markov.html → 200 (176,555 B) and its rendered page carries the %%file web_graph_data.txt cell with no _static/lecture_specific/finite_markov reference. Build log: 0 warnings mentioning the basename. Expected exception: _static/lecture_specific/finite_markov/web_graph_data.txt still serves at 200 (296 B) on the preview — the restored build-cache is the 2026-08-31T05:10Z one, built from main. (This is the stale-_build behaviour from QuantEcon/workspace-lectures#41: the preview restored the build-cache artifact built from main before this branch existed, and this repo has no jb clean . --html prune yet, so the restored copies are re-shipped. It says nothing about the PR; on the live site they clear on the settle policy recorded in the PR body.)

@mmcky
mmcky merged commit 4e83c69 into main Sep 1, 2026
3 checks passed
@mmcky
mmcky deleted the track-x-orphans branch September 1, 2026 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants