-
Notifications
You must be signed in to change notification settings - Fork 5
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
State Pivot Tables #130
base: master
Are you sure you want to change the base?
State Pivot Tables #130
Conversation
Improvements to function for reading data from GitHub, use of new microdf groupby to simplify functions, pivot tables of state-based % changes in poverty and gini.
State Pivot Tables
Hello @jwal1024! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2021-01-22 01:45:56 UTC |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments. Also it'd be good for this to do something that feeds into the broader analysis by exporting results for the notebook to load and display. It currently looks like it prints a couple dataframes but this would only be useful when viewing from terminal or copy/pasting elsewhere.
raw = pd.read_csv( | ||
"https://github.com/UBICenter/child-allowance/blob/master/jb/data/cps_00003.csv.gz", | ||
person_raw = pd.read_csv( | ||
"https://github.com/UBICenter/child-allowance/blob/master/jb/data/cps_00003.csv.gz?raw=true", # noqa |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Load this locally (should also be able to skip compression="gzip"
after doing so
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
Removed demographic Ginis but retained state-based; removed pp_change function; removed special characters. Remaining: Deal with local load + compression
Improvements to function for reading data from GitHub, use of new microdf groupby to simplify functions, pivot tables of state-based % changes in poverty and gini.