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 param for switching stacking order of dependent credit and ctc #1676

Merged
merged 1 commit into from Nov 17, 2017

Conversation

MattHJensen
Copy link
Contributor

Resolves #1671 following recommendation by @codykallen.

See results.json which shows that stacking dependent credit above ctc does reduce tax liabilities as is expected based on the discussion in #1671.

cc @evtedeschi3 @martinholmer

@codecov-io
Copy link

codecov-io commented Nov 17, 2017

Codecov Report

Merging #1676 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1676   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files          37      37           
  Lines        2856    2856           
======================================
  Hits         2856    2856

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ff10abb...04471fe. Read the comment docs.

@martinholmer
Copy link
Collaborator

@MattHJensen said in pull request #1676:

Resolves #1671 following recommendation by @codykallen.

See results.json which shows that stacking dependent credit above CTC does reduce tax liabilities as is expected based on the discussion in #1671.

I haven't been following this issue closely, but I thought #1671 was about how the CTC and the Additional CTC interacted. What does the stacking order of the "dependent credit" and the nonrefundable CTC have to do with #1671? As you can see, I'm confused and need help understanding this issue. If the pull request does not produce the expected results, maybe we need to think about this issue some more.

@MattHJensen
Copy link
Contributor Author

I haven't been following this issue closely, but I thought #1671 was about how the CTC and the Additional CTC interacted. What does the stacking order of the "dependent credit" and the nonrefundable CTC have to do with #1671?

Under current law the maximum CTC is $1000 and up to $1000 can flow through to the ACTC for refundability. The TCJA adds to the CTC an amount that can not flow to the ACTC for refundability. We have been modeling that amount with a parameter that we call the Nonrefundable credit for children, on top of the child tax credit There is also a nonrefundable credit for non-children dependents that is treated similarly. The naming here is definitely somewhat confusing, and perhaps if this is enacted we should rethink it (or beforehand), but no one yet has suggested a better approach.

If the pull request does not produce the expected results, maybe we need to think about this issue some more.

The pull request does produce the expected result.

@martinholmer
Copy link
Collaborator

martinholmer commented Nov 17, 2017

@MattHJensen said:

The pull request does produce the expected result.

Right. Sorry, I misread your original comment in pull request #1676.

@martinholmer martinholmer changed the title add param for switching stacking order of dependent credit and ctc Add param for switching stacking order of dependent credit and ctc Nov 17, 2017
@martinholmer martinholmer merged commit 7f60870 into PSLmodels:master Nov 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants