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

Update money_inflation.md #440

Merged
merged 2 commits into from
May 9, 2024
Merged

Update money_inflation.md #440

merged 2 commits into from
May 9, 2024

Conversation

longye-tian
Copy link
Collaborator

Dear John @jstac ,

I have update lecture money_inflation.md according to #437. In particular, it involves,

Code

  • Remove the grid in the plot
  • Use solve to calculate the inverse

Content

  • Change bold non-definition words to italics
  • $m_{t+1}^d/p_t =\gamma_1 - \gamma_2 \frac{p_{t+1}}{p_t}, \quad t \geq 0$ -> $\frac{m_{t+1}^d}{p_t} =\gamma_1 - \gamma_2 \frac{p_{t+1}}{p_t}, \quad t \geq 0$ where $\gamma_1$ and $\gamma_2$ are positive parameters.
  • Remove the repetition of definition of $b_t$ and $R_{t-1}$
  • Remove the $()$ in 26.6
  • Check capitalization for titles
  • gross rates of return on currency -> gross rates of return on currency and real balance
  • $\bar R < 1$ -> $\bar R \le 1$
  • $\bar R \in [\frac{\gamma2}{\gamma1}, 1] \equiv [\underline R, \overline R]$ -> $\bar R \in [\frac{\gamma_2}{\gamma_1}, 1] \equiv [\underline R, \overline R]$
  • $R_t \in \bar R = [\underline R, \overline R], \quad t \geq 0$ -> $R_t \in [\underline R, \overline R], \quad t \geq 0$
  • two values of $R$ -> Two steady-state values of $R$
  • $R_l$ ->$R_\ell$
  • An equilibrium sequences are not unique -> equilibrium sequences are not unique
  • Remove the repetitive
$$\begin{aligned}
b_t & = b_{t-1} R_{t-1} + g \cr
R_t^{-1} & = \frac{\gamma_1}{\gamma_2} - \gamma_2^{-1} b_t
\end{aligned}$$
  • "code plot outcomes" -> "code plot to outcomes"
  • Set $ \Lambda= \begin{bmatrix} {R_l}^{-1} & 0 \cr 0 & {R_u}^{-1} \end{bmatrix}$
  • $Q y_0 = y^_0 = \begin{bmatrix} y^{1,0} \cr 0 \end{bmatrix}$ -> $Q^{-1} y_0 = y^_0 = \begin{bmatrix} y^{1,0} \cr 0 \end{bmatrix}$.

Best 👻
Longye

This updates lecture money_inflation.md according to #437. In particular, it involves,

## Code
- Remove the grid in the plot
- Use `solve` to calculate the inverse

## Content

- Change bold non-definition words to italics

-  $m_{t+1}^d/p_t =\gamma_1 - \gamma_2 \frac{p_{t+1}}{p_t}, \quad t \geq 0$  -> $\frac{m_{t+1}^d}{p_t} =\gamma_1 - \gamma_2 \frac{p_{t+1}}{p_t}, \quad t \geq 0$ where $\gamma_1$ and $\gamma_2$ are positive parameters.
- Remove the repetition of definition of $b_t$ and $R_{t-1}$
- Remove the $()$ in 26.6
- Check capitalization for titles
- gross rates of return on currency  -> gross rates of return on currency and real balance
- $\bar R < 1$ -> $\bar R \le 1$
- $\bar R \in [\frac{\gamma2}{\gamma1}, 1]
\equiv [\underline R, \overline R]$ -> $\bar R \in [\frac{\gamma_2}{\gamma_1}, 1]  \equiv [\underline R, \overline R]$
- $R_t \in \bar R  = [\underline R, \overline R],  \quad t \geq 0$ -> $R_t  \in [\underline R, \overline R],  \quad t \geq 0$
- two values of $R$ -> Two steady-state values of $R$
- $R_l$ ->$R_\ell$
- An equilibrium sequences are not unique -> equilibrium sequences are not unique
- Remove the repetitive
```
$$\begin{aligned}
b_t & = b_{t-1} R_{t-1} + g \cr
R_t^{-1} & = \frac{\gamma_1}{\gamma_2} - \gamma_2^{-1} b_t
\end{aligned}$$
```
- "code plot outcomes" -> "code plot to outcomes"
- Set $ \Lambda= \begin{bmatrix} {R_l}^{-1} & 0 \cr
                0 & {R_u}^{-1} \end{bmatrix}$
