Skip to content

Conversation

@jstac
Copy link
Contributor

@jstac jstac commented Dec 26, 2025

Summary

  • Fixed 10 grammar and spelling errors across 8 lecture markdown files
  • Corrected article usage, subject-verb agreement, preposition usage, and word duplications
  • Improved word choice for clarity
  • Fixed markdown formatting issues

Files Changed

  • ar1_processes.md: Fixed missing asterisk in math notation ($\psi^*(x)$)
  • business_cycle.md: Fixed preposition usage ("fluctuations in GDP" and "confidence in their")
  • cagan_adaptive.md: Fixed subject-verb agreement ("smoothly declines")
  • long_run_growth.md: Removed redundant "across" and changed "non-continuous" to "incomplete"
  • lp_intro.md: Fixed markdown formatting for italics
  • supply_demand_heterogeneity.md: Fixed article usage ("A simple example")
  • tax_smooth.md: Removed duplicate "chooses" and extra "use"
  • unpleasant.md: Clarified pronoun reference ("that lecture")

Test plan

  • Review each changed file to verify corrections maintain intended meaning
  • Build documentation to ensure no formatting issues introduced
  • Verify all mathematical notation renders correctly

🤖 Generated with Claude Code

- ar1_processes.md: Fix missing asterisk in math notation
- business_cycle.md: Fix preposition usage and grammar
- cagan_adaptive.md: Fix subject-verb agreement
- long_run_growth.md: Remove redundant word and improve word choice
- lp_intro.md: Fix markdown formatting
- supply_demand_heterogeneity.md: Fix article usage
- tax_smooth.md: Remove duplicate word and extra word
- unpleasant.md: Clarify pronoun reference

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@netlify
Copy link

netlify bot commented Dec 26, 2025

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 965aac8
🔍 Latest deploy log https://app.netlify.com/projects/taupe-gaufre-c4e660/deploys/694fba720ddf470008498038
😎 Deploy Preview https://deploy-preview-688--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.

@github-actions
Copy link

github-actions bot commented Dec 26, 2025

@github-actions github-actions bot temporarily deployed to pull request December 26, 2025 22:24 Inactive
@github-actions github-actions bot temporarily deployed to pull request December 26, 2025 22:25 Inactive
And we can use a theoretical AR(1) model to calculate the right hand side.

If $\frac{1}{m} \sum_{t = 1}^m X_t$ is not close to $\psi^(x)$, even for many
If $\frac{1}{m} \sum_{t = 1}^m X_t$ is not close to $\psi^*(x)$, even for many
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jstac what is the meaning of this leading *?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ar1_processes.md: Fixed missing asterisk in math notation ($\psi^*(x)$)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah! yes it is ψ*(x)

@mmcky
Copy link
Contributor

mmcky commented Dec 26, 2025

thanks @jstac this looks great. Just had one question, then I can merge.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes 10 grammar and spelling errors across 8 lecture markdown files, improving readability and correctness of the documentation.

  • Corrected grammar issues including article usage, subject-verb agreement, and preposition usage
  • Fixed markdown formatting inconsistencies for proper rendering
  • Corrected mathematical notation to ensure accurate representation

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
lectures/unpleasant.md Clarified pronoun reference by adding "that" before "lecture"
lectures/tax_smooth.md Removed duplicate words "use" and "chooses" for better clarity
lectures/supply_demand_heterogeneity.md Fixed article usage from "An" to "A" before "simple example"
lectures/lp_intro.md Corrected markdown italic formatting with proper asterisk placement
lectures/long_run_growth.md Removed redundant "across" and changed "non-continuous" to "incomplete" for accuracy
lectures/cagan_adaptive.md Fixed subject-verb agreement by changing "decline" to "declines"
lectures/business_cycle.md Corrected prepositions from "over" to "in" and "towards" to "in"
lectures/ar1_processes.md Fixed mathematical notation by adding missing asterisk in $\psi^*(x)$

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mmcky mmcky added the ready label Dec 27, 2025
@mmcky
Copy link
Contributor

mmcky commented Dec 27, 2025

@jstac please feel free to merge when ready.

@github-actions github-actions bot temporarily deployed to pull request December 27, 2025 11:00 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants