Skip to content

inflation_history: delete the local copies, now that the site is published - #825

Merged
mmcky merged 1 commit into
mainfrom
cleanup/inflation-history-phase2
Aug 6, 2026
Merged

inflation_history: delete the local copies, now that the site is published#825
mmcky merged 1 commit into
mainfrom
cleanup/inflation-history-phase2

Conversation

@mmcky

@mmcky mmcky commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Draft until publish-2026aug06b finishes deploying. Phase 2 of the repoint set opened in #824.

Why this is a separate PR

#824 moved inflation_history's URLs to data-lectures and deliberately kept these two files. This series publishes on a publish* tag, not on push to main, so at the moment #824 merged the already-published notebooks were still fetching lectures/datasets/chapter_3.xlsx and longprices.xls from this repo. Deleting them in the same PR would have 404'd every reader who downloaded the lecture or opened it in Colab, until someone happened to tag a publish.

That is not a hypothetical — it is what happened to set 1. mpd2020.xlsx was deleted in its repoint PR (#823), and the notebook at intro.quantecon.org returned 404 until publish-2026aug06 was tagged. The rendered HTML was fine throughout, because figures are baked at build time, so nothing reported it: not this repo's CI, not the data repo's audit, not a link checker over the live site.

The rule is written up as repoint rule 3 in QuantEcon/data-lectures, and QuantEcon/workspace-lectures#22 carries the general version for any coordinated change across the lecture repos.

Safety checks before deleting

Check Result
References to either file anywhere in this repo zero
lecture-wasm references (it used to fetch these by URL) zero since QuantEcon/lecture-wasm#53
Live notebooks read from data-lectures, once publish-2026aug06b lands
Bytes still available yes — both served from data-lectures, sha256 matching their manifests

Both files remain recoverable from git history, and their content is permanently available at data-lectures with a manifest recording source, licence, schema and provenance — which is more than this repo ever held for them.

Effect upstream

Orphan count in the data-lectures audit drops 34 → 32. No tracker change is needed: migration.yml already records both datasets as repointed (QuantEcon/data-lectures#43), since that status tracks what the lectures read, which changed in #824 and is unaffected by this cleanup.

Closes out set 2 of three for the Track A intro statics.

…ished

Phase 2 of the repoint set opened in #824. That PR moved the URLs and
deliberately KEPT these two files, because this series publishes on a tag
rather than on push to main -- so the already-published notebooks were still
fetching them, and deleting in the same PR would have 404'd every reader who
downloaded or opened the lecture in Colab.

publish-2026aug06b has since built main, so the live notebooks now read from
data-lectures and nothing points at these copies in either repo. Verified
before deleting: zero references remain here, and lecture-wasm -- which used
to fetch this repo's copies by URL -- references neither file since #53.

The two-phase split is repoint rule 3 in QuantEcon/data-lectures. It exists
because set 1 skipped it: mpd2020.xlsx was deleted in its repoint PR and the
notebook at intro.quantecon.org returned 404 until a publish was tagged.
Rendered HTML was fine throughout -- figures are baked at build time -- so
nothing reported the breakage.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Aug 6, 2026

Copy link
Copy Markdown

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 1ef445f
🔍 Latest deploy log https://app.netlify.com/projects/taupe-gaufre-c4e660/deploys/6a743dcd626386000841829c
😎 Deploy Preview https://deploy-preview-825--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

@github-actions
github-actions Bot temporarily deployed to pull request August 6, 2026 08:03 Inactive
@mmcky
mmcky marked this pull request as ready for review August 6, 2026 08:05
Copilot AI lite review requested due to automatic review settings August 6, 2026 08:05
@mmcky

mmcky commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

Gate cleared — publish-2026aug06b deployed, safe to merge

The publish completed successfully and the live site has moved. Verified just now against intro.quantecon.org/_notebooks/inflation_history.ipynb:

Check Result
Live notebook references to this repo's lectures/datasets/ zero
Live notebook now reads data-lectures/raw/main/lectures/{longprices.xls,chapter_3.xlsx}
The {download} link for chapter_3.xlsx also moved to data-lectures
Both served URLs 200, sha256 matching their manifests

So nothing — in this repo, in lecture-wasm, or on the published site — points at the two files this PR deletes. That is the condition repoint rule 3 exists to establish, and it is now true.

Marked ready. Merging takes the data-lectures orphan count 34 → 32 and closes set 2 of three for the Track A intro statics.

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 wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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