Skip to content

Delete committed dataset orphans (Track X) - #11

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

Delete committed dataset orphans (Track X)#11
mmcky merged 1 commit into
mainfrom
track-x-orphans

Conversation

@mmcky

@mmcky mmcky commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Deletes six committed files that no lecture in this edition reads: pandas/data/ticker_data.csv, pandas/data/test_pwt.csv (the fourth copy left behind by the Track D repoint, found in QuantEcon/workspace-lectures#54), numbers.txt, us_cities.txt and both test_table.csv copies. This edition publishes no lecture that mentions any of them (pandas.html is 404 on its site), yet all six are served — the baked-_build behaviour QuantEcon/workspace-lectures#41 describes.

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 / %%writefile before being read. The translation sync is .md-only, so nothing will re-add them.

Published surface and settle. All six serve at 200 on quantecon.github.io/lecture-python-programming.ml/ today. This repo publishes on push to main but restores a cached _build with no prune, so they clear on the next build that follows a clean cache.

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/pandas/data/ticker_data.csv 200, 35,905 B
/_static/lecture_specific/pandas/data/test_pwt.csv 200, 793 B
/_static/lecture_specific/python_advanced_features/numbers.txt 200, 16 B
/_static/lecture_specific/python_advanced_features/test_table.csv 200, 377,272 B
/_static/lecture_specific/python_foundations/test_table.csv 200, 377,272 B
/_static/lecture_specific/python_foundations/us_cities.txt 200, 169 B

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

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.

This edition publishes no lecture that mentions any of these files; test_pwt.csv is the fourth copy left behind by the Track D repoint (QuantEcon/workspace-lectures#54).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 1, 2026 00:08

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes lecture-specific static data files that are no longer referenced by any published lecture content, reducing the published surface area and preventing orphaned datasets from being served from the built site.

Changes:

  • Delete unused python_foundations and python_advanced_features text fixtures from lectures/_static/lecture_specific/.
  • Delete unused pandas CSV fixtures from lectures/_static/lecture_specific/pandas/data/.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.

File Description
lectures/_static/lecture_specific/python_foundations/us_cities.txt Deletes an unused city-population text fixture.
lectures/_static/lecture_specific/python_advanced_features/numbers.txt Deletes an unused numbers text fixture.
lectures/_static/lecture_specific/pandas/data/ticker_data.csv Deletes an unused ticker time-series CSV fixture.
lectures/_static/lecture_specific/pandas/data/test_pwt.csv Deletes an unused PWT sample CSV fixture.

💡 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
Contributor

📖 HTML build - view preview (7820b62)

@mmcky

mmcky commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Validation (2026-09-01, Pages preview …/lecture-python-programming.ml/pr-11/): all six deleted paths → 404 (ticker_data.csv, test_pwt.csv, numbers.txt, both test_table.csv, us_cities.txt); intro.html → 200 (28,552 B); a never-existed path → 404, so the 404s are genuine. No lecture published in this edition mentions any of the six (its pandas.html is not built). Build log: 0 warnings mentioning a deleted basename.

@mmcky
mmcky merged commit eb42856 into main Sep 1, 2026
5 checks passed
github-actions Bot added a commit that referenced this pull request Sep 1, 2026
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