-
-
Notifications
You must be signed in to change notification settings - Fork 27
Fix grammar and spelling errors across lecture files #688
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
- 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]>
✅ Deploy Preview for taupe-gaufre-c4e660 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| 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 |
There was a problem hiding this comment.
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 *?
There was a problem hiding this comment.
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)$)
There was a problem hiding this comment.
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)
|
thanks @jstac this looks great. Just had one question, then I can merge. |
There was a problem hiding this 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 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@jstac please feel free to merge when ready. |
Summary
Files Changed
Test plan
🤖 Generated with Claude Code