{"action":"created","issue":{"url":"https://api.github.com/repos/open-source-economics/Tax-Calculator/issues/638","repository_url":"https://api.github.com/repos/open-source-economics/Tax-Calculator","labels_url":"https://api.github.com/repos/open-source-economics/Tax-Calculator/issues/638/labels{/name}","comments_url":"https://api.github.com/repos/open-source-economics/Tax-Calculator/issues/638/comments","events_url":"https://api.github.com/repos/open-source-economics/Tax-Calculator/issues/638/events","html_url":"https://github.com/open-source-economics/Tax-Calculator/pull/638","id":139430791,"number":638,"title":"Eliminate Records.UNUSED_READ_VARS set given new puf.csv file","user":{"login":"martinholmer","id":12170745,"avatar_url":"https://avatars.githubusercontent.com/u/12170745?v=3","gravatar_id":"","url":"https://api.github.com/users/martinholmer","html_url":"https://github.com/martinholmer","followers_url":"https://api.github.com/users/martinholmer/followers","following_url":"https://api.github.com/users/martinholmer/following{/other_user}","gists_url":"https://api.github.com/users/martinholmer/gists{/gist_id}","starred_url":"https://api.github.com/users/martinholmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinholmer/subscriptions","organizations_url":"https://api.github.com/users/martinholmer/orgs","repos_url":"https://api.github.com/users/martinholmer/repos","events_url":"https://api.github.com/users/martinholmer/events{/privacy}","received_events_url":"https://api.github.com/users/martinholmer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-03-08T23:56:04Z","updated_at":"2016-03-09T02:54:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/open-source-economics/Tax-Calculator/pulls/638","html_url":"https://github.com/open-source-economics/Tax-Calculator/pull/638","diff_url":"https://github.com/open-source-economics/Tax-Calculator/pull/638.diff","patch_url":"https://github.com/open-source-economics/Tax-Calculator/pull/638.patch"},"body":"This pull request simplifies the logic of creating arrays from the Pandas dataframe that is generated in the Records class by reading the input data file.  There is no change in tax-calculation logic and no change in any of the test results.\r\n\r\nHowever, the code in this pull request will work only when using the new smaller ```puf.csv``` file distributed to the core development team by Amy Xu on March 8, 2016.\r\n\r\nAnd, in addition, there was a need to eliminate close to half of the variables in the puf91taxdata.csv file used in the test suite because almost half of the variables are unused in Tax-Calculator.\r\n"},"comment":{"url":"https://api.github.com/repos/open-source-economics/Tax-Calculator/issues/comments/194089630","html_url":"https://github.com/open-source-economics/Tax-Calculator/pull/638#issuecomment-194089630","issue_url":"https://api.github.com/repos/open-source-economics/Tax-Calculator/issues/638","id":194089630,"user":{"login":"martinholmer","id":12170745,"avatar_url":"https://avatars.githubusercontent.com/u/12170745?v=3","gravatar_id":"","url":"https://api.github.com/users/martinholmer","html_url":"https://github.com/martinholmer","followers_url":"https://api.github.com/users/martinholmer/followers","following_url":"https://api.github.com/users/martinholmer/following{/other_user}","gists_url":"https://api.github.com/users/martinholmer/gists{/gist_id}","starred_url":"https://api.github.com/users/martinholmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martinholmer/subscriptions","organizations_url":"https://api.github.com/users/martinholmer/orgs","repos_url":"https://api.github.com/users/martinholmer/repos","events_url":"https://api.github.com/users/martinholmer/events{/privacy}","received_events_url":"https://api.github.com/users/martinholmer/received_events","type":"User","site_admin":false},"created_at":"2016-03-09T02:54:42Z","updated_at":"2016-03-09T02:54:42Z","body":"T.J. said:\r\n> Because this PR would make Tax-Calculator master non-backwards compatible with the existing PUF, can we put a one week hold on this? \r\n\r\nThat was exactly my thought.  I was going to reply to everybody who received Amy's email about the new ```puf.csv``` file, to ask them to begin using the new, smaller puf.csv sometime during the next week.  But I can do it via this comment.\r\n\r\nWould anybody like to suggest a target date for merging this pull request #638 into the master branch?\r\nA date that gives everybody a chance to finish up what they are now working on using the old, bigger ```puf.csv``` file.\r\n\r\n@MattHJensen @talumbau @feenberg @Amy-Xu @GoFroggyRun @rickecon @jdebacker @kerkphil "}}