- $Q y_0 =  y^*_0 = \begin{bmatrix} y^*_{1,0} \cr 0 \end{bmatrix}$ -> $Q^{-1} y_0 =  y^*_0 = \begin{bmatrix} y^*_{1,0} \cr 0 \end{bmatrix}$.
@longye-tian longye-tian requested a review from jstac May 8, 2024 04:25
Copy link

netlify bot commented May 8, 2024

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit c56e46f
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/663b4b18820b7d0008efe349
😎 Deploy Preview https://deploy-preview-440--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 site configuration.

Copy link

github-actions bot commented May 8, 2024

@github-actions github-actions bot temporarily deployed to pull request May 8, 2024 04:33 Inactive
@mmcky
Copy link
Contributor

mmcky commented May 8, 2024

@longye-tian will this close #437?

@jstac this is ready for review

@longye-tian
Copy link
Collaborator Author

@longye-tian will this close #437?

@jstac this is ready for review

Hi @mmcky, Some comments remain unresolved, but I think I need to submit another pull request for that as it might related to the content.

Best,
Longye

@@ -393,7 +388,7 @@ $$ (eq:method1)
**Remark 1:** method 1 uses an indirect approach to computing an equilibrium by first computing an equilibrium $\{R_t, b_t\}_{t=0}^\infty$ sequence and then using it to back out an equilibrium $\{p_t, m_t\}_{t=0}^\infty$ sequence.


**Remark 2:** notice that method 1 starts by picking an **initial condition** $R_0$ from a set $[\frac{\gamma_2}{\gamma_1}, R_u]$. An equilibrium $\{p_t, m_t\}_{t=0}^\infty$ sequences are not unique. There is actually a continuum of equilibria indexed by a choice of $R_0$ from the set $[\frac{\gamma_2}{\gamma_1}, R_u]$.
**Remark 2:** notice that method 1 starts by picking an **initial condition** $R_0$ from a set $[\frac{\gamma_2}{\gamma_1}, R_u]$. Equilibrium $\{p_t, m_t\}_{t=0}^\infty$ sequences are not unique. There is actually a continuum of equilibria indexed by a choice of $R_0$ from the set $[\frac{\gamma_2}{\gamma_1}, R_u]$.
Copy link
Contributor

Choose a reason for hiding this comment

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

@mmcky , is there a suitable environment for these remarks?

Copy link
Contributor

Choose a reason for hiding this comment

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

@jstac there is a remark admonition available in sphinx-proof but they are pretty in your face styling.

Screenshot 2024-05-09 at 10 03 48 AM

I think it would be good to migrate to them and them update the styling perhaps?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, +1 on that @mmcky . These admonitions need simple styling.

Then definitions, theorems, etc. can have stronger styles, so they stand out.

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks @jstac I have opened an issue #442 to track

lectures/money_inflation.md Outdated Show resolved Hide resolved
@jstac
Copy link
Contributor

jstac commented May 8, 2024

Many thanks @longye-tian, much appreciated!

Just some very minor comments above that need to be addressed.

update bold to italics and typos
@github-actions github-actions bot temporarily deployed to pull request May 8, 2024 09:59 Inactive
@jstac
Copy link
Contributor

jstac commented May 8, 2024

Many thanks @longye-tian

@mmcky , please see my comments above and merge when ready.

@mmcky mmcky merged commit ac1d5d3 into main May 9, 2024
6 checks passed
@mmcky mmcky deleted the update_money_inflation branch May 9, 2024 04:03
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.

3 participants