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

Update tbi_*_expect.txt results given fix of DependentCredit phase-out logic #1945

Merged
merged 1 commit into from
Mar 30, 2018
Merged

Conversation

martinholmer
Copy link
Collaborator

This pull request updates the taxcalc/tests/tbi_puf_expect.txt and the taxcalc/tests/tbi_cps_expect.txt test results files to reflect the effects of the fix in the dependent credit phase-out logic in pull request #1937.

The changes in results are very small. For example, when using puf.csv input data, the current-law 2020 aggregate income tax liability declines by about 0.02 percent (that is, by one-fiftieth of one percent). The bug-fix causes the test reform results to change by roughly the same amount. As a result, the change in the reform difference in 2020 aggregate income tax liability is only about 0.003 percent (that is, three one-thousandths of one percent).

@codecov-io
Copy link

codecov-io commented Mar 30, 2018

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1945   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files          38      38           
  Lines        3726    3726           
======================================
  Hits         3726    3726

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 18dec95...a3abda8. Read the comment docs.

@martinholmer martinholmer merged commit dc8b3d4 into PSLmodels:master Mar 30, 2018
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.

2 participants