Record the fitting_distributions lecture as a consumer - #32
Merged
Conversation
QuantEcon/lecture-python-intro#814 merged, adding a lecture that reads five datasets from this repo. - epl_match_goals.csv and japan_earthquakes.csv landed in #31 with empty consumers because the lecture was still being written. They now have their consumer, and their migration records move from `landed` to `repointed`. - ames_house_prices.csv, japan_deaths_by_age.csv and us_adult_heights.csv gain a second (third, for heights) consumer. Their migration records are unchanged: they were already `repointed` by #811, and #814 did not repoint anything -- the new lecture read the canonical URL from its first commit. The lecture also downloads AMZN through yfinance, for the section showing that a normal fit to monthly returns fails in the tails, so it gets an entry in the api section of audit_annotations.yml. Without it the strict scan fails on an unannotated live-API reference, as it did for observed_distributions in data-lectures#30. CATALOG.md regenerated. Part of QuantEcon/meta#336 Co-Authored-By: Claude Opus 5 (1M context) <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
1 participant
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.
QuantEcon/lecture-python-intro#814 merged, adding a lecture that reads five datasets from this repo.
epl_match_goals.csvlanded→repointedjapan_earthquakes.csvlanded→repointedames_house_prices.csvjapan_deaths_by_age.csvus_adult_heights.csvThe two that landed in #31 had empty
consumersbecause the lecture was still being written; they now have it.The other three keep their existing
migration.ymlrecords. They were alreadyrepointedby #811, and #814 did not repoint anything — the new lecture read the canonical URL from its first commit — so adding arepointsentry would misdescribe what happened. Only theconsumerslists change, which is what a correction needs in order to know what to rebuild.The yfinance annotation
The lecture downloads AMZN for the section showing that a normal fit to monthly returns fails in the tails, so it gets an
apientry inaudit_annotations.yml. Without it the strict scan fails on an unannotated live-API reference — exactly what happened toobserved_distributionsin #30, so this time it goes in with the repoint rather than after it.CATALOG.mdregenerated; the "Used by" column now shows the new lecture against all five.Part of QuantEcon/meta#336
🤖 Generated with Claude Code