Skip to content

Conversation

@HumphreyYang
Copy link
Member

No description provided.

@netlify
Copy link

netlify bot commented Jun 1, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit d086978
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/647c15ee85384f0008a65745
😎 Deploy Preview https://deploy-preview-206--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 settings.

@github-actions
Copy link

github-actions bot commented Jun 1, 2023

@jstac
Copy link
Contributor

jstac commented Jun 2, 2023

Add link to pv in "Present value formulas are at the core of the models."

Present value formulas are...

@jstac
Copy link
Contributor

jstac commented Jun 2, 2023

I suggest we drop arrows to be consistent with other lectures ($c$ or $\{c_t\}$ is the vector and $c_t$ is an element of the vector).

@jstac
Copy link
Contributor

jstac commented Jun 2, 2023

Code lines should be 80 characters or less:

ConsumptionSmoothing = namedtuple( "ConsumptionSmoothing", ["R", "g1", "g2", "β_seq", "T"]) 
@jstac
Copy link
Contributor

jstac commented Jun 2, 2023

"Multiplying both sides by inverse of the matrix on the left provides the solution"

It's not so obvious that the inverse is in fact the inverse. Maybe add an unsolved exercise here that calls the matrix $A$ and the inverse $A^{-1}$ and asks the reader to check that $A A^{-1} = I$.

@jstac
Copy link
Contributor

jstac commented Jun 2, 2023

"21.2.2. Second order difference equation"

I think we should state the equation before we show the matrix version.

@jstac
Copy link
Contributor

jstac commented Jun 2, 2023

"Note to John: We can do some fun simple experiments with these variations – we can use graphs to show that, when
and starting from the smooth path, all nontrivial budget-feasible variations lower welfare according to the criterion above."

I think this can be removed now?

@jstac
Copy link
Contributor

jstac commented Jun 2, 2023

Nice job @HumphreyYang, many thanks! Please see the minor comments above.

@realjiachengli We plan to go live and include this lecture on the 5th. If you have edits to add please do so before then, if you can.

@HumphreyYang
Copy link
Member Author

Many thanks @jstac ,

I have updated the PR to reflect your review. I also notice some other very minor typos. Please kindly review my edits.

Add link to pv in "Present value formulas are at the core of the models."

Present value formulas are...

I will add this once the present value lecture is merged.

If it looks good to you, I will remove all the

+++ {"user_expressions": []} 

and hand it over to you to merge. I will move on to cagan_ree as I promised Tom to get it ready as well.

Many thanks in advance.

@jstac
Copy link
Contributor

jstac commented Jun 4, 2023

Nice work @HumphreyYang , well done.

Please proceed and ping me when you are ready for me to merge.

@jstac jstac merged commit 84cc6b3 into main Jun 4, 2023
@jstac jstac deleted the cons_smooth branch June 4, 2023 05:08
@jstac
Copy link
Contributor

jstac commented Jun 4, 2023

Ah, just merged by accident --- sorry, I was on autopilot.

@HumphreyYang , you can leave those +++ {"user_expressions": []} --- let's worry about them later. I think this is good as is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants