-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
Complete and correct work for updates to inflation indexed parameters #2652
Conversation
|
Have checked the |
@bodiyang please see my PR to your branch where I create a json file to handle indexed parameter rounding for the round trip reform. |
@bodiyang Let me know if there are any issues with my PR to your branch. If you think it's ok, let's merge it. Thanks. |
Include inflation rounding json file
Codecov Report
@@ Coverage Diff @@
## master #2652 +/- ##
=======================================
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 I remaining incorrect parameter values for @MattHJensen There's a lot here and I don't want to miss something again. Can you put another set of eyes on this PR and let me know if there are still years out of order or if we failed to correct anything noted in Issues #2647, #2649, #2651 and now Issue #2655? |
Another EITC fix
update cps agg results
Thanks for your work on this @bodiyang! Merging despite a failure when testing on Windows with Python 3.7. Error looks like an idiosyncratic issue with connecting to Codecov.io |
No description provided.