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

Fix expanded income to include non-taxable pension income #1897

Merged
merged 2 commits into from Mar 7, 2018
Merged

Fix expanded income to include non-taxable pension income #1897

merged 2 commits into from Mar 7, 2018

Conversation

martinholmer
Copy link
Collaborator

@martinholmer martinholmer commented Mar 1, 2018

#1888 has raised a number of issues that we hope to resolve in the near future.

One of the questions #1888 raises in my mind is: What exactly is included in expanded_income?

It turns out that answering that question is difficult because of the convoluted logic currently used in Tax-Calculator to compute expanded_income. We hope to simply that logic in a forthcoming pull request.

But one thing is clear: non-taxable pension and annuity income, e01500 - e01700, is not currently included in expanded_income even though it makes up slightly more than one-third of total pension and annuity income, e01500, in the 2011 PUF. The amount of income omitted from expanded_income is roughly $330 billion in 2011.

This pull request corrects the logic of the expanded_income calculation to include non-taxable pension and annuity income. It also revises some puf.csv test results to reflect the fix of the puf.csv file implemented in taxdata pull request 158. And it revises some non-PUF test results to reflect changes caused by the correction of the definition of expanded_income.

@martinholmer martinholmer changed the title Fix expanded income Fix expanded income to include non-taxable pension income Mar 1, 2018
@codecov-io
Copy link

codecov-io commented Mar 1, 2018

Codecov Report

Merging #1897 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1897   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files          37      37           
  Lines        3308    3308           
======================================
  Hits         3308    3308

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 92d0a5a...1ce30f8. 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.

None yet

2 participants