GitHub PullRequestReviewCommentEvent at 2014-02-05 01:40:42
Repository: scipy
Actor: ligon
Payload: {"comment":{"url":"https://api.github.com/repos/ligon/scipy/pulls/comments/9447584","id":9447584,"diff_hunk":"@@ -954,12 +952,8 @@ def check(func, dist, args, alpha):\n             # Only check the length of the return\n             # FIXME: should check the actual results to see if we are 'close'\n             #   to what was created --- but what is 'close' enough\n-            if dist == 'frechet':\n-                assert_(len(vals) == len(args))\n-                assert_(len(vals2) == len(args))\n-            else:\n-                assert_(len(vals) == 2+len(args))\n-                assert_(len(vals2) == 2+len(args))\n+            assert_(len(vals) == 2+len(args))","path":"scipy/stats/tests/test_distributions.py","position":142,"original_position":142,"commit_id":"01d2bc0b8f316bc1f6bf74727217020e54de1b15","original_commit_id":"01d2bc0b8f316bc1f6bf74727217020e54de1b15","user":{"login":"ligon","id":605452,"avatar_url":"https://gravatar.com/avatar/fae6513144be2078c2286672ca8bd904?d=https%3A%2F%2Fidenticons.github.com%2Fcc6d127f75b326f3ef78ceab3cd1a5d6.png&r=x","gravatar_id":"fae6513144be2078c2286672ca8bd904","url":"https://api.github.com/users/ligon","html_url":"https://github.com/ligon","followers_url":"https://api.github.com/users/ligon/followers","following_url":"https://api.github.com/users/ligon/following{/other_user}","gists_url":"https://api.github.com/users/ligon/gists{/gist_id}","starred_url":"https://api.github.com/users/ligon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ligon/subscriptions","organizations_url":"https://api.github.com/users/ligon/orgs","repos_url":"https://api.github.com/users/ligon/repos","events_url":"https://api.github.com/users/ligon/events{/privacy}","received_events_url":"https://api.github.com/users/ligon/received_events","type":"User","site_admin":false},"body":"Ha!  This one wasn't mine at all; evidence of an earlier frechet.  Git-blame tells me Ralf Gommers and Tim Leslie added this material as recently as April and May of last year.","created_at":"2014-02-05T01:40:42Z","updated_at":"2014-02-05T01:40:42Z","html_url":"https://github.com/ligon/scipy/pull/1#discussion_r9447584","pull_request_url":"https://api.github.com/repos/ligon/scipy/pulls/1","_links":{"self":{"href":"https://api.github.com/repos/ligon/scipy/pulls/comments/9447584"},"html":{"href":"https://github.com/ligon/scipy/pull/1#discussion_r9447584"},"pull_request":{"href":"https://api.github.com/repos/ligon/scipy/pulls/1"}}}}