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

Allow population (rather than filing-unit) quantiles in tables and graphs #2322

merged 8 commits into from May 18, 2019


None yet
1 participant
Copy link

commented May 13, 2019

This pull request resolves issue #2310 by adding an optional pop_quantiles argument to all the Tax-Calculator table and graph methods that allow using quantiles for distributional analysis. As @MaxGhenis pointed out in issue #2310, both CBO and TPC use quantiles that contain equal numbers of people (not equal numbers of filing units). And, if you read the cited CBO document, you will see that both CBO and TPC also adjust the incomes of filing units by the square root of the number of people in the filing unit before they sort filing units by income. Setting the new optional pop_quantiles argument to True also does this unit-size adjustment of incomes.

The new optional pop_quantiles argument always has a default value of False, so that backward compatibility is maintained. But when doing Python programming, you have the option of setting it to True.

Implementing this seemingly simple enhancement request required considerable changes in low-level logic. Most, if not all, users will not be affected by the low-level changes: a few column names in the distributional tables have been changed, and the Calculator distribution_table_dataframe method has been integrated into the Calculator distribution_tables method making it easier to generate distribution tables. If you are using the low-level create_*_tables utility functions (rather than the high-level Calculator.*_tables methods, then you should review your application's operation when upgrading to the Tax-Calculator release that contains this pull request. Also, in the course of making these changes, a (never reported) bug in the difference table was found and corrected. Previously, the values of tax_cut and tax_inc were understated by a large factor in difference tables; now they are scaled correctly to the number of filing units (or people) expressed in millions.

A number of new tests have been added to check the new table logic and all seems well. But if you notice any unexpected changes in the contents of distribution or difference tables, please raise an issue.


This comment has been minimized.

Copy link

commented May 13, 2019

Codecov Report

Merging #2322 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #2322   +/-   ##
  Coverage     100%    100%           
  Files          12      12           
  Lines        2784    2703   -81     
- Hits         2784    2703   -81
Impacted Files Coverage Δ
taxcalc/ 100% <ø> (ø) ⬆️
taxcalc/ 100% <100%> (ø) ⬆️
taxcalc/ 100% <100%> (ø) ⬆️
taxcalc/ 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18d8867...b0c3002. Read the comment docs.

@martinholmer martinholmer merged commit 540b7c5 into PSLmodels:master May 18, 2019

4 checks passed

codecov/patch 100% of diff hit (target 100%)
codecov/project 100% (+0%) compared to 18d8867
continuous-integration/appveyor/pr AppVeyor build succeeded
continuous-integration/travis-ci/pr The Travis CI build passed

@martinholmer martinholmer deleted the martinholmer:popquant branch May 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.