Record the #47 consumers: tom-econ370-2025 and python-lecture-sandpit.myst - #105
Merged
Conversation
….myst The sweep recorded in QuantEcon/workspace-lectures#47 found a fifth consumer of caron.npy/nom_balances.npy outside the #46 list -- tom-econ370-2025, a frozen course fork with a live Pages site reading its own blob-identical copies -- plus python-lecture-sandpit.myst as a public holder. Both accepted as-is rather than repointed; these manifest notes are the documentation half of that decision. One correction to the sweep's grouped phrasing, verified today: no sandpit file references its dataBHS.mat copy at all (holder only, orphan), and only sandpit's french_rev.md npy reads actually resolve. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Measured 2026-08-19 while writing the #47 record: the course fork's french_rev.md fetches all three xlsx through a stale base_url pointing at lecture-python-intro's deleted datasets/ copies -- 404 today, so the breakage is current, not prospective as QuantEcon/workspace-lectures#47 assumed. Its published Pages site is unaffected (committed _build); only re-execution breaks. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR updates dataset provenance documentation by recording additional “consumer” (and holder-only) repositories for the French Revolution .npy datasets and for dataBHS, then regenerates CATALOG.md from the manifests to keep the public registry consistent.
Changes:
- Add
tom-econ370-2025andpython-lecture-sandpit.mystentries to theconsumersblocks forcaron.npyandnom_balances.npy, explicitly noting these repos read/hold local copies instead of this repo’s published bytes. - Add a holder-only (non-reader) consumer entry for
python-lecture-sandpit.mystto thedataBHS.csvmanifest. - Regenerate
CATALOG.mdto reflect the updated manifest consumer metadata.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| lectures/nom_balances.npy.yml | Adds documented consumer entries for course-fork and sandpit repos (not repointed; local-copy divergence noted). |
| lectures/dataBHS.csv.yml | Adds holder-only sandpit entry documenting an orphan local copy (and its status). |
| lectures/caron.npy.yml | Mirrors nom_balances.npy.yml consumer documentation updates for the other .npy dataset. |
| CATALOG.md | Regenerated catalog reflecting the updated consumers metadata from manifests. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Owner decision on QuantEcon/workspace-lectures#47 (2026-08-19): the course fork will be archived, so its broken xlsx re-execution reads are accepted rather than repointed. Archived repos keep serving raw and Pages URLs, so nothing changes at archival time. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
Documentation half of the accept-and-document decision on QuantEcon/workspace-lectures#47, following the convention from #69: consumers lists carry local-copy readers with an inline note so divergence stays visible in the record.
caron.npy / nom_balances.npy: adds
tom-econ370-2025(frozen course fork, live Pages site, reads its own blob-identicaldatasets/copies atfrench_rev.md:715-716, stalebase_urlstill pointing at lecture-python-intro) andpython-lecture-sandpit.myst(public holder of_static/copies).dataBHS.csv: adds
python-lecture-sandpit.mystas a holder-only entry, with a correction verified today: contrary to the grouped phrasing in the sweep issue, no sandpit file references itsdataBHS.matcopy (orphan), and of the three sandpit french_rev variants onlyfrench_rev.md's_static/reads resolve —french_rev copy.mdandfrench_rev_tom.mdread adatasets/path that does not exist in that repo.CATALOG.md regenerated in the same commit (
build_catalog.py; CI diffs it).🤖 Generated with Claude Code