-
-
Notifications
You must be signed in to change notification settings - Fork 159
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 calculation of expanded income #1057
Fix calculation of expanded income #1057
Conversation
Current coverage is 98.76% (diff: 100%)@@ master #1057 diff @@
==========================================
Files 38 38
Lines 2745 2745
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
Hits 2711 2711
Misses 34 34
Partials 0 0
|
On Fri, 11 Nov 2016, Martin Holmer wrote:
|
Dan @feenberg asked the following questions about pull request #1057:
Zero under current-law policy; when positive it is computed this way:
c02900, no matter what its value, is excluded from AGI, so adding it back in
That's been there for many, many months. I don't know much about the
Don't know about the others, but the logic of inclusion seems fairly strong. @MattHJensen @Amy-Xu @GoFroggyRun @andersonfrailey @codykallen |
@feenberg asked:
Yes, the purpose of expanded income is as the tab variable for tables, and we do use the base law values. Given that, I think the statement in question about behavioral responses should be removed. |
@feenberg asked:
@martinholmer replied:
I think we should include them based on our reasoning alone. But for reference, JCT also includes them: see page 7 of their report on income measures. |
@MattHJensen said about including
Thanks for this useful JCT reference. It raises a couple of other issues: (1) They don't add into expanded income deductible contributions to IRAs (or other non-employer-sponsored defined-contribution pensions?) as discussed on pages 9-10. They do this because they don't have data on contribution to employer-sponsored pension plans. They admit that if they did have data on employer-sponsored pension contributions that they would include all of these contributions in expanded income. But because they don't have the data on contributions to employer-sponsored plans, they do not add into expanded income the known deductible contributions to IRAs or other non-employer-sponsored defined-contribution pensions ( (2) On page 12, after discussing filing units with negative pass-through income (which is included in expanded income), JCT says: "For this reason, the Joint Committee staff eliminates observations with negative net expanded income from distributional analysis." Does this make sense? What, if anything, should we doing special with those filing units with negative expanded income? @MattHJensen @feenberg @Amy-Xu @GoFroggyRun @andersonfrailey @codykallen |
On Fri, 11 Nov 2016, Martin Holmer wrote:
I think the JCT procedure might make some sense if the two contribution
Back when there were tax shelters, losses were often artificial and it
|
|
||
ALD_EarlyWithdraw_hc : Deduction for forfeited interest penalty haricut | ||
ALD_EarlyWithdraw_hc : Penalty on early withdrawl deduction haricut |
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.
withdrawl - > withdrawal
""" | ||
# Form 2555 foreign earned income deduction is assumed to be zero | ||
# Form 1040 adjustments that are included in expanded income: | ||
c02900_less = ((1. - ALD_StudentLoan_hc) * e03210 + |
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.
What do you think about C02900_in_expanded_income? or C02900_EI?
@martinholmer, all of the substantive changes in this PR look good to me. I left a couple of small comments. |
@MattHJensen said:
Thanks for the suggestions. I've revised pull request #1057 along the lines you suggested. And after doing that, I merged #1057 into the master branch. |
This pull request adds several things into the
_expanded_income
variable that were previously left out of expanded income. There is no change in tax-calculating logic, but this pull request will cause the value of_expanded_income
to be higher for some filing units. Obviously, these changes will affect the results of any post-simulation tabulation that uses the_expanded_income
variable.In the decision making about what new items to add into
_expanded_income
, the JCT principles discussed below in #1057 have been followed with respect to pension contributions and health insurance premiums.@MattHJensen @feenberg @talumbau @Amy-Xu @GoFroggyRun @andersonfrailey @codykallen