-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
[money_inflation_nonlinear] Editorial Suggestions #446
Labels
Comments
longye-tian
added a commit
that referenced
this issue
May 23, 2024
This updates the lecture money_inflation_nonlinear.md according to #446. In particular, this includes ## Code - Remove the grid. - Change the legend for the [Laffer curve plot](https://intro.quantecon.org/money_inflation_nonlinear.html#laffer-curve-nonlinear) to "Laffer curve". Also, use default colors in `matplotlib`. ## Content - Bold face is reserved for definitions and italics are used for emphases. - Only the first word in the title should be capitalized. - Migrate some of "This lecture studies" to "We study ...". - Maintain consistent use of bar and overline. - (28.3) repeats (28.2). Consider removing (28.3) and adjusting the links to (28.2) if (28.3) is removed. - Define all the notations like $\pi$ and $m$. - "limiting values exists" -> exist - "...that studies a linear version of the model in this lecture`" -> "that studies a linear version of the model in this lecture." - In Equation (28.6) we have $x: x \ge 0$. Remove "x:", and put a curly bracket covering the equation after max to indicate the scope of max.
mmcky
pushed a commit
that referenced
this issue
Jun 17, 2024
* update_money_inflation_nonlinear This updates the lecture money_inflation_nonlinear.md according to #446. In particular, this includes ## Code - Remove the grid. - Change the legend for the [Laffer curve plot](https://intro.quantecon.org/money_inflation_nonlinear.html#laffer-curve-nonlinear) to "Laffer curve". Also, use default colors in `matplotlib`. ## Content - Bold face is reserved for definitions and italics are used for emphases. - Only the first word in the title should be capitalized. - Migrate some of "This lecture studies" to "We study ...". - Maintain consistent use of bar and overline. - (28.3) repeats (28.2). Consider removing (28.3) and adjusting the links to (28.2) if (28.3) is removed. - Define all the notations like $\pi$ and $m$. - "limiting values exists" -> exist - "...that studies a linear version of the model in this lecture`" -> "that studies a linear version of the model in this lecture." - In Equation (28.6) we have $x: x \ge 0$. Remove "x:", and put a curly bracket covering the equation after max to indicate the scope of max. * update_money_inflation_nonlinear Delete 'this lecture' before {doc}.
@jstac the last remaining item in this issue is
Do you think this is necessary? There is some discussion around the graph. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Suggestions by @longye-tian
Code
matplotlib
.Content
The text was updated successfully, but these errors were encountered: