{"action":"created","issue":{"url":"https://api.github.com/repos/deeplearning4j/DataVec/issues/231","repository_url":"https://api.github.com/repos/deeplearning4j/DataVec","labels_url":"https://api.github.com/repos/deeplearning4j/DataVec/issues/231/labels{/name}","comments_url":"https://api.github.com/repos/deeplearning4j/DataVec/issues/231/comments","events_url":"https://api.github.com/repos/deeplearning4j/DataVec/issues/231/events","html_url":"https://github.com/deeplearning4j/DataVec/pull/231","id":221667931,"number":231,"title":"Remove .transform(null) in BaseImageRecordReader","user":{"login":"crockpotveggies","id":1445295,"avatar_url":"https://avatars3.githubusercontent.com/u/1445295?v=3","gravatar_id":"","url":"https://api.github.com/users/crockpotveggies","html_url":"https://github.com/crockpotveggies","followers_url":"https://api.github.com/users/crockpotveggies/followers","following_url":"https://api.github.com/users/crockpotveggies/following{/other_user}","gists_url":"https://api.github.com/users/crockpotveggies/gists{/gist_id}","starred_url":"https://api.github.com/users/crockpotveggies/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/crockpotveggies/subscriptions","organizations_url":"https://api.github.com/users/crockpotveggies/orgs","repos_url":"https://api.github.com/users/crockpotveggies/repos","events_url":"https://api.github.com/users/crockpotveggies/events{/privacy}","received_events_url":"https://api.github.com/users/crockpotveggies/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2017-04-13T20:06:48Z","updated_at":"2017-04-26T07:52:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/deeplearning4j/DataVec/pulls/231","html_url":"https://github.com/deeplearning4j/DataVec/pull/231","diff_url":"https://github.com/deeplearning4j/DataVec/pull/231.diff","patch_url":"https://github.com/deeplearning4j/DataVec/pull/231.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nIn hasNext() there were multiple calls to imageTransform.transform(null). From reviewing the code and our transforms, I could not find any benefits to doing so. Removing this code.\r\n\r\n## How was this patch tested?\r\n\r\nRan unit tests and runs on my local.\r\n"},"comment":{"url":"https://api.github.com/repos/deeplearning4j/DataVec/issues/comments/297276175","html_url":"https://github.com/deeplearning4j/DataVec/pull/231#issuecomment-297276175","issue_url":"https://api.github.com/repos/deeplearning4j/DataVec/issues/231","id":297276175,"user":{"login":"saudet","id":3328023,"avatar_url":"https://avatars2.githubusercontent.com/u/3328023?v=3","gravatar_id":"","url":"https://api.github.com/users/saudet","html_url":"https://github.com/saudet","followers_url":"https://api.github.com/users/saudet/followers","following_url":"https://api.github.com/users/saudet/following{/other_user}","gists_url":"https://api.github.com/users/saudet/gists{/gist_id}","starred_url":"https://api.github.com/users/saudet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/saudet/subscriptions","organizations_url":"https://api.github.com/users/saudet/orgs","repos_url":"https://api.github.com/users/saudet/repos","events_url":"https://api.github.com/users/saudet/events{/privacy}","received_events_url":"https://api.github.com/users/saudet/received_events","type":"User","site_admin":false},"created_at":"2017-04-26T07:52:40Z","updated_at":"2017-04-26T07:52:40Z","body":"@AlexDBlack IIRC, you had a similar issue in Arbiter or some such not terminating because of opened windows. What did you end up doing in that case?"}}
{"actor":{"display_login":"saudet"}}