-
-
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
update inflation policy parameter values for the year 2020, 2021, 2022 #2633
Conversation
This PR aims to update the policy parameter values for inflation adjustments of the year 2020, 2021, 2022 The process to update policy parameter values will follow Issue #2183 The inflation adjustments of year 2022 follows the newly published tax law inflation adjustments for 2022 by IRS, as Revenue Procedure 2021-45 The inflation adjustments of year 2020 and 2021 follows the published IRS tax forms and instructions: Form 1040 and instructions, Form 1040 (Schedule 1), Form 1040 (Schedule 2), Form 1040 (Schedule 4), Form 1040 (Schedule A) and instructions cc @MattHJensen |
I'm confused about the objective of this PR. Do you mean this PR will add, to the Right now, the last "know year" in the
|
Thanks for checking with this point. This PR will update the inflation adjusted parameter value for the 2022 only, to the policy_current_law.json file. This update will be based upon the newly published tax law inflation adjustments for 2022 by IRS, as Revenue Procedure 2021-45 |
@bodiyang said:
OK. But what about the known values for 2020 and 2021? Aren't those considered important any more? |
Yes, year 2020 and 2021 are also considered to be added into policy_current_law.json file. I will add them either in two separate commits or create two PRs. |
… Rate, Child Tax Credit, Earned Income Crdit, Exemption Amounts for Alternative Minimum Tax, Standard Deduction, Qualified Business Income
This Commit complete the update of major parameters for the year 2022 (only part of the adjusted parameters, 62 items total) in updated items including: Section 3: did not include the rest of adjusted items --- didn't find corresponding parameters in |
follow up note: the work of updating inflation adjusted parameter value for the year 2020 and 2021 is now included in this PR |
@bodiyang Can you push your latest changes showing how you've modified files for the |
Issue to pass the test: testing failure brief: multiple updated parameters failed the test test_round_trip_reforms(fyear, tests_path), checking if the projected value matches the expected value As an example: unmatched value between clp value -- the expected value and rtr value -- the projected value. For most of these parameters, the test failure can be solved by writing these parameter's expected value (clp) in the TCJA.json file. However, there are several exceptions and the EITC_c in 2023 is an example case. Have written EITC_c :{2023: [ 570.25 3801.31 6174.97 7061.91]} in the TCJA.json file, I still receive the test failure of this parameter from the feedback of local testing, as EITC_c in 2023 : rtr=[[ 556.21 3707.32 6127.7 6894.19]] clp=[[ 570.25 3801.31 6174.97 7061.91]] Please refer to my last commit for the updated TCJA.json file and this attached pdf file for the testing errors |
@bodiyang I've checked out your branch from this PR and have the round-trip-reform test passing (with modifications to reform files reflected in PSL/examples PR #11. I will open a PR to your branch when I get some feedback on exactly how to handle adding new reform files. I posed the relevant question in Issue #2589. |
Codecov Report
@@ Coverage Diff @@
## master #2633 +/- ##
=======================================
Coverage 98.54% 98.54%
=======================================
Files 14 14
Lines 2609 2609
=======================================
Hits 2571 2571
Misses 38 38
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@bodiyang said:
This statement is definitely not true. |
My only reference file for this work of inflation parameter update is the instruction for 2022 tax return file RP-2021-45, in which 2022 Maximum taxable earnings for Social Security, SS_Earnings_c, was not mentioned. I've just checked other online sources which indicating SS_Earnings_c is $147,000. link. Shall I include this number from this source? |
{ | ||
"year": 2021, | ||
"value": 0.0 | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again, 2020 before 2021. And it looks like really no change here.
taxcalc/policy_current_law.json
Outdated
{ | ||
"year": 2021, | ||
"value": 0.0 | ||
}, | ||
{ | ||
"year": 2020, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2020 should be before 2021
{ | ||
"year": 2021, | ||
"value": 0.0 | ||
}, | ||
{ | ||
"year": 2020, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2020 should be before 2021
@jdebacker I've corrected the format & missing value in 2022 issues in your comments in the new commit 8c1b609 PS:changes still required to pass the test ~ working on it |
Have passed all the tests on local machine |
@bodiyang I opened a PR to your branch with updated PUF results. But in reviewing my comments, there are still a few that aren't addressed. One of them has to do with the |
Update PUF results for latest changes
@bodiyang Thank you for this PR! |
add the inflation policy form Revenue Procedure 2021-45