-
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
Add k1bx14p and k1bx14s to puf.csv file #107
Conversation
@andersonfrailey, I know you're busy with the CPS file preparation, but I'm hoping you can help me issue a new version of the Using pull request #107, I get the following information about the new
Do you get the same MD5 checksums for input file |
Sorry for missing this, @martinholmer. I get the same MD5 checksums for both the input and output files as you do:
You're logic in this PR also looks good to me given its goal. Unless there is more you would like to add, I'll merge this PR and distribute the new PUF this afternoon. |
@andersonfrailey said about taxdata pull request #107:
That would be excellent! Thanks for the help. |
Merged #107 |
This pull request changes the logic of splitting the
e00900
(Schedule C) ande02100
(Schedule F) self-employment earnings variables and adds newk1bx14p
andk1bx14s
variables that are part of self-employment income taxable under the SECA tax (Schedule SE).These taxdata changes, along with the changes in Tax-Calculator pull request 1438, are meant to resolve the Tax-Calculator bug described in Tax-Calculator issue 1436.