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

Incorporate slightly different PUF and CPS data files #2185

Merged
merged 1 commit into from
Jan 11, 2019
Merged

Incorporate slightly different PUF and CPS data files #2185

merged 1 commit into from
Jan 11, 2019

Conversation

martinholmer
Copy link
Collaborator

@martinholmer martinholmer commented Jan 10, 2019

This pull request updates the cps.csv.gz file by removing the old filer variable and it also renames the old filer variable in the puf.csv file as data_source. The data_source variable is not used in tax-calculation logic; it just identifies the primary source of the filing unit's data as being either IRS-SOI PUF (=1) or CPS (=0). Because all the filing units in the cps.csv.gz file would have a data_source value of zero, that variable is not included in the cps.csv.gz file. Because this variable is not used in the tax calculations, there are no changes in tax results in this pull request.

The new puf.csv file associated with this pull request has been generated by taxdata pull request 296. Here is the summary information about the new puf.csv file:

MD5 checksum:  4aa15435c319bf5e4d3427faf52384c0
Byte size:  56415704

@codecov-io
Copy link

codecov-io commented Jan 10, 2019

Codecov Report

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

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #2185   +/-   ##
======================================
  Coverage    99.9%   99.9%           
======================================
  Files          15      15           
  Lines        3241    3241           
======================================
  Hits         3238    3238           
  Misses          3       3

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 b07e6fd...9455aa0. Read the comment docs.

@martinholmer martinholmer merged commit d8a6e02 into PSLmodels:master Jan 11, 2019
@martinholmer martinholmer deleted the new-data branch January 11, 2019 14:52
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