[util_rand_resp] Update reference style across lecture and correct typos - #621
Conversation
|
📖 Netlify Preview Ready! Preview URL: https://pr-621--sunny-cactus-210e3e.netlify.app (3c7ae02) 📚 Changed Lecture Pages: util_rand_resp |
|
Hi @mmcky, I think removing the reference in the subtitles fixed the build issue. (I started to use copilot after your workshop : )) |
|
📖 Netlify Preview Ready! Preview URL: https://pr-621--sunny-cactus-210e3e.netlify.app (d10dd4d) 📚 Changed Lecture Pages: util_rand_resp |
|
📖 Netlify Preview Ready! Preview URL: https://pr-621--sunny-cactus-210e3e.netlify.app (088f30f) 📚 Changed Lecture Pages: util_rand_resp |
|
📖 Netlify Preview Ready! Preview URL: https://pr-621--sunny-cactus-210e3e.netlify.app (647b9b5) 📚 Changed Lecture Pages: util_rand_resp |
|
📖 Netlify Preview Ready! Preview URL: https://pr-621--sunny-cactus-210e3e.netlify.app (14530ad) 📚 Changed Lecture Pages: util_rand_resp |
|
🤖 Status note for a future session — from a maintainer investigation on 2026-07-08 into why open-PR previews 404. Context only, not instructions. Netlify preview: https://pr-621--sunny-cactus-210e3e.netlify.app/ currently returns 404. Why previews are down (repo-wide findings)1. This branch is stale — 168 commits behind 2. The arviz failure was a red herring — do NOT pin arviz or rewrite plotting. A 2026-07-07 rebuild also failed in Recommended first step for this PRUpdate this branch to This PR touches: |
The last figure had been replaced by a copy of the Lanke figure, dropping
the convex truth border that the Chaudhuri-Mukerjee section is about and
orphaning f(x). Restore it, keeping only the intended fix (legend label
Z -> Q, matching the prose).
Other fixes:
- restore title case in the lecture title (style guide: title case for
lecture titles, sentence case for section headings)
- split the nested prf:assumption into two directives, so the numbering
doesn't collide with the directive's own "Assumption 1" label
- reword the three {cite:t} conversions that lost a possessive
("Warner (1965) original model" -> "the original model of ...")
- convert the remaining Anderson and Warner citations to {cite:t}
- shorten figure captions to the 5-6 word guideline
- fix duplicate {eq} reference (nine-a -> nine-b)
- undo code wrapping of lines already under 79 characters and replace
backslash continuations
- reference fig-truth-borders with numref; restore the concave/convex
sentence to its bullet; strip trailing whitespace
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
- drop figsize=(12, 10) from all five figures and use the default size
- convert plt.* to the fig, ax = plt.subplots() / ax.* pattern
- use lw=2 on line charts, and lw instead of linewidth
- set in-figure text in roman lower case ("truth telling", not italic
"Truth Telling") at the default size, and drop legend fontsize='large'
- lower-case the legend labels ("truth border")
- label the iso-variance family once instead of V1..V9, which cuts the
legend from twelve entries to four and lets the existing "high
variance" / "low variance" annotations carry the ordering; the prose
bullet that referred to V1..V9 is reworded to match
- shorten the truth-border legend labels, writing p for Pr(A|r_i), with
the shorthand introduced in the preceding sentence
- replace the Iso_Variance class with a plot_iso_variance_curves
function (the class held only two attributes and one plotting method)
- vectorize f() with np.where so the Chaudhuri-Mukerjee truth border is
computed in one place instead of being repeated inline
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Thanks @HumphreyYang — the reference-style conversion and the typo fixes are good, and moving to Since you're tied up, I've pushed the changes directly to the branch rather than leaving them as review comments. Summary of what I changed and why: One substantive issue. The plotting cell in the Chaudhuri and Mukerjee section had been replaced by a copy of the Lanke cell — identical except for the caption, the Style guide. A few things pulled back toward the manual:
Figure pass. I also took the figures the rest of the way, since they were already being touched:
Two calls there worth a look, as they go past mechanical fixes. The iso-variance curves were labelled Along the way All cells execute and I've checked the five figures render without overlap, but the preview is worth a look before merge. |
|
Thanks @HumphreyYang and @nisha617 , above comment posted by Claude. It lists additional style fixes. To save you time I'll go ahead and merge when this goes green. Everything looks good on a local build. |
📖 Netlify Preview Ready!Preview URL: https://pr-621--sunny-cactus-210e3e.netlify.app Commit: 📚 Changed LecturesBuild Info
|
✅ Translation sync completed (zh-cn)Target repo: QuantEcon/lecture-python.zh-cn
|
3 participants
This PR aims to update the reference style across the lecture and correct a few errors identified by me and @nisha617.