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.

Sign up for GitHub

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 a haircut to the state and local tax deduction #234

Merged
merged 4 commits into from
Apr 9, 2015
Merged

Add a haircut to the state and local tax deduction #234

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
e09e157
parameterize StateLocalTax_HC
MattHJensen Apr 8, 2015
acea4a2
clean up formatting
MattHJensen Apr 8, 2015
09d1b6e
fix bug in params.json
MattHJensen Apr 8, 2015
9a48d5f
fix merge conflicts with master
MattHJensen Apr 8, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
45 changes: 26 additions & 19 deletions taxcalc/functions.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,17 +108,21 @@ def AGI( _ymod1, c02500, c02700, e02615, c02900, e00100, e02500, XTOT,
@iterate_jit(parameters=["puf", "ID_ps","ID_Medical_frt", "ID_Casualty_frt",
"ID_Miscellaneous_frt", "ID_Charity_crt_Cash",
"ID_Charity_crt_Asset", "ID_prt", "ID_crt",
"ID_Charity_frt"], nopython=True, puf=True)
"ID_Charity_frt", "ID_StateLocalTax_HC"],
nopython=True, puf=True)

def ItemDed(_posagi, e17500, e18400, e18425, e18450, e18500, e18800, e18900,
e20500, e20400, e19200, e20550, e20600, e20950, e19500,
e19570, e19400, e19550, e19800, e20100, e20200, e20900,
e21000, e21010, MARS, _sep, c00100, ID_ps,ID_Medical_frt,
ID_Casualty_frt, ID_Miscellaneous_frt, ID_Charity_crt_Cash,
ID_Charity_crt_Asset, ID_prt, ID_crt, ID_Charity_frt, puf):
ID_Charity_crt_Asset, ID_prt, ID_crt, ID_StateLocalTax_HC,
ID_Charity_frt, puf):

