Skip to content
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

Open
11 of 12 tasks
HumphreyYang opened this issue May 19, 2024 · 1 comment
Open
11 of 12 tasks

[money_inflation_nonlinear] Editorial Suggestions #446

HumphreyYang opened this issue May 19, 2024 · 1 comment
Assignees

Comments

@HumphreyYang
Copy link
Collaborator

HumphreyYang commented May 19, 2024

Suggestions by @longye-tian

Code

  • Remove the grid.
  • Change the legend for the Laffer curve plot 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."
  • Give a theoretical explanation for the patterns in Figure 28.2.
  • 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.
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}.
@mmcky
Copy link
Contributor

mmcky commented Jul 1, 2024

@jstac the last remaining item in this issue is

  • Give a theoretical explanation for the patterns in Figure 28.2.

Do you think this is necessary? There is some discussion around the graph.

@mmcky mmcky assigned jstac and mmcky Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants