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

Add quintile graph to tc --graphs output #1880

Merged
merged 2 commits into from
Feb 16, 2018
Merged

Add quintile graph to tc --graphs output #1880

merged 2 commits into from
Feb 16, 2018

Conversation

martinholmer
Copy link
Collaborator

Add quintile_graph method to Calculator class and two associated utility functions. Also, include the quintile graph in the output generated by the tc --graphs option.

This enhancement is being added to resolve issue #1806 which highlighted the fact that negative aggregate expanded income in the bottom decile produced misleading (but accurate) decile graphs of percentage change in after-tax expanded income. Since issue #1806 was raised new CPS data have been introduced that allow the inclusion of the consumption value of benefits provided by several benefit programs in expanded income. This CPS data enhancement has made aggregate expanded income in the bottom decile positive, which eliminates the potentially misleading graphical result. However, using the decile graph with PUF data can still produce misleading (but accurate) decile graphs.

The quintile graph is the same as the decile graph except that it uses percentile groups that are twice as big as used in the decile graph. The bottom quintile has positive aggregate expanded income and so it is not possible the produce misleading quintile graphs with either PUF or CPS input data.

@codecov-io
Copy link

codecov-io commented Feb 15, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1880   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files          37      37           
  Lines        3213    3301   +88     
======================================
+ Hits         3213    3301   +88
Impacted Files Coverage Δ
taxcalc/records.py 100% <ø> (ø) ⬆️
taxcalc/taxcalcio.py 100% <100%> (ø) ⬆️
taxcalc/utils.py 100% <100%> (ø) ⬆️
taxcalc/calculate.py 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 b5f1a81...6b9a25d. Read the comment docs.

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.

2 participants