Skip to content

Delete committed dataset orphans (Track X) - #97

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

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

Conversation

@mmcky

@mmcky mmcky commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Deletes nine committed files that no lecture in this edition reads: the pandas_panel trio (countries.csv, employ.csv, realwage.csvpandas_panel.md reads data-lectures), pandas/data/ticker_data.csv (unreferenced), pandas/data/test_pwt.csv (the copy left behind by the Track D repoint — QuantEcon/workspace-lectures#54's row for this edition), numbers.txt and us_cities.txt (written by %%file / %%writefile), and both test_table.csv copies — unlike the English repo, this edition's exercise prose links to the English repo's python_advanced_features/test_table.csv URL, so its own copies are orphans.

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 nine serve at 200 on quantecon.github.io/lecture-python-programming.zh-cn/ today. This repo publishes on a publish* tag and has no _build/html prune, so they clear on the settle policy — next Monday clean rebuild, then the next publish. That publish is the one QuantEcon/workspace-lectures#54 already asks for (it also refreshes the jax install cell on the rendered pages), so merging this and tagging clears both at once.

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_panel/countries.csv 200, 49,605 B
/_static/lecture_specific/pandas_panel/employ.csv 200, 1,725,234 B
/_static/lecture_specific/pandas_panel/realwage.csv 200, 121,589 B
/_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.

Unlike the English repo, this translation's exercise prose links to the English repo's python_advanced_features/test_table.csv URL, so both local test_table.csv copies are orphans here. test_pwt.csv is the 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
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR removes lecture-specific static dataset files that are no longer referenced by the zh-cn lectures (data is either pulled from QuantEcon/data-lectures URLs or created at runtime via %%file / %%writefile). This reduces repo size and eliminates “committed orphan” assets that are currently published under _static/lecture_specific/.

Changes:

  • Delete the pandas_panel static CSV trio (countries.csv, employ.csv, realwage.csv) that lectures now source from QuantEcon/data-lectures.
  • Delete unreferenced pandas/data CSVs (ticker_data.csv, test_pwt.csv) from _static/lecture_specific/.
  • Delete lecture-generated text/CSV artifacts (numbers.txt, us_cities.txt, and both test_table.csv copies) that are written by the lectures themselves.

Reviewed changes

Copilot reviewed 5 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
lectures/_static/lecture_specific/pandas_panel/countries.csv Remove orphaned static dataset copy.
lectures/_static/lecture_specific/pandas_panel/employ.csv Remove orphaned static dataset copy.
lectures/_static/lecture_specific/pandas_panel/realwage.csv Remove orphaned static dataset copy.
lectures/_static/lecture_specific/pandas/data/ticker_data.csv Remove unreferenced static CSV.
lectures/_static/lecture_specific/pandas/data/test_pwt.csv Remove orphaned static CSV copy (lecture uses remote URL).
lectures/_static/lecture_specific/python_advanced_features/numbers.txt Remove lecture-generated file that’s created via %%file in-content.
lectures/_static/lecture_specific/python_advanced_features/test_table.csv Remove orphaned static CSV (lecture creates local test_table.csv via %%file).
lectures/_static/lecture_specific/python_foundations/test_table.csv Remove orphaned static CSV duplicate.
lectures/_static/lecture_specific/python_foundations/us_cities.txt Remove lecture-generated file that’s created via %%writefile / %%file in-content.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mmcky

mmcky commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Validation (2026-09-01): this edition's CI has no preview step, so the channels are the build and the sources. Build Project [using jupyter-book] succeeded on the branch; its log has 0 warnings or errors mentioning any of the nine deleted basenames. In the sources every former reader points at data-lectures: pandas_panel.md lines 89/191/459 (realwage/countries/employ), pandas.md line 184 and polars.md lines 177/360/435 (test_pwt.csv); debugging.md, python_essentials.md and python_advanced_features.md write numbers.txt, us_cities.txt and test_table.csv with %%file/%%writefile; the exercise link at python_advanced_features.md:1199 points at the English repo's kept copy. The identical set was preview-verified on the fr and fa PRs, whose sources match line for line.

@mmcky
mmcky merged commit fdabab3 into main Sep 1, 2026
3 checks passed
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