-
Notifications
You must be signed in to change notification settings - Fork 30
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
Replacing nu05 with nu06 in CPS and PUF #347
Comments
Thanks Peter, what do you think about:
|
@MattHJensen the general plan sounds good. It might be easier to do the PUF/CPS update in one PR from a Tax-Calc testing standpoint, but I'm not sure. @MattHJensen or @andersonfrailey: would you mind advising me on how to generate a new puf.csv? |
@Peter-Metz, all you'll need to do is drop the raw PUF file |
Thanks @andersonfrailey! Is the CPS file called |
@Peter-Metz yep that's the one! |
This has been taken care of. Closing. |
Based on #329 and #314, it seems that the most recent versions of the PUF and CPS both have the
nu06
variable. However, Tax-Calculator does not recognizenu06
as a valid Records variable (see records_variables.json) because the newest versions of the PUF and CPS have not yet been disseminated.I wanted to open this issue to continue the conversation in #329 and come up with a plan to bring Tax-Calculator up to speed with Tax-Data.
cc @andersonfrailey @MattHJensen
The text was updated successfully, but these errors were encountered: