{"action":"created","comment":{"url":"https://api.github.com/repos/nipy/nibabel/pulls/comments/89675827","pull_request_review_id":10218768,"id":89675827,"diff_hunk":"@@ -641,6 +687,11 @@ def test_lazy_tractogram_getitem(self):\n         assert_raises(NotImplementedError,\n                       DATA['lazy_tractogram'].__getitem__, 0)\n \n+    def test_lazy_tractogram_extend(self):\n+        t = DATA['lazy_tractogram'].copy()\n+        new_t = DATA['lazy_tractogram'].copy()\n+        assert_raises(NotImplementedError, new_t.__iadd__, t)","path":"nibabel/streamlines/tests/test_tractogram.py","position":74,"original_position":74,"commit_id":"bd3f921b1fd6908be30e12eb509f2add0b194c0a","original_commit_id":"bd3f921b1fd6908be30e12eb509f2add0b194c0a","user":{"login":"matthew-brett","id":67612,"avatar_url":"https://avatars.githubusercontent.com/u/67612?v=3","gravatar_id":"","url":"https://api.github.com/users/matthew-brett","html_url":"https://github.com/matthew-brett","followers_url":"https://api.github.com/users/matthew-brett/followers","following_url":"https://api.github.com/users/matthew-brett/following{/other_user}","gists_url":"https://api.github.com/users/matthew-brett/gists{/gist_id}","starred_url":"https://api.github.com/users/matthew-brett/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthew-brett/subscriptions","organizations_url":"https://api.github.com/users/matthew-brett/orgs","repos_url":"https://api.github.com/users/matthew-brett/repos","events_url":"https://api.github.com/users/matthew-brett/events{/privacy}","received_events_url":"https://api.github.com/users/matthew-brett/received_events","type":"User","site_admin":false},"body":"Check `extend` method raises?   \r\n\r\nCheck add as well, using trick like the one above?\r\n\r\n```\r\nimport operator\r\nfor op in (operator.add, operator.iadd, extender):\r\n    assert_raises(NotImplementedError, op, new_t, t)\r\n```","created_at":"2016-11-26T18:25:32Z","updated_at":"2016-11-26T18:33:56Z","html_url":"https://github.com/nipy/nibabel/pull/495#discussion_r89675827","pull_request_url":"https://api.github.com/repos/nipy/nibabel/pulls/495","_links":{"self":{"href":"https://api.github.com/repos/nipy/nibabel/pulls/comments/89675827"},"html":{"href":"https://github.com/nipy/nibabel/pull/495#discussion_r89675827"},"pull_request":{"href":"https://api.github.com/repos/nipy/nibabel/pulls/495"}}},"pull_request":{"url":"https://api.github.com/repos/nipy/nibabel/pulls/495","id":93014521,"html_url":"https://github.com/nipy/nibabel/pull/495","diff_url":"https://github.com/nipy/nibabel/pull/495.diff","patch_url":"https://github.com/nipy/nibabel/pull/495.patch","issue_url":"https://api.github.com/repos/nipy/nibabel/issues/495","number":495,"state":"open","locked":false,"title":"ENH: Support + and += operators for Tractogram","user":{"login":"MarcCote","id":660004,"avatar_url":"https://avatars.githubusercontent.com/u/660004?v=3","gravatar_id":"","url":"https://api.github.com/users/MarcCote","html_url":"https://github.com/MarcCote","followers_url":"https://api.github.com/users/MarcCote/followers","following_url":"https://api.github.com/users/MarcCote/following{/other_user}","gists_url":"https://api.github.com/users/MarcCote/gists{/gist_id}","starred_url":"https://api.github.com/users/MarcCote/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MarcCote/subscriptions","organizations_url":"https://api.github.com/users/MarcCote/orgs","repos_url":"https://api.github.com/users/MarcCote/repos","events_url":"https://api.github.com/users/MarcCote/events{/privacy}","received_events_url":"https://api.github.com/users/MarcCote/received_events","type":"User","site_admin":false},"body":"This PR adds the functionality of concatenating two Tractogram objects using either `tractogram += other_tractogram` or `tractogram = tractogram1 + tractogram2`.\r\n\r\nThis will definitively interest @jchoude, @arnaudbore, @Garyfallidis, @FrancoisRheaultUS and many others.","created_at":"2016-11-09T18:31:07Z","updated_at":"2016-11-26T18:40:01Z","closed_at":null,"merged_at":null,"merge_commit_sha":"6677b6bfa5c0d669514c4c2ab80cc63b6d6a6059","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/nipy/nibabel/pulls/495/commits","review_comments_url":"https://api.github.com/repos/nipy/nibabel/pulls/495/comments","review_comment_url":"https://api.github.com/repos/nipy/nibabel/pulls/comments{/number}","comments_url":"https://api.github.com/repos/nipy/nibabel/issues/495/comments","statuses_url":"https://api.github.com/repos/nipy/nibabel/statuses/bd3f921b1fd6908be30e12eb509f2add0b194c0a","head":{"label":"MarcCote:enh_tractogram_operators","ref":"enh_tractogram_operators","sha":"bd3f921b1fd6908be30e12eb509f2add0b194c0a","user":{"login":"MarcCote","id":660004,"avatar_url":"https://avatars.githubusercontent.com/u/660004?v=3","gravatar_id":"","url":"https://api.github.com/users/MarcCote","html_url":"https://github.com/MarcCote","followers_url":"https://api.github.com/users/MarcCote/followers","following_url":"https://api.github.com/users/MarcCote/following{/other_user}","gists_url":"https://api.github.com/users/MarcCote/gists{/gist_id}","starred_url":"https://api.github.com/users/MarcCote/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MarcCote/subscriptions","organizations_url":"https://api.github.com/users/MarcCote/orgs","repos_url":"https://api.github.com/users/MarcCote/repos","events_url":"https://api.github.com/users/MarcCote/events{/privacy}","received_events_url":"https://api.github.com/users/MarcCote/received_events","type":"User","site_admin":false},"repo":{"id":12565378,"name":"nibabel","full_name":"MarcCote/nibabel","owner":{"login":"MarcCote","id":660004,"avatar_url":"https://avatars.githubusercontent.com/u/660004?v=3","gravatar_id":"","url":"https://api.github.com/users/MarcCote","html_url":"https://github.com/MarcCote","followers_url":"https://api.github.com/users/MarcCote/followers","following_url":"https://api.github.com/users/MarcCote/following{/other_user}","gists_url":"https://api.github.com/users/MarcCote/gists{/gist_id}","starred_url":"https://api.github.com/users/MarcCote/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MarcCote/subscriptions","organizations_url":"https://api.github.com/users/MarcCote/orgs","repos_url":"https://api.github.com/users/MarcCote/repos","events_url":"https://api.github.com/users/MarcCote/events{/privacy}","received_events_url":"https://api.github.com/users/MarcCote/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/MarcCote/nibabel","description":"Python package to access a cacophony of neuro-imaging file formats","fork":true,"url":"https://api.github.com/repos/MarcCote/nibabel","forks_url":"https://api.github.com/repos/MarcCote/nibabel/forks","keys_url":"https://api.github.com/repos/MarcCote/nibabel/keys{/key_id}","collaborators_url":"https://api.github.com/repos/MarcCote/nibabel/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/MarcCote/nibabel/teams","hooks_url":"https://api.github.com/repos/MarcCote/nibabel/hooks","issue_events_url":"https://api.github.com/repos/MarcCote/nibabel/issues/events{/number}","events_url":"https://api.github.com/repos/MarcCote/nibabel/events","assignees_url":"https://api.github.com/repos/MarcCote/nibabel/assignees{/user}","branches_url":"https://api.github.com/repos/MarcCote/nibabel/branches{/branch}","tags_url":"https://api.github.com/repos/MarcCote/nibabel/tags","blobs_url":"https://api.github.com/repos/MarcCote/nibabel/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/MarcCote/nibabel/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/MarcCote/nibabel/git/refs{/sha}","trees_url":"https://api.github.com/repos/MarcCote/nibabel/git/trees{/sha}","statuses_url":"https://api.github.com/repos/MarcCote/nibabel/statuses/{sha}","languages_url":"https://api.github.com/repos/MarcCote/nibabel/languages","stargazers_url":"https://api.github.com/repos/MarcCote/nibabel/stargazers","contributors_url":"https://api.github.com/repos/MarcCote/nibabel/contributors","subscribers_url":"https://api.github.com/repos/MarcCote/nibabel/subscribers","subscription_url":"https://api.github.com/repos/MarcCote/nibabel/subscription","commits_url":"https://api.github.com/repos/MarcCote/nibabel/commits{/sha}","git_commits_url":"https://api.github.com/repos/MarcCote/nibabel/git/commits{/sha}","comments_url":"https://api.github.com/repos/MarcCote/nibabel/comments{/number}","issue_comment_url":"https://api.github.com/repos/MarcCote/nibabel/issues/comments{/number}","contents_url":"https://api.github.com/repos/MarcCote/nibabel/contents/{+path}","compare_url":"https://api.github.com/repos/MarcCote/nibabel/compare/{base}...{head}","merges_url":"https://api.github.com/repos/MarcCote/nibabel/merges","archive_url":"https://api.github.com/repos/MarcCote/nibabel/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/MarcCote/nibabel/downloads","issues_url":"https://api.github.com/repos/MarcCote/nibabel/issues{/number}","pulls_url":"https://api.github.com/repos/MarcCote/nibabel/pulls{/number}","milestones_url":"https://api.github.com/repos/MarcCote/nibabel/milestones{/number}","notifications_url":"https://api.github.com/repos/MarcCote/nibabel/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/MarcCote/nibabel/labels{/name}","releases_url":"https://api.github.com/repos/MarcCote/nibabel/releases{/id}","deployments_url":"https://api.github.com/repos/MarcCote/nibabel/deployments","created_at":"2013-09-03T14:05:03Z","updated_at":"2015-02-19T12:09:52Z","pushed_at":"2016-11-09T18:26:44Z","git_url":"git://github.com/MarcCote/nibabel.git","ssh_url":"git@github.com:MarcCote/nibabel.git","clone_url":"https://github.com/MarcCote/nibabel.git","svn_url":"https://github.com/MarcCote/nibabel","homepage":"http://nipy.org/nibabel/","size":8264,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"open_issues_count":0,"forks":1,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"nipy:master","ref":"master","sha":"ec4567fb09b4472c5a4bb9a13dbcc9eb0a63d875","user":{"login":"nipy","id":233707,"avatar_url":"https://avatars.githubusercontent.com/u/233707?v=3","gravatar_id":"","url":"https://api.github.com/users/nipy","html_url":"https://github.com/nipy","followers_url":"https://api.github.com/users/nipy/followers","following_url":"https://api.github.com/users/nipy/following{/other_user}","gists_url":"https://api.github.com/users/nipy/gists{/gist_id}","starred_url":"https://api.github.com/users/nipy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nipy/subscriptions","organizations_url":"https://api.github.com/users/nipy/orgs","repos_url":"https://api.github.com/users/nipy/repos","events_url":"https://api.github.com/users/nipy/events{/privacy}","received_events_url":"https://api.github.com/users/nipy/received_events","type":"Organization","site_admin":false},"repo":{"id":791352,"name":"nibabel","full_name":"nipy/nibabel","owner":{"login":"nipy","id":233707,"avatar_url":"https://avatars.githubusercontent.com/u/233707?v=3","gravatar_id":"","url":"https://api.github.com/users/nipy","html_url":"https://github.com/nipy","followers_url":"https://api.github.com/users/nipy/followers","following_url":"https://api.github.com/users/nipy/following{/other_user}","gists_url":"https://api.github.com/users/nipy/gists{/gist_id}","starred_url":"https://api.github.com/users/nipy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nipy/subscriptions","organizations_url":"https://api.github.com/users/nipy/orgs","repos_url":"https://api.github.com/users/nipy/repos","events_url":"https://api.github.com/users/nipy/events{/privacy}","received_events_url":"https://api.github.com/users/nipy/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/nipy/nibabel","description":"Python package to access a cacophony of neuro-imaging file formats","fork":false,"url":"https://api.github.com/repos/nipy/nibabel","forks_url":"https://api.github.com/repos/nipy/nibabel/forks","keys_url":"https://api.github.com/repos/nipy/nibabel/keys{/key_id}","collaborators_url":"https://api.github.com/repos/nipy/nibabel/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/nipy/nibabel/teams","hooks_url":"https://api.github.com/repos/nipy/nibabel/hooks","issue_events_url":"https://api.github.com/repos/nipy/nibabel/issues/events{/number}","events_url":"https://api.github.com/repos/nipy/nibabel/events","assignees_url":"https://api.github.com/repos/nipy/nibabel/assignees{/user}","branches_url":"https://api.github.com/repos/nipy/nibabel/branches{/branch}","tags_url":"https://api.github.com/repos/nipy/nibabel/tags","blobs_url":"https://api.github.com/repos/nipy/nibabel/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/nipy/nibabel/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/nipy/nibabel/git/refs{/sha}","trees_url":"https://api.github.com/repos/nipy/nibabel/git/trees{/sha}","statuses_url":"https://api.github.com/repos/nipy/nibabel/statuses/{sha}","languages_url":"https://api.github.com/repos/nipy/nibabel/languages","stargazers_url":"https://api.github.com/repos/nipy/nibabel/stargazers","contributors_url":"https://api.github.com/repos/nipy/nibabel/contributors","subscribers_url":"https://api.github.com/repos/nipy/nibabel/subscribers","subscription_url":"https://api.github.com/repos/nipy/nibabel/subscription","commits_url":"https://api.github.com/repos/nipy/nibabel/commits{/sha}","git_commits_url":"https://api.github.com/repos/nipy/nibabel/git/commits{/sha}","comments_url":"https://api.github.com/repos/nipy/nibabel/comments{/number}","issue_comment_url":"https://api.github.com/repos/nipy/nibabel/issues/comments{/number}","contents_url":"https://api.github.com/repos/nipy/nibabel/contents/{+path}","compare_url":"https://api.github.com/repos/nipy/nibabel/compare/{base}...{head}","merges_url":"https://api.github.com/repos/nipy/nibabel/merges","archive_url":"https://api.github.com/repos/nipy/nibabel/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/nipy/nibabel/downloads","issues_url":"https://api.github.com/repos/nipy/nibabel/issues{/number}","pulls_url":"https://api.github.com/repos/nipy/nibabel/pulls{/number}","milestones_url":"https://api.github.com/repos/nipy/nibabel/milestones{/number}","notifications_url":"https://api.github.com/repos/nipy/nibabel/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/nipy/nibabel/labels{/name}","releases_url":"https://api.github.com/repos/nipy/nibabel/releases{/id}","deployments_url":"https://api.github.com/repos/nipy/nibabel/deployments","created_at":"2010-07-22T16:28:30Z","updated_at":"2016-11-23T15:40:52Z","pushed_at":"2016-11-11T19:50:29Z","git_url":"git://github.com/nipy/nibabel.git","ssh_url":"git@github.com:nipy/nibabel.git","clone_url":"https://github.com/nipy/nibabel.git","svn_url":"https://github.com/nipy/nibabel","homepage":"http://nipy.org/nibabel/","size":21080,"stargazers_count":121,"watchers_count":121,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":112,"mirror_url":null,"open_issues_count":70,"forks":112,"open_issues":70,"watchers":121,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/nipy/nibabel/pulls/495"},"html":{"href":"https://github.com/nipy/nibabel/pull/495"},"issue":{"href":"https://api.github.com/repos/nipy/nibabel/issues/495"},"comments":{"href":"https://api.github.com/repos/nipy/nibabel/issues/495/comments"},"review_comments":{"href":"https://api.github.com/repos/nipy/nibabel/pulls/495/comments"},"review_comment":{"href":"https://api.github.com/repos/nipy/nibabel/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/nipy/nibabel/pulls/495/commits"},"statuses":{"href":"https://api.github.com/repos/nipy/nibabel/statuses/bd3f921b1fd6908be30e12eb509f2add0b194c0a"}}}}
{"actor":{"display_login":"matthew-brett"}}