Skip to content

Retire the dashboard's high_dim_data references now the fold is complete #70

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mmcky merged 2 commits into from
Aug 11, 2026
Merged

Retire the dashboard's high_dim_data references now the fold is complete #70

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
2dcfa6f
Retire the dashboard's high_dim_data references now the fold is complete
mmcky Aug 11, 2026
8cedc1e
Describe `external` by what it excludes, not by whose repo it was
mmcky Aug 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 0 additions & 27 deletions scripts/audit_annotations.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,17 +28,6 @@ datasets:
A migration candidate — it is read from lecture-python.myst main by raw
URL, and the lecture documents every transformation and rebuilds the
post-2000Q4 continuation from FRED
SCF_plus_mini.csv:
description: SCF+ wealth/income survey extract
provenance: constructed-committed
note: built by generating_mini.md (executable MyST) in high_dim_data from SCF_plus.dta
SCF_plus_mini_no_weights.csv:
description: SCF+ extract, no survey weights
provenance: constructed-committed
note: >
variant of the SCF_plus_mini pipeline. Was pinned to feature branch
update_scf_noweights (critical flag in the 2026-07-15 audit); reads
high_dim_data main since lecture-python-intro#793
acs_data_summary.csv:
description: ACS occupation summary
provenance: constructed-lost
Expand All @@ -62,14 +51,6 @@ datasets:
description: Hyperinflation tables, Sargent "Ends of Four Big Inflations"
provenance: author-assembled
note: hand-transcribed by authors
cities_brazil.csv:
description: Brazilian city populations
provenance: author-assembled
note: manual download from worldpopulationreview.com, documented in high_dim_data README
cities_us.csv:
description: US city populations
provenance: author-assembled
note: manual download from worldpopulationreview.com, documented in high_dim_data README
dataBHS.mat:
description: US consumption/income series, MATLAB replication bundle
provenance: verbatim
Expand All @@ -80,14 +61,6 @@ datasets:
fig_3.xlsx:
description: French Revolution fiscal data
provenance: author-assembled
forbes-billionaires.csv:
description: Forbes billionaires list
provenance: constructed-committed
note: webscrape_forbes.ipynb in high_dim_data (Forbes API)
forbes-global2000.csv:
description: Forbes Global 2000 firm size
provenance: constructed-committed
note: webscrape_forbes.ipynb in high_dim_data (Forbes API)
fp.dta:
description: Treisman (2016) Russia billionaires replication data
provenance: verbatim
Expand Down
11 changes: 9 additions & 2 deletions scripts/render_audit.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,12 @@
"own-repo": ("p-own", "own-repo URL", "fetches a file committed in its own repo via a GitHub raw URL"),
"local-path": ("p-local", "local path", "pd.read_csv('…') relative path — no URL; breaks in Colab/download"),
"sibling": ("p-sib", "sibling repo URL", "fetches another lecture repo's committed copy by URL"),
"external": ("p-ext", "external data repo", "QuantEcon/high_dim_data via raw and media (LFS) hosts"),
# `external` is defined by what it EXCLUDES, and classify_url reaches it by
# two routes: a non-QuantEcon org (build_audit.py:180-181) and a QuantEcon
# repo that is none of this repo, the consuming repo, a legacy name or a
# scanned sibling (:190-191). Describe the exclusion, not whichever repo
# happens to be using it — this label has now been too narrow twice.
"external": ("p-ext", "external data repo", "reads a data file by URL from a GitHub repo outside the audited lecture set — any org"),
"external-web": ("p-ext", "external web host", "fetches a data file from a non-GitHub host by URL"),
"legacy": ("p-legacy", "legacy-repo URL", "fetches from the retired pre-MyST lecture-python repo"),
"embedded": ("p-embed", "%%file embedded", "written by the lecture itself, then read back"),
Expand Down Expand Up @@ -428,7 +433,9 @@ def what_changed(audit: dict) -> str:
(lecture-python-programming#576, lecture-python.myst#968) removed every live reference.</p></div>
<div class="finding ok"><b>Branch-pinned refs: 1 → 0.</b>
<p>The critical flag on <code>SCF_plus_mini_no_weights.csv</code> — pinned to a feature branch of
high_dim_data — was fixed by lecture-python-intro#793; it now reads <code>main</code>.</p></div>
the external repo it then lived in — was fixed by lecture-python-intro#793. That file has since been
folded into this repo (data-lectures#62) and every consumer repointed here, so the pin is doubly
retired.</p></div>
<div class="finding"><b>lecture-wasm joined the audit — and changed one fact.</b>
<p>The WASM mirror carries its own copies of intro's data files (all unread — its lectures fetch
intro's copies by URL), and its <code>short_path</code> fetches intro's committed
Expand Down
Loading