Skip to content

Remove six stale dataset files from the site (keep_files leftovers from Track X) - #27

Merged
mmcky merged 1 commit into
gh-pagesfrom
cleanup/track-x-stale-assets
Sep 29, 2026
Merged

mmcky merged 1 commit into
gh-pagesfrom
cleanup/track-x-stale-assets

Conversation

@mmcky

@mmcky mmcky commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

This PR removes six data files from gh-pages that the Track X orphan sweep deleted from the source (eb42856, 2026-09-01). The site still serves them because publish.yml deploys with peaceiris/actions-gh-pages and keep_files: true, which never deletes anything from this branch. No number of publishes will clear them, so this one-off removal is the only way they go.

Path on gh-pages Size
_static/lecture_specific/pandas/data/test_pwt.csv 793 B
_static/lecture_specific/pandas/data/ticker_data.csv 35,905 B
_static/lecture_specific/python_advanced_features/numbers.txt 16 B
_static/lecture_specific/python_advanced_features/test_table.csv 377,272 B
_static/lecture_specific/python_foundations/test_table.csv 377,272 B
_static/lecture_specific/python_foundations/us_cities.txt 169 B

Nothing reads them. This edition translates five lectures (functions, intro, matplotlib, numpy, python_by_example), and none of them references these files. Outside this repo they appear only in the migration's audit ledgers. The test_table.csv that python-programming.quantecon.org keeps as an exercise download lives on a different site and is untouched.

Merging redeploys the site, because Pages builds from this branch. keep_files cannot bring the files back, since they no longer exist in the source, and the next push to main deploys on top of this branch as usual.

Not in scope: the structural fix, which is to make the deploy remove deleted files while keeping the pr-* preview folders. That would also clear about 194 orphaned figures, but it changes publish.yml and deserves its own issue.

To verify after merge: the six URLs under https://quantecon.github.io/lecture-python-programming.ml/_static/lecture_specific/ should return 404, while /intro.html stays 200. The Pages edge caches for up to 10 minutes.

This is part of the site-clearance check in QuantEcon/workspace-lectures#40. The test_pwt.csv row is the last item of QuantEcon/workspace-lectures#54.

🤖 Generated with Claude Code

These six were deleted from lectures/_static in the Track X orphan sweep
(eb42856, 2026-09-01), but publish.yml deploys with peaceiris keep_files:
true, which never removes anything from gh-pages, so the site has kept
serving them byte-identical to the deleted blobs.

Nothing reads them. This edition translates five lectures (functions,
intro, matplotlib, numpy, python_by_example) and none references these
files; outside this repo they appear only in the migration's audit
ledgers. python-programming.quantecon.org's own test_table.csv, kept as an
exercise download, is a different site and is untouched.

Part of the site-clearance check in QuantEcon/workspace-lectures#40;
the test_pwt.csv row is the last item of QuantEcon/workspace-lectures#54.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings September 29, 2026 03:55

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.

Copilot review overview

🔵 Needs a closer look

It manually mutates the published gh-pages deployment branch, an operationally sensitive action that warrants final human confirmation despite the deletions being verified unreferenced.

Review effort: Balanced
Findings: None

What changed in this PR

This PR removes six stale dataset files from the gh-pages (built site) branch. These files were deleted from the lecture source during the "Track X orphan sweep," but the site still serves them because deployment uses peaceiris/actions-gh-pages with keep_files: true, which never prunes files from the published branch. A one-off manual removal is the only way to clear them. No lecture in the current edition (functions, intro, matplotlib, numpy, python_by_example) references any of these files.

Changes:

  • Delete two pandas data files (test_pwt.csv, ticker_data.csv).
  • Delete python_advanced_features leftovers (numbers.txt, test_table.csv).
  • Delete python_foundations leftovers (us_cities.txt, test_table.csv).

I verified that no main-site HTML page or searchindex.js references the deleted paths; the only test_pwt.csv mentions live in the pr-26/ preview folder and point to a remote data-lectures URL, so no on-site links break. The two large test_table.csv files are not shown in the diff (likely omitted due to size) but are confirmed removed from the branch.

File Description
_static/​lecture_specific/​pandas/​data/​test_pwt.csv Removes stale Penn World Table sample CSV; unreferenced by current lectures.
_static/​lecture_specific/​pandas/​data/​ticker_data.csv Removes stale ticker price CSV; unreferenced by current lectures.
_static/​lecture_specific/​python_advanced_features/​numbers.txt Removes stale sample text file; unreferenced.
_static/​lecture_specific/​python_advanced_features/​test_table.csv Removes stale sample CSV (not shown in diff, confirmed removed).
_static/​lecture_specific/​python_foundations/​test_table.csv Removes stale sample CSV (not shown in diff, confirmed removed).
_static/​lecture_specific/​python_foundations/​us_cities.txt Removes stale city-population text file; unreferenced.

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

@mmcky
mmcky merged commit 7973976 into gh-pages Sep 29, 2026
1 check passed
@mmcky
mmcky deleted the cleanup/track-x-stale-assets branch September 29, 2026 03:58
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