PSLmodels / Tax-Calculator

USA Federal Individual Income and Payroll Tax Microsimulation Model
https://taxcalc.pslmodels.org
Other
255 stars 154 forks source link

Possible issue in reform modifying AMT exemption in 2013 #2323

Closed donboyd5 closed 5 years ago

donboyd5 commented 5 years ago

We (@donboyd5, @feenberg, @MaxGhenis) run several reforms against our synthetic PUF using the tc cli (latest version) to test how the synthetic file does. We run the reforms against 2013 baseline law because 2013 is the Tax-Calculator year that is closest to our data year (2011).

One reform gave surprising results so I ran the same reform against the actual puf. csv and it still gave surprising results. I am not sure there is any problem with Tax-Calculator - we may just not understand the reform properly - but I wanted to raise an alert in case it is a problem.

The reform is a $1,000 reduction in the AMT exemption. The entire reform json looks like this: { "AMT_em": {"2013": [50900.0, 79800.0, 39400.0, 50900.0, 79800.0]} }

I did not change the phase out start, the phase out rate, the AMT bracket threshold, AMT rate or surcharge, or capital gains/dividends parameters.

The table below shows the number of puf.csv records that had an increase in tax before credits (taxbc), a decrease, or no change:

image

We expected many more records to have an increase - we expected many of the records in the phaseout range of the exemption to have a reduction in the exemption, and therefore an increase in AMT, and therefore an increase in taxbc. Thus, the table above is a surprise.

An examination of the details shows:

I also ran the same reform using TaxBrain (both the OSPC and compmodels versions), which uses an older version of Tax-Calculator, and found that the reform led to a large number of tax increases. This suggests that the tc cli calculation might be incorrect, but certainly doesn't prove it.

I can provide further information, or look more deeply but I wanted to pass this on in case it is a problem.

donboyd5 commented 5 years ago

Here is some more-informative output from the same run. The rows are Tax-Calculator variable names. The columns are:

image

donboyd5 commented 5 years ago

Oh. If I read documentation correctly, taxbc is before considering AMT. Sorry and embarrassed. Will verify when back at computer.

martinholmer commented 5 years ago

In issue #2323, @donboyd5 said:

Oh. If I read documentation correctly, taxbc is before considering AMT. Sorry and embarrassed. Will verify when back at computer.

No problem and no need to be embarrassed!
After all, I didn't know that about taxbc off the top of my head either.

I can replicate your taxbc results and get more reasonable results when looking at iitax changes.

taxbc counts (>0,<0,=0): 25 1 248565
iitax counts (>0,<0,=0): 13212 346 235033
martinholmer commented 5 years ago

@donboyd5, if you want to tabulate income tax liability before credits, then this is the variable you want to use:

Screen Shot 2019-05-14 at 6 43 31 PM
donboyd5 commented 5 years ago

Thanks!

Sent by an android

On Tue, May 14, 2019, 6:45 PM Martin Holmer notifications@github.com wrote:

@donboyd5 https://github.com/donboyd5, if you want to tabulate income tax liability before credits, then this is the variable you want to use:

[image: Screen Shot 2019-05-14 at 6 43 31 PM] https://user-images.githubusercontent.com/12170745/57737094-668a3f00-7678-11e9-8e7c-485f48207893.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PSLmodels/Tax-Calculator/issues/2323?email_source=notifications&email_token=ABR4JGBJTOIGHSMZFI3OXBDPVM6KJA5CNFSM4HMZ6J22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVNAHKQ#issuecomment-492438442, or mute the thread https://github.com/notifications/unsubscribe-auth/ABR4JGADUAMBHYU63LVTSSTPVM6KJANCNFSM4HMZ6J2Q .