-
-
Notifications
You must be signed in to change notification settings - Fork 157
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
Including children age 17 in the child tax credit calculator possible error? #2630
Comments
cc'ing @MaxGhenis and @kpomerleau who may be able to help here (I know they've both thought about this). |
I remember noticing this issue before. |
@Thirdhuman, n24 is misleadingly titled, (number of people under age 17), but in the (possible) error above I describe it correctly.... Maybe I'm misunderstanding total number of exemptions is not equivilent to total number of people in the filing unit? |
@JackLandry and @Thirdhuman, This is a complicated issue (which is probably why you haven't heard much from the Tax-Calculator maintainers over the last two weeks). The first thing to always keep in mind is that there are many possibilities here: there could be a variable documentation error, there could be a Tax-Calculator coding error, there could be internally inconsistent (CPS and/or PUF) data being used by Tax-Calculator users, we could be confused, or there could be a combination of those things going on all at the same time. Over the past few days, I've been tabulating dump output for 2021 generated by the I'll be sharing some of my tabulation results here in the coming days, but so far it seems to me that things are not right and that @JackLandry was right to raise an issue about this matter. One question I have for the two of you: how are you using Tax-Calculator? On the web or on you own computer? If the later, are you writing Python scripts or are you using the |
Glad to here I'm not totally off-base! I've been using CPS data and the command line tool, though I construct my own tax units in the CPS rather than using the taxcalc built in CPS data. @martinholmer I agree that there could be something going on between IRS-PUF vs. CPS given the comments in #2571 Also, this issue has an easy workaround (changing ages directly in the data) so no worries about the delay. I could make a quick exsample if that's helpful. |
@JackLandry said:
That's very interesting. |
@JackLandry (Jack) raised issue #2630 by making this observation:
I have done some tabulation of 2021 data dumped from a current-law-policy My conclusion is that the expression:
is logically incorrect as Jack indicated. The above statement generates estimates of age17 eligibles that are However, there is a logically correct method of estimating the number The logically correct method computes the number of dependents under One way to code the logically correct method is this:
When the data being processed by Tax-Calculator are logically I have no access to the REPLICATION NOTE: I dumped
and getting no differences. NOTE ON THE 1.906 ESTIMATE: Strictly speaking 2021 tax liability before the CTC should have been computed with |
@jdebacker and @MattHJensen, |
I responded about a month ago replying to this thread directly through email, but it didn't seem to work because I don't see my comments. I agree that the code in question has a logical flaw. Here is what I wrote responding to your suggested code. From November 12th 2021: Hi Martin, I tried your suggestion using the PUF-derived data. While I follow the logic of your suggestion, it ends up producing an increase in eligible children of about 22 million. The revenue impact of extending the CTC to 17-year-olds also reflects this. The original flawed code in question produces a revenue impact of $1.75 billion a year while your code produces a revenue impact of $20 billion a year. If I drop the 'max(0,' test, the increase in eligible children drops from 22 million to 10 million and the revenue impact per year is $7.44 billion. More reasonable, but 80 million total children still seems a little large. Also worth nothing that without the max statement, we end up with 101 observations with negative age 17 children. Kyle |
@kpomerleau (Kyle) wrote on 2021-12-29:
Kyle, thanks for posting this comment. So, I guess we are in agreement that the current Tax-Calculator logic is incorrect. But your point is that when using my "suggested code" (which does seem logical to me and to @JackLandry) along with the Tax-Calculator PUF data, there are way too many 17-year-olds, even though the "suggest code" gives about the correct number of 17-year-olds when using the Tax-Calculator CPS data. To me, this points to serious problems with the age variables in the Tax-Calculator PUF data. I have already posted two issues that document (less severe) problems with CPS age variables at the The Tax-Calculator PUF data is known to have problems with its age variables as described in issue #2469. This issue (originally posted about a year and a half ago) is still open, so I assume nothing has been done in The obvious solution to this problem is to fix the age variables in the Tax-Calculator PUF data and then fix the invalid logic (being discussed in this issue #2630) when the new PUF data is introduced as part of Tax-Calculator 4.0, for which there is already a GutHub development branch. Does this make sense to you, @kpomerleau? |
@JackLandry, @kpomerleau, and @martinholmer thanks for your investigation of this issue! Tax-Calculator accommodates numerous datasets from other open source projects as well as user-submitted data. I suggest that this project fix the known bug here and issue a release with the fix now, rather than sequence our work based on another project's timeline. TaxData contributors (I'm sure all of us are very welcome) can tackle the issue over there on their own timeline. TaxData PUF users should review that project's open issues to understand the TaxData PUF's advantages and limitations. If we think TaxData users don't do that already, we could revise Tax-Calculator's documentation, here, to suggest it. @JackLandry or @martinholmer, do you have any interest in opening the Tax-Calculator PR, or would you prefer for me to do so building on your work here? |
@MattHJensen said in issue #2630:
Speaking for myself, I have the interest (which is why I investigated this issue), but don't have the "bandwidth" to prepare and test a pull request. So, it makes more sense for you to build on @JackLandry's insight. |
@MattHJensen, Thanks for your attention to this, I'm such a novice Python user that I think it would make sense for you to take the lead on this. Thanks! |
I will tee up the PR. Thanks again all for your help with this issue! |
Resolved by #2644. |
Part of the child tax credit is calculated as follows:
It seems to me that the formula for childnum when CTC_include17=true is not correct. The total number of exemptions (basically total number of people in the filing unit) minus the number of people age 21 and over, the number of people age 18-20, the number of people under age 17, and then the indicator for marital status (2 when married filing jointly, otherwise one) seems like it will always be negative or zero. For instance, imagine a married couple with one child age 17. The formula would work out as follows: 3 - 2 - 0 - 0 - 2 = -1. Alternatively, a single-parent household with one child age 17: 2 - 1 - 0 - 0 - 1 = 0. Any children age 0 - 16 are canceled out by increasing exceptions and subtracting n24. Am I missing something, or is this an error? It relates to issue #2571
The text was updated successfully, but these errors were encountered: