{"action":"created","comment":{"url":"https://api.github.com/repos/Slicer/Slicer/pulls/comments/26520706","id":26520706,"diff_hunk":"@@ -312,6 +312,93 @@ void vtkMRMLApplicationLogic::PropagateVolumeSelection(int fit)\n     }\n }\n \n+//----------------------------------------------------------------------------\n+void vtkMRMLApplicationLogic::PropagateBackgroundVolumeSelection(int fit)","path":"Libs/MRML/Logic/vtkMRMLApplicationLogic.cxx","position":5,"original_position":5,"commit_id":"0b02b0a85b787106c5ab392a13a12d1807047f6f","original_commit_id":"0b02b0a85b787106c5ab392a13a12d1807047f6f","user":{"login":"pieper","id":126077,"avatar_url":"https://avatars.githubusercontent.com/u/126077?v=3","gravatar_id":"","url":"https://api.github.com/users/pieper","html_url":"https://github.com/pieper","followers_url":"https://api.github.com/users/pieper/followers","following_url":"https://api.github.com/users/pieper/following{/other_user}","gists_url":"https://api.github.com/users/pieper/gists{/gist_id}","starred_url":"https://api.github.com/users/pieper/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pieper/subscriptions","organizations_url":"https://api.github.com/users/pieper/orgs","repos_url":"https://api.github.com/users/pieper/repos","events_url":"https://api.github.com/users/pieper/events{/privacy}","received_events_url":"https://api.github.com/users/pieper/received_events","type":"User","site_admin":false},"body":"I want the separate methods in the API, but I want to avoid the duplication in the code.  So the only confusion is I didn't explain that well : )","created_at":"2015-03-16T19:49:29Z","updated_at":"2015-03-16T19:49:29Z","html_url":"https://github.com/Slicer/Slicer/pull/248#discussion_r26520706","pull_request_url":"https://api.github.com/repos/Slicer/Slicer/pulls/248","_links":{"self":{"href":"https://api.github.com/repos/Slicer/Slicer/pulls/comments/26520706"},"html":{"href":"https://github.com/Slicer/Slicer/pull/248#discussion_r26520706"},"pull_request":{"href":"https://api.github.com/repos/Slicer/Slicer/pulls/248"}}},"pull_request":{"url":"https://api.github.com/repos/Slicer/Slicer/pulls/248","id":31262081,"html_url":"https://github.com/Slicer/Slicer/pull/248","diff_url":"https://github.com/Slicer/Slicer/pull/248.diff","patch_url":"https://github.com/Slicer/Slicer/pull/248.patch","issue_url":"https://api.github.com/repos/Slicer/Slicer/issues/248","number":248,"state":"open","locked":false,"title":"ENH: Added option to control the PropagateVolumeSelection behavior","user":{"login":"mehrtash","id":2693004,"avatar_url":"https://avatars.githubusercontent.com/u/2693004?v=3","gravatar_id":"","url":"https://api.github.com/users/mehrtash","html_url":"https://github.com/mehrtash","followers_url":"https://api.github.com/users/mehrtash/followers","following_url":"https://api.github.com/users/mehrtash/following{/other_user}","gists_url":"https://api.github.com/users/mehrtash/gists{/gist_id}","starred_url":"https://api.github.com/users/mehrtash/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mehrtash/subscriptions","organizations_url":"https://api.github.com/users/mehrtash/orgs","repos_url":"https://api.github.com/users/mehrtash/repos","events_url":"https://api.github.com/users/mehrtash/events{/privacy}","received_events_url":"https://api.github.com/users/mehrtash/received_events","type":"User","site_admin":false},"body":"Added the following features:\r\n* Volume selection can propagate selectively to background, foreground and label layers.\r\n* HelperBox and EditUtil were modified to use the new propagation mode. Default mode will propagate through all layers while \"BackgroundAndLabel\" mode only propagates to background and label and doesn't touch foreground. \r\n\r\nThis feature is useful in cases where different foregrounds are set to different views and we don't want editor to reset foregrounds through propagate volume selection. A use case is the ongoing project PCampReview [1].\r\n\r\n[1]  [PCampReview](https://github.com/fedorov/PCampReview )","created_at":"2015-03-16T16:54:56Z","updated_at":"2015-03-16T19:49:29Z","closed_at":null,"merged_at":null,"merge_commit_sha":"3ba8abbcce0c7d7a61a685cc83dbb50e98144948","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/Slicer/Slicer/pulls/248/commits","review_comments_url":"https://api.github.com/repos/Slicer/Slicer/pulls/248/comments","review_comment_url":"https://api.github.com/repos/Slicer/Slicer/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Slicer/Slicer/issues/248/comments","statuses_url":"https://api.github.com/repos/Slicer/Slicer/statuses/0b02b0a85b787106c5ab392a13a12d1807047f6f","head":{"label":"mehrtash:update-PropagateVolumeSelection","ref":"update-PropagateVolumeSelection","sha":"0b02b0a85b787106c5ab392a13a12d1807047f6f","user":{"login":"mehrtash","id":2693004,"avatar_url":"https://avatars.githubusercontent.com/u/2693004?v=3","gravatar_id":"","url":"https://api.github.com/users/mehrtash","html_url":"https://github.com/mehrtash","followers_url":"https://api.github.com/users/mehrtash/followers","following_url":"https://api.github.com/users/mehrtash/following{/other_user}","gists_url":"https://api.github.com/users/mehrtash/gists{/gist_id}","starred_url":"https://api.github.com/users/mehrtash/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mehrtash/subscriptions","organizations_url":"https://api.github.com/users/mehrtash/orgs","repos_url":"https://api.github.com/users/mehrtash/repos","events_url":"https://api.github.com/users/mehrtash/events{/privacy}","received_events_url":"https://api.github.com/users/mehrtash/received_events","type":"User","site_admin":false},"repo":{"id":12352357,"name":"Slicer","full_name":"mehrtash/Slicer","owner":{"login":"mehrtash","id":2693004,"avatar_url":"https://avatars.githubusercontent.com/u/2693004?v=3","gravatar_id":"","url":"https://api.github.com/users/mehrtash","html_url":"https://github.com/mehrtash","followers_url":"https://api.github.com/users/mehrtash/followers","following_url":"https://api.github.com/users/mehrtash/following{/other_user}","gists_url":"https://api.github.com/users/mehrtash/gists{/gist_id}","starred_url":"https://api.github.com/users/mehrtash/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mehrtash/subscriptions","organizations_url":"https://api.github.com/users/mehrtash/orgs","repos_url":"https://api.github.com/users/mehrtash/repos","events_url":"https://api.github.com/users/mehrtash/events{/privacy}","received_events_url":"https://api.github.com/users/mehrtash/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/mehrtash/Slicer","description":"Multi-platform, free open source software for visualization and image computing.","fork":true,"url":"https://api.github.com/repos/mehrtash/Slicer","forks_url":"https://api.github.com/repos/mehrtash/Slicer/forks","keys_url":"https://api.github.com/repos/mehrtash/Slicer/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mehrtash/Slicer/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mehrtash/Slicer/teams","hooks_url":"https://api.github.com/repos/mehrtash/Slicer/hooks","issue_events_url":"https://api.github.com/repos/mehrtash/Slicer/issues/events{/number}","events_url":"https://api.github.com/repos/mehrtash/Slicer/events","assignees_url":"https://api.github.com/repos/mehrtash/Slicer/assignees{/user}","branches_url":"https://api.github.com/repos/mehrtash/Slicer/branches{/branch}","tags_url":"https://api.github.com/repos/mehrtash/Slicer/tags","blobs_url":"https://api.github.com/repos/mehrtash/Slicer/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mehrtash/Slicer/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mehrtash/Slicer/git/refs{/sha}","trees_url":"https://api.github.com/repos/mehrtash/Slicer/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mehrtash/Slicer/statuses/{sha}","languages_url":"https://api.github.com/repos/mehrtash/Slicer/languages","stargazers_url":"https://api.github.com/repos/mehrtash/Slicer/stargazers","contributors_url":"https://api.github.com/repos/mehrtash/Slicer/contributors","subscribers_url":"https://api.github.com/repos/mehrtash/Slicer/subscribers","subscription_url":"https://api.github.com/repos/mehrtash/Slicer/subscription","commits_url":"https://api.github.com/repos/mehrtash/Slicer/commits{/sha}","git_commits_url":"https://api.github.com/repos/mehrtash/Slicer/git/commits{/sha}","comments_url":"https://api.github.com/repos/mehrtash/Slicer/comments{/number}","issue_comment_url":"https://api.github.com/repos/mehrtash/Slicer/issues/comments{/number}","contents_url":"https://api.github.com/repos/mehrtash/Slicer/contents/{+path}","compare_url":"https://api.github.com/repos/mehrtash/Slicer/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mehrtash/Slicer/merges","archive_url":"https://api.github.com/repos/mehrtash/Slicer/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mehrtash/Slicer/downloads","issues_url":"https://api.github.com/repos/mehrtash/Slicer/issues{/number}","pulls_url":"https://api.github.com/repos/mehrtash/Slicer/pulls{/number}","milestones_url":"https://api.github.com/repos/mehrtash/Slicer/milestones{/number}","notifications_url":"https://api.github.com/repos/mehrtash/Slicer/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mehrtash/Slicer/labels{/name}","releases_url":"https://api.github.com/repos/mehrtash/Slicer/releases{/id}","created_at":"2013-08-25T01:19:00Z","updated_at":"2015-03-13T13:34:37Z","pushed_at":"2015-03-16T16:54:28Z","git_url":"git://github.com/mehrtash/Slicer.git","ssh_url":"git@github.com:mehrtash/Slicer.git","clone_url":"https://github.com/mehrtash/Slicer.git","svn_url":"https://github.com/mehrtash/Slicer","homepage":"www.slicer.org","size":422155,"stargazers_count":0,"watchers_count":0,"language":"C++","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"Slicer:master","ref":"master","sha":"5ca7d0f5b0ef949164469a8c43f32e096dc85969","user":{"login":"Slicer","id":324362,"avatar_url":"https://avatars.githubusercontent.com/u/324362?v=3","gravatar_id":"","url":"https://api.github.com/users/Slicer","html_url":"https://github.com/Slicer","followers_url":"https://api.github.com/users/Slicer/followers","following_url":"https://api.github.com/users/Slicer/following{/other_user}","gists_url":"https://api.github.com/users/Slicer/gists{/gist_id}","starred_url":"https://api.github.com/users/Slicer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Slicer/subscriptions","organizations_url":"https://api.github.com/users/Slicer/orgs","repos_url":"https://api.github.com/users/Slicer/repos","events_url":"https://api.github.com/users/Slicer/events{/privacy}","received_events_url":"https://api.github.com/users/Slicer/received_events","type":"Organization","site_admin":false},"repo":{"id":2006302,"name":"Slicer","full_name":"Slicer/Slicer","owner":{"login":"Slicer","id":324362,"avatar_url":"https://avatars.githubusercontent.com/u/324362?v=3","gravatar_id":"","url":"https://api.github.com/users/Slicer","html_url":"https://github.com/Slicer","followers_url":"https://api.github.com/users/Slicer/followers","following_url":"https://api.github.com/users/Slicer/following{/other_user}","gists_url":"https://api.github.com/users/Slicer/gists{/gist_id}","starred_url":"https://api.github.com/users/Slicer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Slicer/subscriptions","organizations_url":"https://api.github.com/users/Slicer/orgs","repos_url":"https://api.github.com/users/Slicer/repos","events_url":"https://api.github.com/users/Slicer/events{/privacy}","received_events_url":"https://api.github.com/users/Slicer/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Slicer/Slicer","description":"Multi-platform, free open source software for visualization and image computing.","fork":false,"url":"https://api.github.com/repos/Slicer/Slicer","forks_url":"https://api.github.com/repos/Slicer/Slicer/forks","keys_url":"https://api.github.com/repos/Slicer/Slicer/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Slicer/Slicer/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Slicer/Slicer/teams","hooks_url":"https://api.github.com/repos/Slicer/Slicer/hooks","issue_events_url":"https://api.github.com/repos/Slicer/Slicer/issues/events{/number}","events_url":"https://api.github.com/repos/Slicer/Slicer/events","assignees_url":"https://api.github.com/repos/Slicer/Slicer/assignees{/user}","branches_url":"https://api.github.com/repos/Slicer/Slicer/branches{/branch}","tags_url":"https://api.github.com/repos/Slicer/Slicer/tags","blobs_url":"https://api.github.com/repos/Slicer/Slicer/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Slicer/Slicer/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Slicer/Slicer/git/refs{/sha}","trees_url":"https://api.github.com/repos/Slicer/Slicer/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Slicer/Slicer/statuses/{sha}","languages_url":"https://api.github.com/repos/Slicer/Slicer/languages","stargazers_url":"https://api.github.com/repos/Slicer/Slicer/stargazers","contributors_url":"https://api.github.com/repos/Slicer/Slicer/contributors","subscribers_url":"https://api.github.com/repos/Slicer/Slicer/subscribers","subscription_url":"https://api.github.com/repos/Slicer/Slicer/subscription","commits_url":"https://api.github.com/repos/Slicer/Slicer/commits{/sha}","git_commits_url":"https://api.github.com/repos/Slicer/Slicer/git/commits{/sha}","comments_url":"https://api.github.com/repos/Slicer/Slicer/comments{/number}","issue_comment_url":"https://api.github.com/repos/Slicer/Slicer/issues/comments{/number}","contents_url":"https://api.github.com/repos/Slicer/Slicer/contents/{+path}","compare_url":"https://api.github.com/repos/Slicer/Slicer/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Slicer/Slicer/merges","archive_url":"https://api.github.com/repos/Slicer/Slicer/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Slicer/Slicer/downloads","issues_url":"https://api.github.com/repos/Slicer/Slicer/issues{/number}","pulls_url":"https://api.github.com/repos/Slicer/Slicer/pulls{/number}","milestones_url":"https://api.github.com/repos/Slicer/Slicer/milestones{/number}","notifications_url":"https://api.github.com/repos/Slicer/Slicer/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Slicer/Slicer/labels{/name}","releases_url":"https://api.github.com/repos/Slicer/Slicer/releases{/id}","created_at":"2011-07-06T12:27:18Z","updated_at":"2015-03-16T13:31:11Z","pushed_at":"2015-03-15T00:52:04Z","git_url":"git://github.com/Slicer/Slicer.git","ssh_url":"git@github.com:Slicer/Slicer.git","clone_url":"https://github.com/Slicer/Slicer.git","svn_url":"https://github.com/Slicer/Slicer","homepage":"www.slicer.org","size":437901,"stargazers_count":129,"watchers_count":129,"language":"C++","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":166,"mirror_url":null,"open_issues_count":4,"forks":166,"open_issues":4,"watchers":129,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Slicer/Slicer/pulls/248"},"html":{"href":"https://github.com/Slicer/Slicer/pull/248"},"issue":{"href":"https://api.github.com/repos/Slicer/Slicer/issues/248"},"comments":{"href":"https://api.github.com/repos/Slicer/Slicer/issues/248/comments"},"review_comments":{"href":"https://api.github.com/repos/Slicer/Slicer/pulls/248/comments"},"review_comment":{"href":"https://api.github.com/repos/Slicer/Slicer/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Slicer/Slicer/pulls/248/commits"},"statuses":{"href":"https://api.github.com/repos/Slicer/Slicer/statuses/0b02b0a85b787106c5ab392a13a12d1807047f6f"}}}}