"""
WARNING: Any additional keyword args, such as 'puf=True' here, must be passed
to the function at the END of the argument list. If you stick the argument
somewhere in the middle of the signature, you will get errors.
WARNING: Any additional keyword args, such as 'puf=True' here, must be
passed to the function at the END of the argument list. If you stick the
argument somewhere in the middle of the signature, you will get errors.
"""
# Medical #
c17750 = ID_Medical_frt * _posagi
Expand Down Expand Up @@ -164,8 +168,8 @@ def ItemDed(_posagi, e17500, e18400, e18425, e18450, e18500, e18800, e18900,


# Gross Itemized Deductions #
c21060 = (e20900 + c17000 + c18300 + c19200 + c19700
+ c20500 + c20800 + e21000 + e21010)
c21060 = (e20900 + c17000 + (1-ID_StateLocalTax_HC)*c18300 + c19200
+ c19700 + c20500 + c20800 + e21000 + e21010)

_phase2_i = ID_ps[MARS-1]

Expand Down Expand Up @@ -581,9 +585,11 @@ def TaxGains(e00650, c04800, e01000, c23650, e23250, e01100, e58990,
# TODO should we be returning c00650 instead of e00650??? Would need to change tests


@iterate_jit(parameters=["AMT_tthd", "II_brk6", "II_brk2", "AMT_Child_em", "cgrate1",
"cgrate2", "AMT_em_ps", "AMT_em_pe", "KT_c_Age", "AMT_thd_MarriedS",
"AMT_em", "AMT_prt","AMT_trt1", "AMT_trt2", "puf"],
@iterate_jit(parameters=["AMT_tthd", "II_brk6", "II_brk2", "AMT_Child_em",
"cgrate1", "cgrate2", "AMT_em_ps", "AMT_em_pe",
"KT_c_Age", "AMT_thd_MarriedS", "AMT_em", "AMT_prt",
"AMT_trt1", "AMT_trt2", "ID_StateLocalTax_HC",
"puf"],
nopython=True, puf=True)
def AMTI( c60000, _exact, e60290, _posagi, e07300, x60260, c24517,
e60300, e60860, e60100, e60840, e60630, e60550,
Expand All @@ -598,15 +604,16 @@ def AMTI( c60000, _exact, e60290, _posagi, e07300, x60260, c24517,
II_brk6, MARS, _sep, II_brk2, AMT_Child_em, cgrate1,
cgrate2, AMT_em_ps, AMT_em_pe, x62720, e00700, c24516,
c24520, c04800, e10105, c05700, e05800, e05100, e09600,
KT_c_Age, x62740, e62900, AMT_thd_MarriedS, _earned, e62600, AMT_em,
AMT_prt, AMT_trt1, AMT_trt2, _cmbtp_itemizer, _cmbtp_standard, puf):
KT_c_Age, x62740, e62900, AMT_thd_MarriedS, _earned, e62600,
AMT_em, AMT_prt, AMT_trt1, AMT_trt2, _cmbtp_itemizer,
_cmbtp_standard, ID_StateLocalTax_HC, puf):

c62720 = c24517 + x62720
c60260 = e00700 + x60260
## QUESTION: c63100 variable is reassigned below before use, is this a BUG?
c63100 = max(0., _taxbc - e07300)
c60200 = min(c17000, AMT_prt * _posagi)
c60240 = c18300 + x60240
c60240 = (1-ID_StateLocalTax_HC)*c18300 + x60240
c60220 = c20800 + x60220
c60130 = c21040 + x60130
c62730 = e24515 + x62730
Expand All @@ -623,10 +630,10 @@ def AMTI( c60000, _exact, e60290, _posagi, e07300, x60260, c24517,


if _cmp == 1:
c62100 = (_addamt + e60300 + e60860 + e60100 + e60840 + e60630 + e60550
+ e60720 + e60430 + e60500 + e60340 + e60680 + e60600 + e60405
+ e60440 + e60420 + e60410 + e61400 + e60660 - c60260 - e60480
- e62000 + c60000 - e60250)
c62100 = (_addamt + e60300 + e60860 + e60100 + e60840 + e60630
+ e60550 + e60720 + e60430 + e60500 + e60340 + e60680 + e60600
+ e60405 + e60440 + e60420 + e60410 + e61400 + e60660 - c60260
- e60480 - e62000 + c60000 - e60250)


if (puf and (_standard == 0 or (_exact == 1 and e04470 > 0))):
Expand All @@ -643,8 +650,8 @@ def AMTI( c60000, _exact, e60290, _posagi, e07300, x60260, c24517,


if (puf == True and ((_standard == 0 or (_exact == 1 and e04470 > 0)))):
c62100 = (c00100 - c04470 + min(c17000, 0.025 * max(0., c00100)) + _sit
+ e18500 - c60260 + c20800 - c21040 )
c62100 = (c00100 - c04470 + min(c17000, 0.025 * max(0., c00100)) +
(1-ID_StateLocalTax_HC)*_sit + e18500 - c60260 + c20800 - c21040 )
# c62100 += _cmbtp


Expand Down
13 changes: 13 additions & 0 deletions taxcalc/params.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -905,6 +905,19 @@
"col_label": "",
"value": [0.8]
},
"_ID_StateLocalTax_HC":{
"long_name": "Deduction for state and local taxes haircut.",
"description": "This percentage reduces the state and local tax deduction." ,
"irs_ref": "",
"notes": "This parameter allows for the implementation of Option 51 from https://www.cbo.gov/sites/default/files/cbofiles/attachments/49638-BudgetOptions.pdf.",
"start_year": 2013,
"col_var": "",
"row_var": "",
"row_label": "",
"cpi_inflated": false,
"col_label": "",
"value": [0]
},
"_NIIT_trt":{
"long_name": "Net Investment Income Tax rate (UIMC)",
"description": "AKA Unearned Income Medicare Distribution Tax. Taxed on net investment income of individuals with MAGI higher than certain thresholds.",
Expand Down