Skip to content

[price_level] New Lecture on Inflation #244

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

Merged
merged 7 commits into from
Jun 19, 2023
Merged

[price_level] New Lecture on Inflation #244

merged 7 commits into from
Jun 19, 2023

Conversation

mmcky
Copy link
Contributor

@mmcky mmcky commented Jun 15, 2023

This PR is a new lecture on inflation and price level

@netlify
Copy link

netlify bot commented Jun 15, 2023

Deploy Preview for taupe-gaufre-c4e660 ready!

Name Link
🔨 Latest commit d476aaa
🔍 Latest deploy log https://app.netlify.com/sites/taupe-gaufre-c4e660/deploys/648fa3ad2e0d9100089a0f00
😎 Deploy Preview https://deploy-preview-244--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 15, 2023

@HumphreyYang
Copy link
Collaborator

Hi @jstac,

This PR is ready to merge. I fixed a few minor typos and prevent x-label from overlapping with tick labels.

Could you please kindly review this PR?

Many thanks in advance.

@jstac
Copy link
Contributor

jstac commented Jun 19, 2023

Many thanks @thomassargent30 and also @HumphreyYang for the review. This is a great lecture.

The long term trends in inflation in the advanced countries are striking.

The lecture pip installs xlrd but I think it is never used? @HumphreyYang , could you please check and cut that line if I'm right?

@HumphreyYang
Copy link
Collaborator

Hi @jstac, xlrd is used in

# import data
df_fig5 = pd.read_excel('datasets/longprices.xls', sheet_name='all', header=2, index_col=0).iloc[1:]

If we remove it, we will get

ImportError: Missing optional dependency 'xlrd'. Install xlrd >= 1.0.0 for Excel support Use pip or conda to install xlrd.

@jstac
Copy link
Contributor

jstac commented Jun 19, 2023

Roger that @HumphreyYang . Merging.

@jstac jstac merged commit 154a773 into main Jun 19, 2023
@jstac jstac deleted the hyperinflate branch June 19, 2023 02:27
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.

4 participants