Retire the last references to the retired lecture-python repo#968
Merged
Conversation
Nothing in this repo points at QuantEcon/lecture-python any more. pandas_panel read realwage.csv and countries.csv from that repo while reading employ.csv — the third file in the same trio, in the same lecture — from this repo's own copy. All three copies are already committed under _static/lecture_specific/pandas_panel/, so two of the three reads were fetching from a repo last touched in January 2022 for no reason. Now all three are consistent and use the same URL form. troubleshooting pointed readers at that same dead repo's issue tracker to report problems with this one. Repointed at this repo's issues, matching what the sibling lecture repos do. Verified byte-identical before repointing: each own-repo copy has the same sha256 as the legacy file it replaces, and the new URLs serve those same bytes, so lecture output cannot change. Note this is deliberately an interim step. P2 of the datasets pilot migrates this trio to QuantEcon/data-lectures and will rewrite these three lines again — but it rewrites them whether they say "legacy" or "own-repo" today, so this costs P2 nothing and stops the lecture depending on an unmaintained repo in the meantime. Part of QuantEcon/meta#337 See QuantEcon/meta#336 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the remaining references to the retired QuantEcon/lecture-python repository by updating lecture content to consistently point at this repository’s own assets and issue tracker.
Changes:
- Update
pandas_paneldataset URLs to loadrealwage.csvandcountries.csvfromQuantEcon/lecture-python.myst(matching the existingemploy.csvpattern). - Update the troubleshooting guide’s “Reporting an Issue” link to point to
QuantEcon/lecture-python.myst/issues.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| lectures/pandas_panel.md | Repoints legacy raw GitHub URLs to this repo’s lectures/_static/lecture_specific/pandas_panel/ copies for consistent data loading. |
| lectures/troubleshooting.md | Fixes issue-tracker link to direct users to the correct repository. |
📖 Netlify Preview Ready!Preview URL: https://pr-968--sunny-cactus-210e3e.netlify.app Commit: 📚 Changed LecturesBuild Info
|
mmcky
added a commit
to QuantEcon/lecture-python-programming
that referenced
this pull request
Jul 16, 2026
* Retire the last references to the retired lecture-python repo pandas_panel read all three of realwage.csv, countries.csv and employ.csv from QuantEcon/lecture-python — a repo last touched in January 2022 — even though this repo already has its own committed copies of all three under _static/lecture_specific/pandas_panel/. Now it reads its own. Verified byte-identical before repointing: each own-repo copy has the same sha256 as the legacy file it replaces, and the new URLs serve those same bytes, so lecture output cannot change. Mirrors QuantEcon/lecture-python.myst#968, which did the same for the myst series. With this, no English lecture source in the Python family points at the retired repo except lecture-stats. Interim by design: P2 of the datasets pilot migrates this trio to QuantEcon/data-lectures and will rewrite these three lines again — but it rewrites them either way, so this costs P2 nothing and stops the lecture depending on an unmaintained repo meanwhile. Part of QuantEcon/meta#337 See QuantEcon/meta#336 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> * Use this repo's existing raw.githubusercontent URL form Addresses Copilot review on #576. The repoint used github.com/.../raw/refs/heads/main/..., which introduced a third URL form into a repo that already had one: pandas.md and the other own-data reads all use raw.githubusercontent.com/.../main/.... Now all five own-data URLs match. Both forms fetch these files correctly — checked, they parse to identical frames — so this is about consistency, not correctness. Copilot's stated reason (that the redirect makes /raw/ less reliable for read_csv) does not hold; the redirect is real but pandas follows it. The one thing /raw/ buys is LFS safety: raw.githubusercontent.com serves pointer text rather than data for LFS-tracked files. That does not apply here — this repo has no .gitattributes, nothing is LFS — and these three files move to data-lectures under pilot P2 regardless, which is where the LFS-safe form actually matters. Part of QuantEcon/meta#337 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This was referenced Jul 16, 2026
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.
After this, nothing in this repo references
QuantEcon/lecture-pythonany more — the repo it was migrated from, last touched 2022-01-21. Follows #966, which did theolshalf.pandas_panel— two of three reads were still remoteThe lecture read
realwage.csvandcountries.csvfrom the retired repo while readingemploy.csv— the third file of the same trio, in the same lecture — from this repo's own copy. All three copies are already committed under_static/lecture_specific/pandas_panel/, so two of the reads were reaching for a four-year-dead repo for no reason. Now all three are consistent and use the same URL form.troubleshooting— sent readers to the wrong issue trackerIt asked readers to report problems with this lecture series by opening an issue on
QuantEcon/lecture-python— which still has issues enabled, so reports would land in a repo nobody watches. Repointed at this repo's tracker, matching the sibling lecture repos.Verification
realwage.csvcountries.csvemploy.csvsha256-compared before repointing, so lecture output cannot change. All three also confirmed readable over https from the new URLs.
This is deliberately interim — and it costs P2 nothing
The plan in QuantEcon/workspace-lectures#14 says to skip
pandas_panelbecause pilot P2 (QuantEcon/meta#338) migrates this trio toQuantEcon/data-lectures, making an interim repoint churn. That reasoning is sound but the arithmetic changed:pandas_paneltrio is consumed by 6 source repos / 17 references — the translations (lecture-python.zh-cn,lecture-python-programming.{fa,fr,zh-cn}) each carry their own copy of the lecture and were never counted.Happy to close this and wait for P2 if you'd rather hold the line on the original plan — it is two lines either way.
Part of QuantEcon/meta#337
See QuantEcon/meta#336