Skip to content

Conversation

@HumphreyYang
Copy link
Member

This resolves #120.

@netlify
Copy link

netlify bot commented Jun 3, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit 5e94636
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/647d573b466829000890ce84
😎 Deploy Preview https://deploy-preview-221--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 3, 2023

@HumphreyYang
Copy link
Member Author

Hi @jstac,

I have improved plots in this lecture and Tom has kindly provided observations on the growth trajectory. Please kindly let me know if anything can be further improved.

Many thanks in advance.

@jstac
Copy link
Contributor

jstac commented Jun 4, 2023

There are beautiful and enlightening graphs on long run growth @HumphreyYang . Outstanding work.

Thanks also for working so hard over the weekend.

Can you please make the small changes requested above? Then I will do a final pass.

@HumphreyYang
Copy link
Member Author

Hi @mmcky,

I just realized that in the introduction, you had GNP, but the graph and data indicate that it is GDP. Just to double check if I should change the GNP to GDP in the introduction prior to the imports.

Many thanks in advance.

@jstac
Copy link
Contributor

jstac commented Jun 5, 2023

Thanks for picking that up @HumphreyYang . @mmcky is traveling today so please just go ahead and make the change.

Please also note the merge conflicts.

@HumphreyYang
Copy link
Member Author

HumphreyYang commented Jun 5, 2023

Hi @jstac,

This is all green now. Please feel free to make edits in this PR or after the merge.

Many thanks in advance.

@jstac
Copy link
Contributor

jstac commented Jun 5, 2023

Thanks again @HumphreyYang . I'm going to merge and then iterate.

@jstac jstac merged commit 8d11249 into main Jun 5, 2023
@jstac jstac deleted the update-long-run branch June 5, 2023 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants