Flip wave C2 + Track D to repointed — all four datasets - #99
Merged
Conversation
Both repoint PRs merged 2026-08-18: the three advanced.myst reads in QuantEcon/lecture-python-advanced.myst#375 and the four programming reads in QuantEcon/lecture-python-programming#612. Statuses flip to repointed, repoints entries filled, the landed-state consumer notes come off the two direct repos, and the dataBHS.mat annotation is deleted now that nothing reads the .mat (its successor manifest is dataBHS.csv.yml). The six translation-repo consumer notes on test_pwt.csv.yml stay: those repos read lecture-python-programming's raw URL until their sync-translations runs (fired by the #612 merge, in progress at this commit) land and their sites republish — which is what gates the source-copy deletion. Measured in both directions against post-merge consumer mains: repointed gives exit 0 with all five problem buckets empty; main's landed state gives exit 1 with four migration_inconsistencies, one per file. Verified the manifest edits touched nothing else: for each of the four manifests, the only top-level key differing from main is consumers (parsed comparison, not a patch eyeball). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Updates the migration tracker and dataset manifests to reflect that Wave C2 / Track D repoints have landed in the consuming lecture repos, and removes now-obsolete audit annotations and catalog warnings tied to the pre-repoint “landed” window.
Changes:
- Flip four datasets in
migration.ymlfromlanded→repointed, populatingrepointsmetadata for the two lecture-repo PRs. - Remove the
dataBHS.mataudit annotation now that the.matread has been eliminated by repointing. - Trim pre-repoint consumer notes from affected manifests and refresh
CATALOG.mdso it no longer advertises “reads its own copy” warnings for these now-repointed consumers.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/audit_annotations.yml | Deletes the dataBHS.mat annotation entry now that nothing reads the .mat. |
| migration.yml | Marks four datasets as repointed and records repoint PR metadata. |
| lectures/test_pwt.csv.yml | Removes now-stale per-consumer “landed” notes for source-repo consumers (translations remain noted). |
| lectures/fred_data.csv.yml | Removes pre-repoint per-consumer “landed” note from the advanced lecture consumer. |
| lectures/dataBHS.csv.yml | Removes pre-repoint per-consumer “landed” note from the advanced lecture consumer. |
| lectures/acs_data_summary.csv.yml | Removes pre-repoint per-consumer “landed” note from the advanced lecture consumer. |
| CATALOG.md | Regenerated catalog output to remove pre-repoint warning subnotes for these datasets. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
157
to
161
| consumers: | ||
| - repo: QuantEcon/lecture-python-programming | ||
| file: lectures/pandas.md | ||
| note: > | ||
| Reads the repo's own raw URL (one unsplit literal, pandas.md:172), not | ||
| this file — the repoint PR follows this one, and migration.yml records | ||
| the dataset as `landed` until it merges. The prose at pandas.md:156-176 | ||
| names the file and tabulates its columns, and must move with the read | ||
| (QuantEcon/data-lectures#91). | ||
| - repo: QuantEcon/lecture-python-programming | ||
| file: lectures/polars.md |
Comment on lines
178
to
180
| consumers: | ||
| - repo: QuantEcon/lecture-python-advanced.myst | ||
| file: lectures/risk_aversion_or_mistaken_beliefs.md |
Comment on lines
142
to
144
| consumers: | ||
| - repo: QuantEcon/lecture-python-advanced.myst | ||
| file: lectures/match_transport.md |
27 tasks
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.
Closes the red window opened by the two repoint merges of 2026-08-18 (QuantEcon/lecture-python-advanced.myst#375, QuantEcon/lecture-python-programming#612): the four wave C2 / Track D records flip to
repointedwith their repoints entries filled, the landed-state consumer notes come off the direct repos, and thedataBHS.matannotation is deleted now that nothing reads the .mat.The six translation-repo consumer notes on
test_pwt.csv.ymldeliberately stay — those repos read lecture-python-programming's raw URL until their sync-translations runs (auto-fired by the merge, in progress as this PR opens) land and their sites republish, which is what gates the source-copy deletion.Measured in both directions against post-merge consumer mains: this branch gives exit 0 with all five problem buckets empty; main's
landedstate gives exit 1 with exactly fourmigration_inconsistencies, one per file. The manifest edits are surgically scoped: for each manifest, the only top-level key differing from main isconsumers(parsed comparison — the anchor-on-consumers rule from the C1 flip).🤖 Generated with Claude Code