GitHub PullRequestReviewCommentEvent at 2014-03-04 08:58:44
Repository: cartopy
Actor: pelson
Payload: {"comment":{"url":"https://api.github.com/repos/SciTools/cartopy/pulls/comments/10246927","id":10246927,"diff_hunk":"@@ -59,9 +59,12 @@ def srtm_composite(lon_min, lat_min, nx, ny):\n             x_img_slice = slice(i * shape[0], (i + 1) * shape[0])\n             y_img_slice = slice(j * shape[1], (j + 1) * shape[1])\n \n-            tile_img, crs, extent = srtm(bottom_left_ll[0] + j,\n+            try:\n+               tile_img, crs, extent = srtm(bottom_left_ll[0] + j,\n                                          bottom_left_ll[1] + i)\n-            img[x_img_slice, y_img_slice] = tile_img\n+               img[x_img_slice, y_img_slice] = tile_img\n+            except:","path":"lib/cartopy/io/srtm.py","position":10,"original_position":10,"commit_id":"7f5abdcf4fac466d992b6944a0c38cd08568f131","original_commit_id":"7f5abdcf4fac466d992b6944a0c38cd08568f131","user":{"login":"pelson","id":810663,"avatar_url":"https://avatars.githubusercontent.com/u/810663","gravatar_id":"a3a8307ffcd780e908d042bcc97cd2f8","url":"https://api.github.com/users/pelson","html_url":"https://github.com/pelson","followers_url":"https://api.github.com/users/pelson/followers","following_url":"https://api.github.com/users/pelson/following{/other_user}","gists_url":"https://api.github.com/users/pelson/gists{/gist_id}","starred_url":"https://api.github.com/users/pelson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pelson/subscriptions","organizations_url":"https://api.github.com/users/pelson/orgs","repos_url":"https://api.github.com/users/pelson/repos","events_url":"https://api.github.com/users/pelson/events{/privacy}","received_events_url":"https://api.github.com/users/pelson/received_events","type":"User","site_admin":false},"body":"I'd rather this wasn't a bare except, and the specific exception was handled (was it an IndexError?).","created_at":"2014-03-04T08:58:44Z","updated_at":"2014-03-04T08:58:44Z","html_url":"https://github.com/SciTools/cartopy/pull/395#discussion_r10246927","pull_request_url":"https://api.github.com/repos/SciTools/cartopy/pulls/395","_links":{"self":{"href":"https://api.github.com/repos/SciTools/cartopy/pulls/comments/10246927"},"html":{"href":"https://github.com/SciTools/cartopy/pull/395#discussion_r10246927"},"pull_request":{"href":"https://api.github.com/repos/SciTools/cartopy/pulls/395"}}}}