Delete the three migrated wave-C2 copies (Track C) - #376
Merged
Conversation
All three now serve from data-lectures: the repoint merged in #375 and published in publish-2026aug18, whose notebooks and the .notebooks mirror both carry the new URLs (verified by basename sweep — one data-lectures read per notebook, zero refs/heads or loadmat residue). fred_data.csv was the live runtime-fetch case, which is why deletion waited for the publish. Deletion-time re-derivation, 2026-08-18: a fresh authenticated Trees sweep over all 278 org repos (278 covered, 0 skipped) finds no other reader of these paths — every other holder of these basenames owns its own copy (lecture-dp / lecture-dp.monorepo / 2026-tom-course for the ACS file; lecture-tools-techniques / lecture-mapping / python-lecture-sandpit.myst for the .mat, with tools-techniques reading its own copy, tracked in QuantEcon/lecture-tools-techniques#11). match_transport's lecture_specific directory empties with this change; risk_aversion keeps its three figure PNGs. The published site keeps serving the two _static copies until the next cache rebuild plus publish, per the settle policy — the re-audit booked on QuantEcon/workspace-lectures#40 verifies that; CI previews stop serving them immediately now that the prune (#374) is in. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
✅ Deploy Preview for lustrous-melomakarona-3ee73e ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull request overview
Removes local copies of wave-C2 datasets that have been migrated to and are now consumed from QuantEcon/data-lectures, reducing duplicated data blobs in this lecture repository.
Changes:
- Deletes
fred_data.csvfrom the risk-aversion lecture-specific_staticdirectory. - Deletes
acs_data_summary.csvfrom the match-transport lecture-specific_staticdirectory.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| lectures/_static/lecture_specific/risk_aversion_or_mistaken_beliefs/fred_data.csv | Deletes the local dataset copy (no remaining in-repo references found). |
| lectures/_static/lecture_specific/match_transport/acs_data_summary.csv | Deletes the local dataset copy (lecture now reads acs_data_summary.csv from data-lectures/raw/main). |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Contributor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes the local copies of the three wave-C2 datasets now served from data-lectures (landed in QuantEcon/data-lectures#98, repointed in #375, published in
publish-2026aug18).The publish-before-delete gate is satisfied and was verified on the live site: each of the three published notebooks carries exactly one
data-lectures/raw/mainread and zerorefs/headsorloadmatresidue (basename sweep of the served.ipynbfiles, with the three figure PNGs inrisk_aversion's directory identified as the only remaining_staticreferences — they stay). The.notebooksmirror self-healed with the same bytes. This matters most forfred_data.csv, which is fetched live at cell execution — with the published notebooks repointed, deleting the old blob breaks nothing.Deletion-time consumer re-derivation, 2026-08-18: fresh authenticated Trees sweep over all 278 org repos, 278 covered and 0 skipped. No repo reads these paths from this repo — every other basename holder owns its own copy (the ACS orphans in lecture-dp / lecture-dp.monorepo / 2026-tom-course; the .mat copies in lecture-tools-techniques / lecture-mapping / python-lecture-sandpit.myst, with the tools-techniques mirror repoint tracked in QuantEcon/lecture-tools-techniques#11).
The published site keeps serving the two
_staticcopies until the natural weekly cache rebuild plus the next publish (the settle policy); the re-audit booked on QuantEcon/workspace-lectures#40 verifies the clearance. CI previews stop serving them immediately, now that #374's prune is in both workflows — this PR's own preview is the first real production test of that.🤖 Generated with Claude Code