{"action":"created","comment":{"url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/comments/84318391","pull_request_review_id":5103492,"id":84318391,"diff_hunk":"@@ -283,12 +283,27 @@ def cluster_commands_single(options_to_ignore={}) #:nodoc:\n     args += \"-multispectral \"                                       if mybool(file0[:multispectral])\n     args += \"-spectral_mask \"                                       if mybool(file0[:spectral_mask])\n \n+    # PVE\n+    if self.tool_config.is_at_least_version(\"2.1.0\")\n+      if    params[:pve] == \"classic\"\n+        args += \"-no-correct-pve -no-subcortical -no-mask-cerebellum \"\n+      elsif params[:pve] == \"advanced\"\n+        args += \"-correct-pve -subcortical -mask-cerebellum \"\n+      end\n+    end\n+\n     # Thickness methods and kernel\n     if ( params[:thickness_method].present? &&\n          params[:thickness_kernel].present? &&\n          is_valid_integer_list(params[:thickness_kernel], allow_blanks: false)\n        )\n-      args += \"-thickness #{params[:thickness_method].bash_escape} #{params[:thickness_kernel].bash_escape} \"\n+      if self.tool_config.is_at_least_version(\"2.1.0\")\n+        thickness_methods = params[:thickness_method].unshift(params[:thickness_method_for_qc]) if params[:thickness_method_for_qc].present?","path":"cbrain_task/civet/bourreau/civet.rb","position":33,"original_position":33,"commit_id":"001f1ffe42ee469a51930502fb673a346800f052","original_commit_id":"001f1ffe42ee469a51930502fb673a346800f052","user":{"login":"prioux","id":777588,"avatar_url":"https://avatars.githubusercontent.com/u/777588?v=3","gravatar_id":"","url":"https://api.github.com/users/prioux","html_url":"https://github.com/prioux","followers_url":"https://api.github.com/users/prioux/followers","following_url":"https://api.github.com/users/prioux/following{/other_user}","gists_url":"https://api.github.com/users/prioux/gists{/gist_id}","starred_url":"https://api.github.com/users/prioux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/prioux/subscriptions","organizations_url":"https://api.github.com/users/prioux/orgs","repos_url":"https://api.github.com/users/prioux/repos","events_url":"https://api.github.com/users/prioux/events{/privacy}","received_events_url":"https://api.github.com/users/prioux/received_events","type":"User","site_admin":false},"body":"check for ambiguity/types ?","created_at":"2016-10-20T16:07:21Z","updated_at":"2016-10-20T16:07:21Z","html_url":"https://github.com/aces/cbrain-plugins-neuro/pull/74#discussion_r84318391","pull_request_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/74","_links":{"self":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/comments/84318391"},"html":{"href":"https://github.com/aces/cbrain-plugins-neuro/pull/74#discussion_r84318391"},"pull_request":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/74"}}},"pull_request":{"url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/74","id":90073033,"html_url":"https://github.com/aces/cbrain-plugins-neuro/pull/74","diff_url":"https://github.com/aces/cbrain-plugins-neuro/pull/74.diff","patch_url":"https://github.com/aces/cbrain-plugins-neuro/pull/74.patch","issue_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/issues/74","number":74,"state":"open","locked":false,"title":"Add support for CIVET 2.1.0","user":{"login":"natacha-beck","id":777590,"avatar_url":"https://avatars.githubusercontent.com/u/777590?v=3","gravatar_id":"","url":"https://api.github.com/users/natacha-beck","html_url":"https://github.com/natacha-beck","followers_url":"https://api.github.com/users/natacha-beck/followers","following_url":"https://api.github.com/users/natacha-beck/following{/other_user}","gists_url":"https://api.github.com/users/natacha-beck/gists{/gist_id}","starred_url":"https://api.github.com/users/natacha-beck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/natacha-beck/subscriptions","organizations_url":"https://api.github.com/users/natacha-beck/orgs","repos_url":"https://api.github.com/users/natacha-beck/repos","events_url":"https://api.github.com/users/natacha-beck/events{/privacy}","received_events_url":"https://api.github.com/users/natacha-beck/received_events","type":"User","site_admin":false},"body":"","created_at":"2016-10-19T19:18:24Z","updated_at":"2016-10-20T16:15:07Z","closed_at":null,"merged_at":null,"merge_commit_sha":"5daeba593bba71e80c8e620ab53c08dff65ddca5","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/74/commits","review_comments_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/74/comments","review_comment_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/comments{/number}","comments_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/issues/74/comments","statuses_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/statuses/001f1ffe42ee469a51930502fb673a346800f052","head":{"label":"natacha-beck:civet_2_1_0","ref":"civet_2_1_0","sha":"001f1ffe42ee469a51930502fb673a346800f052","user":{"login":"natacha-beck","id":777590,"avatar_url":"https://avatars.githubusercontent.com/u/777590?v=3","gravatar_id":"","url":"https://api.github.com/users/natacha-beck","html_url":"https://github.com/natacha-beck","followers_url":"https://api.github.com/users/natacha-beck/followers","following_url":"https://api.github.com/users/natacha-beck/following{/other_user}","gists_url":"https://api.github.com/users/natacha-beck/gists{/gist_id}","starred_url":"https://api.github.com/users/natacha-beck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/natacha-beck/subscriptions","organizations_url":"https://api.github.com/users/natacha-beck/orgs","repos_url":"https://api.github.com/users/natacha-beck/repos","events_url":"https://api.github.com/users/natacha-beck/events{/privacy}","received_events_url":"https://api.github.com/users/natacha-beck/received_events","type":"User","site_admin":false},"repo":{"id":52912592,"name":"cbrain-plugins-neuro","full_name":"natacha-beck/cbrain-plugins-neuro","owner":{"login":"natacha-beck","id":777590,"avatar_url":"https://avatars.githubusercontent.com/u/777590?v=3","gravatar_id":"","url":"https://api.github.com/users/natacha-beck","html_url":"https://github.com/natacha-beck","followers_url":"https://api.github.com/users/natacha-beck/followers","following_url":"https://api.github.com/users/natacha-beck/following{/other_user}","gists_url":"https://api.github.com/users/natacha-beck/gists{/gist_id}","starred_url":"https://api.github.com/users/natacha-beck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/natacha-beck/subscriptions","organizations_url":"https://api.github.com/users/natacha-beck/orgs","repos_url":"https://api.github.com/users/natacha-beck/repos","events_url":"https://api.github.com/users/natacha-beck/events{/privacy}","received_events_url":"https://api.github.com/users/natacha-beck/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/natacha-beck/cbrain-plugins-neuro","description":null,"fork":true,"url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro","forks_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/forks","keys_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/keys{/key_id}","collaborators_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/teams","hooks_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/hooks","issue_events_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/issues/events{/number}","events_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/events","assignees_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/assignees{/user}","branches_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/branches{/branch}","tags_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/tags","blobs_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/git/refs{/sha}","trees_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/git/trees{/sha}","statuses_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/statuses/{sha}","languages_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/languages","stargazers_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/stargazers","contributors_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/contributors","subscribers_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/subscribers","subscription_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/subscription","commits_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/commits{/sha}","git_commits_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/git/commits{/sha}","comments_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/comments{/number}","issue_comment_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/issues/comments{/number}","contents_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/contents/{+path}","compare_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/compare/{base}...{head}","merges_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/merges","archive_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/downloads","issues_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/issues{/number}","pulls_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/pulls{/number}","milestones_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/milestones{/number}","notifications_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/labels{/name}","releases_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/releases{/id}","deployments_url":"https://api.github.com/repos/natacha-beck/cbrain-plugins-neuro/deployments","created_at":"2016-03-01T21:49:13Z","updated_at":"2016-03-01T21:49:15Z","pushed_at":"2016-10-19T19:17:08Z","git_url":"git://github.com/natacha-beck/cbrain-plugins-neuro.git","ssh_url":"git@github.com:natacha-beck/cbrain-plugins-neuro.git","clone_url":"https://github.com/natacha-beck/cbrain-plugins-neuro.git","svn_url":"https://github.com/natacha-beck/cbrain-plugins-neuro","homepage":null,"size":2225,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":false,"has_downloads":true,"has_wiki":true,"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":"aces:dev","ref":"dev","sha":"1f4c438e8b7108671519533dfc474884dac78ef5","user":{"login":"aces","id":472868,"avatar_url":"https://avatars.githubusercontent.com/u/472868?v=3","gravatar_id":"","url":"https://api.github.com/users/aces","html_url":"https://github.com/aces","followers_url":"https://api.github.com/users/aces/followers","following_url":"https://api.github.com/users/aces/following{/other_user}","gists_url":"https://api.github.com/users/aces/gists{/gist_id}","starred_url":"https://api.github.com/users/aces/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aces/subscriptions","organizations_url":"https://api.github.com/users/aces/orgs","repos_url":"https://api.github.com/users/aces/repos","events_url":"https://api.github.com/users/aces/events{/privacy}","received_events_url":"https://api.github.com/users/aces/received_events","type":"Organization","site_admin":false},"repo":{"id":41695010,"name":"cbrain-plugins-neuro","full_name":"aces/cbrain-plugins-neuro","owner":{"login":"aces","id":472868,"avatar_url":"https://avatars.githubusercontent.com/u/472868?v=3","gravatar_id":"","url":"https://api.github.com/users/aces","html_url":"https://github.com/aces","followers_url":"https://api.github.com/users/aces/followers","following_url":"https://api.github.com/users/aces/following{/other_user}","gists_url":"https://api.github.com/users/aces/gists{/gist_id}","starred_url":"https://api.github.com/users/aces/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aces/subscriptions","organizations_url":"https://api.github.com/users/aces/orgs","repos_url":"https://api.github.com/users/aces/repos","events_url":"https://api.github.com/users/aces/events{/privacy}","received_events_url":"https://api.github.com/users/aces/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/aces/cbrain-plugins-neuro","description":null,"fork":false,"url":"https://api.github.com/repos/aces/cbrain-plugins-neuro","forks_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/forks","keys_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/keys{/key_id}","collaborators_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/teams","hooks_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/hooks","issue_events_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/issues/events{/number}","events_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/events","assignees_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/assignees{/user}","branches_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/branches{/branch}","tags_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/tags","blobs_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/git/refs{/sha}","trees_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/git/trees{/sha}","statuses_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/statuses/{sha}","languages_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/languages","stargazers_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/stargazers","contributors_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/contributors","subscribers_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/subscribers","subscription_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/subscription","commits_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/commits{/sha}","git_commits_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/git/commits{/sha}","comments_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/comments{/number}","issue_comment_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/issues/comments{/number}","contents_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/contents/{+path}","compare_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/compare/{base}...{head}","merges_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/merges","archive_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/downloads","issues_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/issues{/number}","pulls_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls{/number}","milestones_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/milestones{/number}","notifications_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/labels{/name}","releases_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/releases{/id}","deployments_url":"https://api.github.com/repos/aces/cbrain-plugins-neuro/deployments","created_at":"2015-08-31T19:09:29Z","updated_at":"2016-01-13T16:52:07Z","pushed_at":"2016-10-19T19:18:24Z","git_url":"git://github.com/aces/cbrain-plugins-neuro.git","ssh_url":"git@github.com:aces/cbrain-plugins-neuro.git","clone_url":"https://github.com/aces/cbrain-plugins-neuro.git","svn_url":"https://github.com/aces/cbrain-plugins-neuro","homepage":null,"size":2224,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":5,"mirror_url":null,"open_issues_count":28,"forks":5,"open_issues":28,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/74"},"html":{"href":"https://github.com/aces/cbrain-plugins-neuro/pull/74"},"issue":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/issues/74"},"comments":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/issues/74/comments"},"review_comments":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/74/comments"},"review_comment":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/pulls/74/commits"},"statuses":{"href":"https://api.github.com/repos/aces/cbrain-plugins-neuro/statuses/001f1ffe42ee469a51930502fb673a346800f052"}}}}
{"actor":{"display_login":"prioux"}}