search

PSLmodels / Tax-Brain

Tax-Brain is an integrator model for PSL tax models
http://taxbrain.pslmodels.org/
MIT License
9 stars 14 forks source link

Small issue: SALT cap should not be indexed #101

Closed TylerEvilsizer closed 4 years ago

TylerEvilsizer commented 4 years ago

It appears that Tax Brain's baseline settings index the $10,000 cap on the state and local tax deduction for inflation, but it should not be. The limit does not increase with inflation...it's $10,000 for all years ($5,000 for married filing separately)

image

hdoupe commented 4 years ago

@TylerEvilsizer thanks for opening this issue!

Tax-Calculator sets this as an indexed parameter (see value of "indexed" in the parameter definition).

@MattHJensen @Peter-Metz @andersonfrailey Can you comment on whether this parameter should be indexed or not?

MattHJensen commented 4 years ago

@TylerEvilsizer is correct that the cap should not be indexed to inflation. I will fix this in Tax-Calculator. Thanks very much for the bug report @TylerEvilsizer!

MattHJensen commented 4 years ago

This is now fixed in taxcalc 2.7.0 thanks to a PR from @MaxGhenis.

@hdoupe, is there something you can do to trigger a rebuild of Tax-Brain on Compute Studio, or do we need a new Tax-Brain release?

hdoupe commented 4 years ago

Yep, I'll build it from master. I'll comment back when it's up.

TylerEvilsizer commented 4 years ago

Thanks all!

On Fri, Dec 13, 2019 at 2:21 PM Henry Doupe notifications@github.com wrote:

Yep, I'll build it from master. I'll comment back when it's up.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PSLmodels/Tax-Brain/issues/101?email_source=notifications&email_token=AOAFPV6YU44JIGT56A3PXELQYPOCBA5CNFSM4JZBSRPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEG27TIQ#issuecomment-565574050, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOAFPV3MQHV2O4OKWHMHXUDQYPOCBANCNFSM4JZBSRPA .

hdoupe commented 4 years ago

All set! Tax-Brain, Tax-Cruncher, and Cost-of-Capital-Calculator are all running Tax-Calculator 2.7.0 Thanks again for the bug report @TylerEvilsizer!