{"action":"created","comment":{"url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/comments/165357039","pull_request_review_id":93281803,"id":165357039,"diff_hunk":"@@ -372,7 +381,215 @@ def statisticImageString(statImage):\n \t\n \t\treturn(\"Z (Gaussianised T/F)\")\n \n-\t\n+def formatClusterStats(g, excName):\n+\n+        \n+        clusterData = {}\n+        \n+        #---------------------------------------------------------------------------------------------------------\n+        #First we gather data for peaks table.\n+        #---------------------------------------------------------------------------------------------------------\n+\n+        #We must include cluster p values in the query.\n+        peak_query = \"\"\"prefix nidm_SupraThresholdCluster: <http://purl.org/nidash/nidm#NIDM_0000070>\n+                       prefix nidm_clusterSizeInVoxels: <http://purl.org/nidash/nidm#NIDM_0000084>\n+                       prefix nidm_clusterLabelID: <http://purl.org/nidash/nidm#NIDM_0000082>\n+                       prefix nidm_equivalentZStatistic: <http://purl.org/nidash/nidm#NIDM_0000092>\n+                       prefix prov: <http://www.w3.org/ns/prov#>\n+                       prefix nidm_coordinateVector: <http://purl.org/nidash/nidm#NIDM_0000086>\n+                       prefix nidm_pValueUncorrected: <http://purl.org/nidash/nidm#NIDM_0000116> \n+                       prefix nidm_pValueFWER: <http://purl.org/nidash/nidm#NIDM_0000115> \n+                       prefix nidm_qValueFDR: <http://purl.org/nidash/nidm#NIDM_0000119>\n+                       \n+                       \n+                       SELECT ?peakStat ?clus_index ?loc\n+\n+                       WHERE {{?exc a nidm_ExcursionSetMap: . ?clus prov:wasDerivedFrom ?exc . ?clus a nidm_SupraThresholdCluster: .","path":"nidmviewerfsl/viewer.py","position":71,"original_position":71,"commit_id":"98e5e800d29ef3b1eec051d8674ba4dea8a5e183","original_commit_id":"98e5e800d29ef3b1eec051d8674ba4dea8a5e183","user":{"login":"TomMaullin","id":22241320,"avatar_url":"https://avatars3.githubusercontent.com/u/22241320?v=4","gravatar_id":"","url":"https://api.github.com/users/TomMaullin","html_url":"https://github.com/TomMaullin","followers_url":"https://api.github.com/users/TomMaullin/followers","following_url":"https://api.github.com/users/TomMaullin/following{/other_user}","gists_url":"https://api.github.com/users/TomMaullin/gists{/gist_id}","starred_url":"https://api.github.com/users/TomMaullin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomMaullin/subscriptions","organizations_url":"https://api.github.com/users/TomMaullin/orgs","repos_url":"https://api.github.com/users/TomMaullin/repos","events_url":"https://api.github.com/users/TomMaullin/events{/privacy}","received_events_url":"https://api.github.com/users/TomMaullin/received_events","type":"User","site_admin":false},"body":"Ah yes! Sure! Can I suggest this change is done in the cleanUp PR however as that PR was solely for restructuring the code!","created_at":"2018-02-01T13:37:58Z","updated_at":"2018-02-01T13:37:58Z","html_url":"https://github.com/incf-nidash/nidmresults-fslhtml/pull/17#discussion_r165357039","pull_request_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/17","author_association":"COLLABORATOR","_links":{"self":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/comments/165357039"},"html":{"href":"https://github.com/incf-nidash/nidmresults-fslhtml/pull/17#discussion_r165357039"},"pull_request":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/17"}},"in_reply_to_id":165345934},"pull_request":{"url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/17","id":163712421,"html_url":"https://github.com/incf-nidash/nidmresults-fslhtml/pull/17","diff_url":"https://github.com/incf-nidash/nidmresults-fslhtml/pull/17.diff","patch_url":"https://github.com/incf-nidash/nidmresults-fslhtml/pull/17.patch","issue_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/issues/17","number":17,"state":"open","locked":false,"title":"Cluster stats","user":{"login":"TomMaullin","id":22241320,"avatar_url":"https://avatars3.githubusercontent.com/u/22241320?v=4","gravatar_id":"","url":"https://api.github.com/users/TomMaullin","html_url":"https://github.com/TomMaullin","followers_url":"https://api.github.com/users/TomMaullin/followers","following_url":"https://api.github.com/users/TomMaullin/following{/other_user}","gists_url":"https://api.github.com/users/TomMaullin/gists{/gist_id}","starred_url":"https://api.github.com/users/TomMaullin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomMaullin/subscriptions","organizations_url":"https://api.github.com/users/TomMaullin/orgs","repos_url":"https://api.github.com/users/TomMaullin/repos","events_url":"https://api.github.com/users/TomMaullin/events{/privacy}","received_events_url":"https://api.github.com/users/TomMaullin/received_events","type":"User","site_admin":false},"body":"The cluster statistics tables have been added and the tests have been updated to use the data from the 2210 repository on NeuroVault","created_at":"2018-01-18T13:32:24Z","updated_at":"2018-02-01T13:37:58Z","closed_at":null,"merged_at":null,"merge_commit_sha":"d62ac19696687df8340396187a4974221490e469","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"milestone":null,"commits_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/17/commits","review_comments_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/17/comments","review_comment_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/comments{/number}","comments_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/issues/17/comments","statuses_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/statuses/98e5e800d29ef3b1eec051d8674ba4dea8a5e183","head":{"label":"TomMaullin:clusterStats","ref":"clusterStats","sha":"98e5e800d29ef3b1eec051d8674ba4dea8a5e183","user":{"login":"TomMaullin","id":22241320,"avatar_url":"https://avatars3.githubusercontent.com/u/22241320?v=4","gravatar_id":"","url":"https://api.github.com/users/TomMaullin","html_url":"https://github.com/TomMaullin","followers_url":"https://api.github.com/users/TomMaullin/followers","following_url":"https://api.github.com/users/TomMaullin/following{/other_user}","gists_url":"https://api.github.com/users/TomMaullin/gists{/gist_id}","starred_url":"https://api.github.com/users/TomMaullin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomMaullin/subscriptions","organizations_url":"https://api.github.com/users/TomMaullin/orgs","repos_url":"https://api.github.com/users/TomMaullin/repos","events_url":"https://api.github.com/users/TomMaullin/events{/privacy}","received_events_url":"https://api.github.com/users/TomMaullin/received_events","type":"User","site_admin":false},"repo":{"id":111576782,"name":"nidmresults-fslhtml","full_name":"TomMaullin/nidmresults-fslhtml","owner":{"login":"TomMaullin","id":22241320,"avatar_url":"https://avatars3.githubusercontent.com/u/22241320?v=4","gravatar_id":"","url":"https://api.github.com/users/TomMaullin","html_url":"https://github.com/TomMaullin","followers_url":"https://api.github.com/users/TomMaullin/followers","following_url":"https://api.github.com/users/TomMaullin/following{/other_user}","gists_url":"https://api.github.com/users/TomMaullin/gists{/gist_id}","starred_url":"https://api.github.com/users/TomMaullin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TomMaullin/subscriptions","organizations_url":"https://api.github.com/users/TomMaullin/orgs","repos_url":"https://api.github.com/users/TomMaullin/repos","events_url":"https://api.github.com/users/TomMaullin/events{/privacy}","received_events_url":"https://api.github.com/users/TomMaullin/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/TomMaullin/nidmresults-fslhtml","description":"Generate a set of FSL-like HTML pages from a NIDM-Results pack","fork":true,"url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml","forks_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/forks","keys_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/keys{/key_id}","collaborators_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/teams","hooks_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/hooks","issue_events_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/issues/events{/number}","events_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/events","assignees_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/assignees{/user}","branches_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/branches{/branch}","tags_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/tags","blobs_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/git/refs{/sha}","trees_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/git/trees{/sha}","statuses_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/statuses/{sha}","languages_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/languages","stargazers_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/stargazers","contributors_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/contributors","subscribers_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/subscribers","subscription_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/subscription","commits_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/commits{/sha}","git_commits_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/git/commits{/sha}","comments_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/comments{/number}","issue_comment_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/issues/comments{/number}","contents_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/contents/{+path}","compare_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/compare/{base}...{head}","merges_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/merges","archive_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/downloads","issues_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/issues{/number}","pulls_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/pulls{/number}","milestones_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/milestones{/number}","notifications_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/labels{/name}","releases_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/releases{/id}","deployments_url":"https://api.github.com/repos/TomMaullin/nidmresults-fslhtml/deployments","created_at":"2017-11-21T16:54:42Z","updated_at":"2017-11-21T16:54:43Z","pushed_at":"2018-01-29T11:39:49Z","git_url":"git://github.com/TomMaullin/nidmresults-fslhtml.git","ssh_url":"git@github.com:TomMaullin/nidmresults-fslhtml.git","clone_url":"https://github.com/TomMaullin/nidmresults-fslhtml.git","svn_url":"https://github.com/TomMaullin/nidmresults-fslhtml","homepage":null,"size":2544,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":null,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"incf-nidash:master","ref":"master","sha":"eccf5321a3d3fa078451f814da5676c45dd2915e","user":{"login":"incf-nidash","id":3944799,"avatar_url":"https://avatars1.githubusercontent.com/u/3944799?v=4","gravatar_id":"","url":"https://api.github.com/users/incf-nidash","html_url":"https://github.com/incf-nidash","followers_url":"https://api.github.com/users/incf-nidash/followers","following_url":"https://api.github.com/users/incf-nidash/following{/other_user}","gists_url":"https://api.github.com/users/incf-nidash/gists{/gist_id}","starred_url":"https://api.github.com/users/incf-nidash/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/incf-nidash/subscriptions","organizations_url":"https://api.github.com/users/incf-nidash/orgs","repos_url":"https://api.github.com/users/incf-nidash/repos","events_url":"https://api.github.com/users/incf-nidash/events{/privacy}","received_events_url":"https://api.github.com/users/incf-nidash/received_events","type":"Organization","site_admin":false},"repo":{"id":63703677,"name":"nidmresults-fslhtml","full_name":"incf-nidash/nidmresults-fslhtml","owner":{"login":"incf-nidash","id":3944799,"avatar_url":"https://avatars1.githubusercontent.com/u/3944799?v=4","gravatar_id":"","url":"https://api.github.com/users/incf-nidash","html_url":"https://github.com/incf-nidash","followers_url":"https://api.github.com/users/incf-nidash/followers","following_url":"https://api.github.com/users/incf-nidash/following{/other_user}","gists_url":"https://api.github.com/users/incf-nidash/gists{/gist_id}","starred_url":"https://api.github.com/users/incf-nidash/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/incf-nidash/subscriptions","organizations_url":"https://api.github.com/users/incf-nidash/orgs","repos_url":"https://api.github.com/users/incf-nidash/repos","events_url":"https://api.github.com/users/incf-nidash/events{/privacy}","received_events_url":"https://api.github.com/users/incf-nidash/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/incf-nidash/nidmresults-fslhtml","description":"Generate a set of FSL-like HTML pages from a NIDM-Results pack","fork":false,"url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml","forks_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/forks","keys_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/keys{/key_id}","collaborators_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/teams","hooks_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/hooks","issue_events_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/issues/events{/number}","events_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/events","assignees_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/assignees{/user}","branches_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/branches{/branch}","tags_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/tags","blobs_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/git/refs{/sha}","trees_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/git/trees{/sha}","statuses_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/statuses/{sha}","languages_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/languages","stargazers_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/stargazers","contributors_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/contributors","subscribers_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/subscribers","subscription_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/subscription","commits_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/commits{/sha}","git_commits_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/git/commits{/sha}","comments_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/comments{/number}","issue_comment_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/issues/comments{/number}","contents_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/contents/{+path}","compare_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/compare/{base}...{head}","merges_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/merges","archive_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/downloads","issues_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/issues{/number}","pulls_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls{/number}","milestones_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/milestones{/number}","notifications_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/labels{/name}","releases_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/releases{/id}","deployments_url":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/deployments","created_at":"2016-07-19T15:00:59Z","updated_at":"2016-12-08T11:54:15Z","pushed_at":"2018-01-24T14:13:45Z","git_url":"git://github.com/incf-nidash/nidmresults-fslhtml.git","ssh_url":"git@github.com:incf-nidash/nidmresults-fslhtml.git","clone_url":"https://github.com/incf-nidash/nidmresults-fslhtml.git","svn_url":"https://github.com/incf-nidash/nidmresults-fslhtml","homepage":null,"size":1982,"stargazers_count":1,"watchers_count":1,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"archived":false,"open_issues_count":12,"license":{"key":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0"},"forks":3,"open_issues":12,"watchers":1,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/17"},"html":{"href":"https://github.com/incf-nidash/nidmresults-fslhtml/pull/17"},"issue":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/issues/17"},"comments":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/issues/17/comments"},"review_comments":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/17/comments"},"review_comment":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/pulls/17/commits"},"statuses":{"href":"https://api.github.com/repos/incf-nidash/nidmresults-fslhtml/statuses/98e5e800d29ef3b1eec051d8674ba4dea8a5e183"}},"author_association":"COLLABORATOR"}}
{"actor":{"display_login":"TomMaullin"}}