{"action":"created","issue":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/4043","labels_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/4043/labels{/name}","comments_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/4043/comments","events_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/4043/events","html_url":"https://github.com/scikit-learn/scikit-learn/issues/4043","id":53247901,"number":4043,"title":"PEP8 Inconsistencies","user":{"login":"ragv","id":9487348,"avatar_url":"https://avatars.githubusercontent.com/u/9487348?v=3","gravatar_id":"","url":"https://api.github.com/users/ragv","html_url":"https://github.com/ragv","followers_url":"https://api.github.com/users/ragv/followers","following_url":"https://api.github.com/users/ragv/following{/other_user}","gists_url":"https://api.github.com/users/ragv/gists{/gist_id}","starred_url":"https://api.github.com/users/ragv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ragv/subscriptions","organizations_url":"https://api.github.com/users/ragv/orgs","repos_url":"https://api.github.com/users/ragv/repos","events_url":"https://api.github.com/users/ragv/events{/privacy}","received_events_url":"https://api.github.com/users/ragv/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-01-02T11:20:16Z","updated_at":"2015-01-02T21:24:24Z","closed_at":null,"body":"We have 479 PEP8 inconsistencies in the master. [This pastebin file](http://pastebin.com/BiFS0uTi)  contains the entire list.\r\n\r\nI would like to fix these all... but there is a definite possibility of merge conflicts with several PRs... Is fixing all this in a single PR recommended? ( Fixing part by part along with other PRs clutters the diff of other PRs and makes it not so easy to review the more significant changes in those PRs... )"},"comment":{"url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/comments/68563745","html_url":"https://github.com/scikit-learn/scikit-learn/issues/4043#issuecomment-68563745","issue_url":"https://api.github.com/repos/scikit-learn/scikit-learn/issues/4043","id":68563745,"user":{"login":"ragv","id":9487348,"avatar_url":"https://avatars.githubusercontent.com/u/9487348?v=3","gravatar_id":"","url":"https://api.github.com/users/ragv","html_url":"https://github.com/ragv","followers_url":"https://api.github.com/users/ragv/followers","following_url":"https://api.github.com/users/ragv/following{/other_user}","gists_url":"https://api.github.com/users/ragv/gists{/gist_id}","starred_url":"https://api.github.com/users/ragv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ragv/subscriptions","organizations_url":"https://api.github.com/users/ragv/orgs","repos_url":"https://api.github.com/users/ragv/repos","events_url":"https://api.github.com/users/ragv/events{/privacy}","received_events_url":"https://api.github.com/users/ragv/received_events","type":"User","site_admin":false},"created_at":"2015-01-02T21:24:24Z","updated_at":"2015-01-02T21:24:24Z","body":"Hmm... okay so these are the categories of pep8 issues we have at master...\r\n\r\n1       E101 indentation contains mixed spaces and tabs\r\n5       E111 indentation is not a multiple of four\r\n2       E112 expected an indented block\r\n5       E113 unexpected indentation\r\n6       E121 continuation line under-indented for hanging indent\r\n7       E124 closing bracket does not match visual indentation\r\n1       E125 continuation line with same indent as next logical line\r\n16      E126 continuation line over-indented for hanging indent\r\n21      E127 continuation line over-indented for visual indent\r\n67      E128 continuation line under-indented for visual indent\r\n1       E129 visually indented line with same indent as next logical line\r\n5       E131 continuation line unaligned for hanging indent\r\n1       E201 whitespace after '['\r\n1       E203 whitespace before ':'\r\n2       E221 multiple spaces before operator\r\n4       E222 multiple spaces after operator\r\n4       E231 missing whitespace after ','\r\n7       E261 at least two spaces before inline comment\r\n1       E262 inline comment should start with '# '\r\n116     E265 block comment should start with '# '\r\n6       E301 expected 1 blank line, found 0\r\n19      E302 expected 2 blank lines, found 1\r\n17      E303 too many blank lines (2)\r\n125     E501 line too long (94 > 79 characters)\r\n20      E713 test for membership should be 'not in'\r\n2       E714 test for object identity should be 'is not'\r\n1       W191 indentation contains tabs\r\n4       W291 trailing whitespace\r\n8       W293 blank line contains whitespace\r\n4       W391 blank line at end of file\r\n\r\nCould you please categorize to either one of:\r\n\r\n1. for \"Fix should be done regardless of context\" - using autopep8 + manual verification\r\n2. for \"Fix should be done but favor code readability\" - only manual\r\n3. for \"Do not fix\" or \"Unsure better leave it as such\" - leave it as such"}}