You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
After #144: point licence-review links at #35, record the pandas_panel translation readers, fix two stale consumer notes #149
The 2026-09-29 close-out review of the datasets migration found four small corrections. Each one changes a manifest note or consumers list, so CATALOG.md has to be regenerated with scripts/build_catalog.py. That regeneration would conflict with open #144, so these wait until it merges and then land as one PR.
The pandas_panel files have three unrecorded readers. As measured on 2026-09-29, lecture-python-programming.zh-cn, .fr and .fa read countries.csv, employ.csv and realwage.csv from this repo. The three manifests list only lecture-python-programming, lecture-python.myst and lecture-python.zh-cn. test_pwt.csv.yml already lists the same translations, so add them to all three consumers lists.
us_adult_heights.csv.yml has a stale consumer note. It says lecture-intro.zh-cn's prob_dist.md reads a local _static/lecture_specific/prob_dist/ copy. In fact it reads this repo's copy (github.com/QuantEcon/data-lectures/raw/main/lectures/us_adult_heights.csv, lines 75–76 of that file), and the local folder no longer exists. Drop the note.
Two more small corrections for the same PR, found on 2026-09-30 while reviewing #152 and the canary proof (#147). Neither needs a CATALOG.md regeneration.
manifest-schema.yml line 113, in the license.note comment: "In use by 16." is stale, since 27 manifests set license.note on main today. It is a count that drifts with every manifest, so delete the sentence rather than update the number. Lines 79, 84 and 92 carry the same kind of count; I did not check those.
.github/workflows/refresh-snapshots.yml line 162, the notify job: the issue body opens "The weekly sources-alive canary failed", which is wrong for a dispatched run (seen on refresh-snapshots: a dynamic snapshot's builder is failing #150). "The sources-alive canary failed" fits both triggers.
Activity
mmcky commentedon Sep 30, 2026
Two more small corrections for the same PR, found on 2026-09-30 while reviewing #152 and the canary proof (#147). Neither needs a
CATALOG.mdregeneration.manifest-schema.ymlline 113, in thelicense.notecomment: "In use by 16." is stale, since 27 manifests setlicense.noteonmaintoday. It is a count that drifts with every manifest, so delete the sentence rather than update the number. Lines 79, 84 and 92 carry the same kind of count; I did not check those..github/workflows/refresh-snapshots.ymlline 162, thenotifyjob: the issue body opens "The weekly sources-alive canary failed", which is wrong for a dispatched run (seen on refresh-snapshots: a dynamic snapshot's builder is failing #150). "The sources-alive canary failed" fits both triggers.