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 Wealth Distribution Dynamics Lecture #170

Merged
merged 3 commits into from
May 7, 2024
Merged

Conversation

HumphreyYang
Copy link
Collaborator

This PR resolves #169.

Copy link

netlify bot commented Apr 1, 2024

Deploy Preview for incomparable-parfait-2417f8 ready!

Name Link
🔨 Latest commit ec20d3c
🔍 Latest deploy log https://app.netlify.com/sites/incomparable-parfait-2417f8/deploys/6614d0b5edfcbb00085d1ff3
😎 Deploy Preview https://deploy-preview-170--incomparable-parfait-2417f8.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.

@mmcky
Copy link
Contributor

mmcky commented Apr 1, 2024

@HumphreyYang this will need a pip install of quantecon

No module named 'quantecon'

to fix the execution issue.

lectures/wealth_dynamics.md Outdated Show resolved Hide resolved
@mmcky
Copy link
Contributor

mmcky commented Apr 1, 2024

thanks @HumphreyYang just some minor comments and fixes required.

Copy link

github-actions bot commented Apr 1, 2024

@github-actions github-actions bot temporarily deployed to pull request April 1, 2024 22:30 Inactive
@HumphreyYang HumphreyYang requested a review from mmcky April 2, 2024 05:38
@HumphreyYang HumphreyYang added ready and removed in-work labels Apr 2, 2024
@HumphreyYang HumphreyYang requested a review from jstac April 8, 2024 04:52
@jstac
Copy link
Contributor

jstac commented Apr 9, 2024

Thanks @HumphreyYang.

Is this based on an old version of main? The TOC is not up to date

https://jax.quantecon.org/intro.html
https://660b3591fe556334aa88cf97--incomparable-parfait-2417f8.netlify.app/intro.html

@HumphreyYang
Copy link
Collaborator Author

Many thanks @jstac,

Yes, this PR was opened before the TOC change was merged. I will merge the main into this branch and generate a new preview.

@jstac
Copy link
Contributor

jstac commented Apr 9, 2024

Thanks @HumphreyYang
CC @mmcky

@github-actions github-actions bot temporarily deployed to pull request April 9, 2024 05:36 Inactive
@mmcky
Copy link
Contributor

mmcky commented May 7, 2024

thanks @HumphreyYang

@jstac this looks good. Would you like this merged for Chile?

Wow! JAX is 290x faster. That's impressive!

@jstac
Copy link
Contributor

jstac commented May 7, 2024

Many thanks @HumphreyYang . Merging.

@jstac jstac merged commit b132f6b into main May 7, 2024
7 checks passed
@jstac jstac deleted the update-wealth-dist branch May 7, 2024 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update wealth distribution lecture
3 participants