{"action":"created","issue":{"url":"https://api.github.com/repos/drphilmarshall/StatisticalMethods/issues/55","labels_url":"https://api.github.com/repos/drphilmarshall/StatisticalMethods/issues/55/labels{/name}","comments_url":"https://api.github.com/repos/drphilmarshall/StatisticalMethods/issues/55/comments","events_url":"https://api.github.com/repos/drphilmarshall/StatisticalMethods/issues/55/events","html_url":"https://github.com/drphilmarshall/StatisticalMethods/pull/55","id":107820140,"number":55,"title":"fix the bug and use better coding style in XrayImage/Summarizing.ipynb","user":{"login":"yymao","id":3792659,"avatar_url":"https://avatars.githubusercontent.com/u/3792659?v=3","gravatar_id":"","url":"https://api.github.com/users/yymao","html_url":"https://github.com/yymao","followers_url":"https://api.github.com/users/yymao/followers","following_url":"https://api.github.com/users/yymao/following{/other_user}","gists_url":"https://api.github.com/users/yymao/gists{/gist_id}","starred_url":"https://api.github.com/users/yymao/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yymao/subscriptions","organizations_url":"https://api.github.com/users/yymao/orgs","repos_url":"https://api.github.com/users/yymao/repos","events_url":"https://api.github.com/users/yymao/events{/privacy}","received_events_url":"https://api.github.com/users/yymao/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-09-22T23:06:40Z","updated_at":"2015-09-23T06:00:18Z","closed_at":"2015-09-23T05:15:55Z","pull_request":{"url":"https://api.github.com/repos/drphilmarshall/StatisticalMethods/pulls/55","html_url":"https://github.com/drphilmarshall/StatisticalMethods/pull/55","diff_url":"https://github.com/drphilmarshall/StatisticalMethods/pull/55.diff","patch_url":"https://github.com/drphilmarshall/StatisticalMethods/pull/55.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/drphilmarshall/StatisticalMethods/issues/comments/142503721","html_url":"https://github.com/drphilmarshall/StatisticalMethods/pull/55#issuecomment-142503721","issue_url":"https://api.github.com/repos/drphilmarshall/StatisticalMethods/issues/55","id":142503721,"user":{"login":"drphilmarshall","id":710903,"avatar_url":"https://avatars.githubusercontent.com/u/710903?v=3","gravatar_id":"","url":"https://api.github.com/users/drphilmarshall","html_url":"https://github.com/drphilmarshall","followers_url":"https://api.github.com/users/drphilmarshall/followers","following_url":"https://api.github.com/users/drphilmarshall/following{/other_user}","gists_url":"https://api.github.com/users/drphilmarshall/gists{/gist_id}","starred_url":"https://api.github.com/users/drphilmarshall/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drphilmarshall/subscriptions","organizations_url":"https://api.github.com/users/drphilmarshall/orgs","repos_url":"https://api.github.com/users/drphilmarshall/repos","events_url":"https://api.github.com/users/drphilmarshall/events{/privacy}","received_events_url":"https://api.github.com/users/drphilmarshall/received_events","type":"User","site_admin":false},"created_at":"2015-09-23T06:00:18Z","updated_at":"2015-09-23T06:00:18Z","body":"Yeah, I saw that - it's nice, but not as \"literate\" as my version. (Also,\nthe \"~\" operator looks like \"approximately\" rather than \"not\"...  I know\nyou just have to learn these things, but to keep things clear I went with\nthe longer, more readable version. I'll remember the \"~\" for later though -\nthanks! :-) )\n\nOn Tue, Sep 22, 2015 at 10:32 PM, Yao-Yuan Mao <notifications@github.com>\nwrote:\n\n> Cool. One small note, the code\n>\n> maskedimage[signal] = -1\n> maskedimage[outskirts] = -1\n>\n> can be written as\n>\n> maskedimage[~background] = -1\n>\n> This is slightly more readable because it reads \"setting all\n> non-background pixels to -1\".\n>\n> Similarly, later in the notebook,\n>\n> maskedimage[background] = 0\n> maskedimage[outskirts] = 0\n>\n> can be shorten to\n>\n> maskedimage[~signal] = 0\n>\n> which sets all non-signal pixels to 0\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/drphilmarshall/StatisticalMethods/pull/55#issuecomment-142498594>\n> .\n>\n"}}