payload: {"action":"created","comment":{"url":"https://api.github.com/repos/nf-core/rnaseq/pulls/comments/334645287","pull_request_review_id":301514197,"id":334645287,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDMzNDY0NTI4Nw==","diff_hunk":"@@ -67,25 +67,24 @@ write.csv(MDSxy, 'edgeR_MDS_Aplot_coordinates_mqc.csv', quote=FALSE, append=TRUE\n # Get the log counts per million values\n logcpm <- cpm(dataNorm, prior.count=2, log=TRUE)\n \n-# Calculate the euclidean distances between samples\n-dists = dist(t(logcpm))\n-\n+# Calculate the Pearsons correlation between samples\n # Plot a heatmap of correlations\n-pdf('log2CPM_sample_distances_heatmap.pdf')\n-hmap <- heatmap.2(as.matrix(dists),\n-  main=\"Sample Correlations\", key.title=\"Distance\", trace=\"none\",\n+pdf('log2CPM_sample_correlation_heatmap.pdf')\n+hmap <- heatmap.2(as.matrix(cor(logcpm, method=\"pearson\")),\n+  key.title=\"Pearsons Correlation\", trace=\"none\",\n   dendrogram=\"row\", margin=c(9, 9)\n )\n dev.off()\n \n+# Write correlation values to file\n+write.csv(hmap$carpet, 'log2CPM_sample_correlation_mqc.csv', quote=FALSE, append=TRUE)\n+\n # Plot the heatmap dendrogram\n pdf('log2CPM_sample_distances_dendrogram.pdf')\n-plot(hmap$rowDendrogram, main=\"Sample Dendrogram\")\n+hmap <- heatmap.2(as.matrix(dist(t(logcpm))))\n+plot(hmap$rowDendrogram, main=\"Sample Euclidean Distance Clustering\")","path":"bin/edgeR_heatmap_MDS.r","position":26,"original_position":26,"commit_id":"8f0b25775fb61e3ae4222419776e64482b191ce7","original_commit_id":"8f0b25775fb61e3ae4222419776e64482b191ce7","user":{"login":"olgabot","id":806256,"node_id":"MDQ6VXNlcjgwNjI1Ng==","avatar_url":"https://avatars2.githubusercontent.com/u/806256?v=4","gravatar_id":"","url":"https://api.github.com/users/olgabot","html_url":"https://github.com/olgabot","followers_url":"https://api.github.com/users/olgabot/followers","following_url":"https://api.github.com/users/olgabot/following{/other_user}","gists_url":"https://api.github.com/users/olgabot/gists{/gist_id}","starred_url":"https://api.github.com/users/olgabot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/olgabot/subscriptions","organizations_url":"https://api.github.com/users/olgabot/orgs","repos_url":"https://api.github.com/users/olgabot/repos","events_url":"https://api.github.com/users/olgabot/events{/privacy}","received_events_url":"https://api.github.com/users/olgabot/received_events","type":"User","site_admin":false},"body":"```suggestion\r\nplot(hmap$rowDendrogram, main=\"Sample Pearson's Correlation Clustering\")\r\n```\r\n\r\nI'm confused... I thought this was changed to Pearson's rather than Euclidean? Am I understanding correctly?","created_at":"2019-10-14T20:15:39Z","updated_at":"2019-10-14T20:34:40Z","html_url":"https://github.com/nf-core/rnaseq/pull/300#discussion_r334645287","pull_request_url":"https://api.github.com/repos/nf-core/rnaseq/pulls/300","author_association":"NONE","_links":{"self":{"href":"https://api.github.com/repos/nf-core/rnaseq/pulls/comments/334645287"},"html":{"href":"https://github.com/nf-core/rnaseq/pull/300#discussion_r334645287"},"pull_request":{"href":"https://api.github.com/repos/nf-core/rnaseq/pulls/300"}}},"pull_request":{"url":"https://api.github.com/repos/nf-core/rnaseq/pulls/300","id":326241511,"node_id":"MDExOlB1bGxSZXF1ZXN0MzI2MjQxNTEx","html_url":"https://github.com/nf-core/rnaseq/pull/300","diff_url":"https://github.com/nf-core/rnaseq/pull/300.diff","patch_url":"https://github.com/nf-core/rnaseq/pull/300.patch","issue_url":"https://api.github.com/repos/nf-core/rnaseq/issues/300","number":300,"state":"open","locked":false,"title":"PR for Release 1.4","user":{"login":"apeltzer","id":2359510,"node_id":"MDQ6VXNlcjIzNTk1MTA=","avatar_url":"https://avatars3.githubusercontent.com/u/2359510?v=4","gravatar_id":"","url":"https://api.github.com/users/apeltzer","html_url":"https://github.com/apeltzer","followers_url":"https://api.github.com/users/apeltzer/followers","following_url":"https://api.github.com/users/apeltzer/following{/other_user}","gists_url":"https://api.github.com/users/apeltzer/gists{/gist_id}","starred_url":"https://api.github.com/users/apeltzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apeltzer/subscriptions","organizations_url":"https://api.github.com/users/apeltzer/orgs","repos_url":"https://api.github.com/users/apeltzer/repos","events_url":"https://api.github.com/users/apeltzer/events{/privacy}","received_events_url":"https://api.github.com/users/apeltzer/received_events","type":"User","site_admin":false},"body":"This is the `dev` to `master` PR for making a release 1.4 for nf-core/rnaseq. Lots of changes, so please have a look and review 👍 \r\n\r\n## PR checklist\r\n - [x] This comment contains a description of changes (with reason)\r\n - [x] If you've fixed a bug or added code that should be tested, add tests!\r\n - [x] If necessary, also make a PR on the [nf-core/rnaseq branch on the nf-core/test-datasets repo]( https://github.com/nf-core/test-datasets/pull/new/nf-core/rnaseq)\r\n - [x] Ensure the test suite passes (`nextflow run . -profile test,docker`).\r\n - [x] Make sure your code lints (`nf-core lint .`).\r\n - [x] Documentation in `docs` is updated\r\n - [x] `CHANGELOG.md` is updated\r\n - [x] `README.md` is updated\r\n","created_at":"2019-10-09T13:00:10Z","updated_at":"2019-10-14T20:34:39Z","closed_at":null,"merged_at":null,"merge_commit_sha":"96f2baf66df62dd9b99f9a5142e957922763882d","assignee":null,"assignees":[],"requested_reviewers":[{"login":"ewels","id":465550,"node_id":"MDQ6VXNlcjQ2NTU1MA==","avatar_url":"https://avatars3.githubusercontent.com/u/465550?v=4","gravatar_id":"","url":"https://api.github.com/users/ewels","html_url":"https://github.com/ewels","followers_url":"https://api.github.com/users/ewels/followers","following_url":"https://api.github.com/users/ewels/following{/other_user}","gists_url":"https://api.github.com/users/ewels/gists{/gist_id}","starred_url":"https://api.github.com/users/ewels/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ewels/subscriptions","organizations_url":"https://api.github.com/users/ewels/orgs","repos_url":"https://api.github.com/users/ewels/repos","events_url":"https://api.github.com/users/ewels/events{/privacy}","received_events_url":"https://api.github.com/users/ewels/received_events","type":"User","site_admin":false}],"requested_teams":[],"labels":[{"id":1605317728,"node_id":"MDU6TGFiZWwxNjA1MzE3NzI4","url":"https://api.github.com/repos/nf-core/rnaseq/labels/Release","name":"Release","color":"3d86ba","default":false},{"id":1572702361,"node_id":"MDU6TGFiZWwxNTcyNzAyMzYx","url":"https://api.github.com/repos/nf-core/rnaseq/labels/Review%20&%20Merge","name":"Review & Merge","color":"7d8e0e","default":false}],"milestone":null,"commits_url":"https://api.github.com/repos/nf-core/rnaseq/pulls/300/commits","review_comments_url":"https://api.github.com/repos/nf-core/rnaseq/pulls/300/comments","review_comment_url":"https://api.github.com/repos/nf-core/rnaseq/pulls/comments{/number}","comments_url":"https://api.github.com/repos/nf-core/rnaseq/issues/300/comments","statuses_url":"https://api.github.com/repos/nf-core/rnaseq/statuses/8f0b25775fb61e3ae4222419776e64482b191ce7","head":{"label":"nf-core:dev","ref":"dev","sha":"8f0b25775fb61e3ae4222419776e64482b191ce7","user":{"login":"nf-core","id":35520196,"node_id":"MDEyOk9yZ2FuaXphdGlvbjM1NTIwMTk2","avatar_url":"https://avatars0.githubusercontent.com/u/35520196?v=4","gravatar_id":"","url":"https://api.github.com/users/nf-core","html_url":"https://github.com/nf-core","followers_url":"https://api.github.com/users/nf-core/followers","following_url":"https://api.github.com/users/nf-core/following{/other_user}","gists_url":"https://api.github.com/users/nf-core/gists{/gist_id}","starred_url":"https://api.github.com/users/nf-core/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nf-core/subscriptions","organizations_url":"https://api.github.com/users/nf-core/orgs","repos_url":"https://api.github.com/users/nf-core/repos","events_url":"https://api.github.com/users/nf-core/events{/privacy}","received_events_url":"https://api.github.com/users/nf-core/received_events","type":"Organization","site_admin":false},"repo":{"id":127293091,"node_id":"MDEwOlJlcG9zaXRvcnkxMjcyOTMwOTE=","name":"rnaseq","full_name":"nf-core/rnaseq","private":false,"owner":{"login":"nf-core","id":35520196,"node_id":"MDEyOk9yZ2FuaXphdGlvbjM1NTIwMTk2","avatar_url":"https://avatars0.githubusercontent.com/u/35520196?v=4","gravatar_id":"","url":"https://api.github.com/users/nf-core","html_url":"https://github.com/nf-core","followers_url":"https://api.github.com/users/nf-core/followers","following_url":"https://api.github.com/users/nf-core/following{/other_user}","gists_url":"https://api.github.com/users/nf-core/gists{/gist_id}","starred_url":"https://api.github.com/users/nf-core/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nf-core/subscriptions","organizations_url":"https://api.github.com/users/nf-core/orgs","repos_url":"https://api.github.com/users/nf-core/repos","events_url":"https://api.github.com/users/nf-core/events{/privacy}","received_events_url":"https://api.github.com/users/nf-core/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/nf-core/rnaseq","description":"RNA sequencing analysis pipeline using STAR or HISAT2, with gene counts and quality control","fork":true,"url":"https://api.github.com/repos/nf-core/rnaseq","forks_url":"https://api.github.com/repos/nf-core/rnaseq/forks","keys_url":"https://api.github.com/repos/nf-core/rnaseq/keys{/key_id}","collaborators_url":"https://api.github.com/repos/nf-core/rnaseq/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/nf-core/rnaseq/teams","hooks_url":"https://api.github.com/repos/nf-core/rnaseq/hooks","issue_events_url":"https://api.github.com/repos/nf-core/rnaseq/issues/events{/number}","events_url":"https://api.github.com/repos/nf-core/rnaseq/events","assignees_url":"https://api.github.com/repos/nf-core/rnaseq/assignees{/user}","branches_url":"https://api.github.com/repos/nf-core/rnaseq/branches{/branch}","tags_url":"https://api.github.com/repos/nf-core/rnaseq/tags","blobs_url":"https://api.github.com/repos/nf-core/rnaseq/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/nf-core/rnaseq/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/nf-core/rnaseq/git/refs{/sha}","trees_url":"https://api.github.com/repos/nf-core/rnaseq/git/trees{/sha}","statuses_url":"https://api.github.com/repos/nf-core/rnaseq/statuses/{sha}","languages_url":"https://api.github.com/repos/nf-core/rnaseq/languages","stargazers_url":"https://api.github.com/repos/nf-core/rnaseq/stargazers","contributors_url":"https://api.github.com/repos/nf-core/rnaseq/contributors","subscribers_url":"https://api.github.com/repos/nf-core/rnaseq/subscribers","subscription_url":"https://api.github.com/repos/nf-core/rnaseq/subscription","commits_url":"https://api.github.com/repos/nf-core/rnaseq/commits{/sha}","git_commits_url":"https://api.github.com/repos/nf-core/rnaseq/git/commits{/sha}","comments_url":"https://api.github.com/repos/nf-core/rnaseq/comments{/number}","issue_comment_url":"https://api.github.com/repos/nf-core/rnaseq/issues/comments{/number}","contents_url":"https://api.github.com/repos/nf-core/rnaseq/contents/{+path}","compare_url":"https://api.github.com/repos/nf-core/rnaseq/compare/{base}...{head}","merges_url":"https://api.github.com/repos/nf-core/rnaseq/merges","archive_url":"https://api.github.com/repos/nf-core/rnaseq/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/nf-core/rnaseq/downloads","issues_url":"https://api.github.com/repos/nf-core/rnaseq/issues{/number}","pulls_url":"https://api.github.com/repos/nf-core/rnaseq/pulls{/number}","milestones_url":"https://api.github.com/repos/nf-core/rnaseq/milestones{/number}","notifications_url":"https://api.github.com/repos/nf-core/rnaseq/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/nf-core/rnaseq/labels{/name}","releases_url":"https://api.github.com/repos/nf-core/rnaseq/releases{/id}","deployments_url":"https://api.github.com/repos/nf-core/rnaseq/deployments","created_at":"2018-03-29T13:10:58Z","updated_at":"2019-10-09T17:09:33Z","pushed_at":"2019-10-14T16:20:11Z","git_url":"git://github.com/nf-core/rnaseq.git","ssh_url":"git@github.com:nf-core/rnaseq.git","clone_url":"https://github.com/nf-core/rnaseq.git","svn_url":"https://github.com/nf-core/rnaseq","homepage":"http://nf-co.re","size":51751,"stargazers_count":133,"watchers_count":133,"language":"Nextflow","has_issues":true,"has_projects":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":93,"mirror_url":null,"archived":false,"disabled":false,"open_issues_count":33,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit","node_id":"MDc6TGljZW5zZTEz"},"forks":93,"open_issues":33,"watchers":133,"default_branch":"master"}},"base":{"label":"nf-core:master","ref":"master","sha":"37f260d360e59df7166cfd60e2b3c9a3999adf75","user":{"login":"nf-core","id":35520196,"node_id":"MDEyOk9yZ2FuaXphdGlvbjM1NTIwMTk2","avatar_url":"https://avatars0.githubusercontent.com/u/35520196?v=4","gravatar_id":"","url":"https://api.github.com/users/nf-core","html_url":"https://github.com/nf-core","followers_url":"https://api.github.com/users/nf-core/followers","following_url":"https://api.github.com/users/nf-core/following{/other_user}","gists_url":"https://api.github.com/users/nf-core/gists{/gist_id}","starred_url":"https://api.github.com/users/nf-core/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nf-core/subscriptions","organizations_url":"https://api.github.com/users/nf-core/orgs","repos_url":"https://api.github.com/users/nf-core/repos","events_url":"https://api.github.com/users/nf-core/events{/privacy}","received_events_url":"https://api.github.com/users/nf-core/received_events","type":"Organization","site_admin":false},"repo":{"id":127293091,"node_id":"MDEwOlJlcG9zaXRvcnkxMjcyOTMwOTE=","name":"rnaseq","full_name":"nf-core/rnaseq","private":false,"owner":{"login":"nf-core","id":35520196,"node_id":"MDEyOk9yZ2FuaXphdGlvbjM1NTIwMTk2","avatar_url":"https://avatars0.githubusercontent.com/u/35520196?v=4","gravatar_id":"","url":"https://api.github.com/users/nf-core","html_url":"https://github.com/nf-core","followers_url":"https://api.github.com/users/nf-core/followers","following_url":"https://api.github.com/users/nf-core/following{/other_user}","gists_url":"https://api.github.com/users/nf-core/gists{/gist_id}","starred_url":"https://api.github.com/users/nf-core/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nf-core/subscriptions","organizations_url":"https://api.github.com/users/nf-core/orgs","repos_url":"https://api.github.com/users/nf-core/repos","events_url":"https://api.github.com/users/nf-core/events{/privacy}","received_events_url":"https://api.github.com/users/nf-core/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/nf-core/rnaseq","description":"RNA sequencing analysis pipeline using STAR or HISAT2, with gene counts and quality control","fork":true,"url":"https://api.github.com/repos/nf-core/rnaseq","forks_url":"https://api.github.com/repos/nf-core/rnaseq/forks","keys_url":"https://api.github.com/repos/nf-core/rnaseq/keys{/key_id}","collaborators_url":"https://api.github.com/repos/nf-core/rnaseq/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/nf-core/rnaseq/teams","hooks_url":"https://api.github.com/repos/nf-core/rnaseq/hooks","issue_events_url":"https://api.github.com/repos/nf-core/rnaseq/issues/events{/number}","events_url":"https://api.github.com/repos/nf-core/rnaseq/events","assignees_url":"https://api.github.com/repos/nf-core/rnaseq/assignees{/user}","branches_url":"https://api.github.com/repos/nf-core/rnaseq/branches{/branch}","tags_url":"https://api.github.com/repos/nf-core/rnaseq/tags","blobs_url":"https://api.github.com/repos/nf-core/rnaseq/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/nf-core/rnaseq/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/nf-core/rnaseq/git/refs{/sha}","trees_url":"https://api.github.com/repos/nf-core/rnaseq/git/trees{/sha}","statuses_url":"https://api.github.com/repos/nf-core/rnaseq/statuses/{sha}","languages_url":"https://api.github.com/repos/nf-core/rnaseq/languages","stargazers_url":"https://api.github.com/repos/nf-core/rnaseq/stargazers","contributors_url":"https://api.github.com/repos/nf-core/rnaseq/contributors","subscribers_url":"https://api.github.com/repos/nf-core/rnaseq/subscribers","subscription_url":"https://api.github.com/repos/nf-core/rnaseq/subscription","commits_url":"https://api.github.com/repos/nf-core/rnaseq/commits{/sha}","git_commits_url":"https://api.github.com/repos/nf-core/rnaseq/git/commits{/sha}","comments_url":"https://api.github.com/repos/nf-core/rnaseq/comments{/number}","issue_comment_url":"https://api.github.com/repos/nf-core/rnaseq/issues/comments{/number}","contents_url":"https://api.github.com/repos/nf-core/rnaseq/contents/{+path}","compare_url":"https://api.github.com/repos/nf-core/rnaseq/compare/{base}...{head}","merges_url":"https://api.github.com/repos/nf-core/rnaseq/merges","archive_url":"https://api.github.com/repos/nf-core/rnaseq/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/nf-core/rnaseq/downloads","issues_url":"https://api.github.com/repos/nf-core/rnaseq/issues{/number}","pulls_url":"https://api.github.com/repos/nf-core/rnaseq/pulls{/number}","milestones_url":"https://api.github.com/repos/nf-core/rnaseq/milestones{/number}","notifications_url":"https://api.github.com/repos/nf-core/rnaseq/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/nf-core/rnaseq/labels{/name}","releases_url":"https://api.github.com/repos/nf-core/rnaseq/releases{/id}","deployments_url":"https://api.github.com/repos/nf-core/rnaseq/deployments","created_at":"2018-03-29T13:10:58Z","updated_at":"2019-10-09T17:09:33Z","pushed_at":"2019-10-14T16:20:11Z","git_url":"git://github.com/nf-core/rnaseq.git","ssh_url":"git@github.com:nf-core/rnaseq.git","clone_url":"https://github.com/nf-core/rnaseq.git","svn_url":"https://github.com/nf-core/rnaseq","homepage":"http://nf-co.re","size":51751,"stargazers_count":133,"watchers_count":133,"language":"Nextflow","has_issues":true,"has_projects":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":93,"mirror_url":null,"archived":false,"disabled":false,"open_issues_count":33,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit","node_id":"MDc6TGljZW5zZTEz"},"forks":93,"open_issues":33,"watchers":133,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/nf-core/rnaseq/pulls/300"},"html":{"href":"https://github.com/nf-core/rnaseq/pull/300"},"issue":{"href":"https://api.github.com/repos/nf-core/rnaseq/issues/300"},"comments":{"href":"https://api.github.com/repos/nf-core/rnaseq/issues/300/comments"},"review_comments":{"href":"https://api.github.com/repos/nf-core/rnaseq/pulls/300/comments"},"review_comment":{"href":"https://api.github.com/repos/nf-core/rnaseq/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/nf-core/rnaseq/pulls/300/commits"},"statuses":{"href":"https://api.github.com/repos/nf-core/rnaseq/statuses/8f0b25775fb61e3ae4222419776e64482b191ce7"}},"author_association":"MEMBER"}}
other: {"actor":{"display_login":"olgabot"}}