Skip to content

[olg] Address editorial suggestions from #506#752

Merged
jstac merged 2 commits into
mainfrom
minor-issues
May 28, 2026
Merged

[olg] Address editorial suggestions from #506#752
jstac merged 2 commits into
mainfrom
minor-issues

Conversation

@jstac

@jstac jstac commented May 27, 2026

Copy link
Copy Markdown
Contributor

Summary

olg.md — Address editorial suggestions from #506:

  • Clarify prose throughout: old agents' income, population normalization, first-order conditions, aggregate demand for capital, saving function, CRRA motivation
  • Fix bug in capital_demand (used global α instead of model.α)
  • Fix typos ("intital" → "initial", missing article, missing period)
  • Fix broken "redo our plot above" reference (plot was removed)
  • Remove unused ymin, ymax variables
  • Rename "Example: log preferences" → "Special case: log preferences"

lln_clt.md — Bundled for convenience (uncontroversial):

  • Add bullet clarifying what "independent" means in the coin flip example
  • Add Wikipedia link for IID

Closes #506

Test plan

  • Verify both lectures build without errors
  • Check that all plots render correctly (especially the CRRA supply/demand plot, which was affected by the capital_demand bug fix)

🤖 Generated with Claude Code

Clarify prose (old agents' income, population normalization, FOCs,
aggregate demand, saving function, CRRA motivation), fix capital_demand
bug (used global α instead of model.α), fix typos and broken reference.

Closes #506

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented May 27, 2026

Copy link
Copy Markdown

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit cf59d25
🔍 Latest deploy log https://app.netlify.com/projects/taupe-gaufre-c4e660/deploys/6a1750af4ba09400086f7f1b
😎 Deploy Preview https://deploy-preview-752--taupe-gaufre-c4e660.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented May 27, 2026

Copy link
Copy Markdown

@github-actions github-actions Bot temporarily deployed to pull request May 27, 2026 20:22 Inactive
@github-actions github-actions Bot temporarily deployed to pull request May 27, 2026 20:24 Inactive
@jstac jstac merged commit 0637b04 into main May 28, 2026
7 checks passed
@jstac jstac deleted the minor-issues branch May 28, 2026 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[olg] Unfinished editorial suggestions

1 participant