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

Add tax years 2011 and 2012 to policy_current_law.json #2519

Open
wants to merge 29 commits into
base: master
Choose a base branch
from

Conversation

jdebacker
Copy link
Member

This PR adds 2011 and 2012 tax law to the policy_current_law.json file.

The PR is currently incomplete, but I am opening it so the others can feel free to review changes as I make them.

@jdebacker jdebacker changed the title [WIP] Add tax years 2011 and 2012 to policy_current_law.json Add tax years 2011 and 2012 to policy_current_law.json Jan 3, 2021
@jdebacker jdebacker mentioned this pull request Jan 3, 2021
@jdebacker
Copy link
Member Author

This PR will be ready for review following the resolution of unit testing issues noted in Issue #2525.

@jdebacker
Copy link
Member Author

Note that Travis CI doesn't seem to be reported in the checks here, but it is passing on 3.6 and failing on 3.7 and 3.8. The test that fails is one noted in Issue #2525, tests/test_cpscsv.py::test_agg.

@MattHJensen
Copy link
Contributor

@jdebacker, thanks a lot for this pull request. It will be a big help to some of Tax-Calculator's users.

Are you open to adding an internet-taxsim validation exercise for 2011 and 2012 as a prereq for merging this?

Tax-Calculator's current validation materials are here and @chusloj has a related PR here.

@jdebacker
Copy link
Member Author

@MattHJensen Yes, I'm glad to try to add some validation exercises. I think those would be very important as a check on updates to the policy parameters. Thanks for pointing me towards those resources.

@jdebacker
Copy link
Member Author

I'm not getting zero diffs for 2011 and 2012. I've been able to diagnose a couple errors from the validation process, but have been spinning my wheels with another. In 2011, for filers who are old (>65) and with dependents, I'm getting small (<$20) diffs in iitax that are mirrored in the c62100 output variable. I've searched all kinds of age related parameters but can't find an issue. Any suggestions on the parameters to look for discrepancy in @MattHJensen @Peter-Metz @chusloj?

@jdebacker
Copy link
Member Author

@feenberg I'm trying to validate these additions of 2011 and 2012 tax law against TAXSIM27. I've spent a fair bit of time and can't find the root of some differences that seem related to the AMT in 2011. Would it be possible to get the values of intermediate variables from TAXSIM to help with my validation efforts? The input file I'm using is attached here.

Thanks!

a11.in.txt

@feenberg
Copy link
Contributor

feenberg commented Jan 29, 2021 via email

@jdebacker
Copy link
Member Author

@feenberg I just followed up via email - perhaps that will be easier. Thanks!

@MattHJensen MattHJensen changed the title Add tax years 2011 and 2012 to policy_current_law.json [WIP] Add tax years 2011 and 2012 to policy_current_law.json Feb 1, 2021
@jdebacker
Copy link
Member Author

Update on the validation process: I was able to get the "a" file to match for 2011, but it required setting EITC_ps_MarriedJ to the following:

{
                "year": 2011,
                "EIC": "0kids",
                "value": 5000.0
            },
            {
                "year": 2011,
                "EIC": "1kid",
                "value": 5000.0
            },
            {
                "year": 2011,
                "EIC": "2kids",
                "value": 5000.0
            },
            {
                "year": 2011,
                "EIC": "3+kids",
                "value": 5100.0
            },

These values are inconsistent with the values TPC reports and from what is on the 2011 edition of Publication 596. Both those sources suggest the value should be 5080 for all numbers of qualifying children.

@MattHJensen
Copy link
Contributor

Thanks for the update @jdebacker. It is nice to have a path to test the b file now.

@MattHJensen
Copy link
Contributor

@jdebacker, when you have an opportunity, could you update us on the status of this PR?

@jdebacker
Copy link
Member Author

@MattHJensen This validation is really time consuming and I haven't returned to it since my last comment - which means I still need to validate the "b" and "c" files for 2011 and 2012.

I was also kind of hoping that PR #2453 would be finished in the meantime for two reasons: (1) I think it will greatly improve the validation workflow and (2) TAXSIM 32 has fixed some bugs from TAXSIM27 and given my noticing an issue with the EITC parameters above it'd be really nice to have the most accurate calculator to validate against.

@martinholmer martinholmer changed the title [WIP] Add tax years 2011 and 2012 to policy_current_law.json Add tax years 2011 and 2012 to policy_current_law.json May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants