[hoist_failure] Revise fault-tree lecture and add exercises - #1067
Merged
Merged
Conversation
…rgent (1993), add exercises Split out of branch tom_mfg (commit 8bf3b2c). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Sep 26, 2026
Merged
Contributor
Author
|
TODO before merge: check this lecture against the QuantEcon style guide (writing, code, math, figures, references, doc links, admonitions). |
📖 Netlify Preview Ready!Preview URL: https://pr-1067--sunny-cactus-210e3e.netlify.app Commit: 📚 Changed LecturesBuild Info
|
- Replace hard-coded speed claims with qualitative wording, and drop the
convolution-vs-simulation speed comparison in the hoist_ex4 solution,
which runs the other way (~2 s for the 13 convolutions vs ~0.2 s for
10^6 draws)
- One sentence per paragraph in the hoist_ex1 solution
- Italic rather than bold for emphasis
- \Pr(...) -> \mathbb{P}\{...\}, and "i.i.d." -> "IID"
- Sentence case for the fast Fourier transform heading and its
mid-sentence use
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Contributor
|
@jstac Style guide check done ✅ Nice set of new exercises! The lecture was already in good shape (
Nothing changes the substance. Happy to adjust anything. |
Contributor
Author
|
Thanks @mmcky @thomassargent30 . Merging. |
Contributor
✅ Translation sync completed (zh-cn)Target repo: QuantEcon/lecture-python.zh-cn
|
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.
Revisions by @thomassargent30 to
hoist_failure:exercise-start/endblocks toexerciseand adds further exercisesNo new bib entries.
Split out of
tom_mfg(commit 8bf3b2c) so each lecture can be reviewed separately. Content is unchanged from Tom's commit; only the relevantquant-econ.bibentries and_toc.ymllines are included here.Merge order: independent — can be merged at any time, in any order relative to the other split PRs (#1067, #1068, #1069, #1070/#1071). Each PR adds its bib entries at a different place in
quant-econ.bib, so they do not conflict.🤖 Generated with Claude Code