{"action":"created","comment":{"url":"https://api.github.com/repos/autoreject/autoreject/pulls/comments/218020640","pull_request_review_id":155864250,"id":218020640,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIxODAyMDY0MA==","diff_hunk":"@@ -566,7 +566,8 @@ def get_reject_log(self, epochs, threshes=None, picks=None):\n             bad_sensor_counts, ch_type=ch_type, picks=this_picks)\n \n         reject_log = RejectLog(labels=labels, bad_epochs=bad_epochs,\n-                               ch_names=epochs.ch_names)\n+                               ch_names=epochs.ch_names,\n+                               bad_sensor_counts=bad_sensor_counts)","path":"autoreject/autoreject.py","position":6,"original_position":6,"commit_id":"17823a9dabbb40795e3a1a32aa0e6b66eff28e15","original_commit_id":"17823a9dabbb40795e3a1a32aa0e6b66eff28e15","user":{"login":"agramfort","id":161052,"node_id":"MDQ6VXNlcjE2MTA1Mg==","avatar_url":"https://avatars2.githubusercontent.com/u/161052?v=4","gravatar_id":"","url":"https://api.github.com/users/agramfort","html_url":"https://github.com/agramfort","followers_url":"https://api.github.com/users/agramfort/followers","following_url":"https://api.github.com/users/agramfort/following{/other_user}","gists_url":"https://api.github.com/users/agramfort/gists{/gist_id}","starred_url":"https://api.github.com/users/agramfort/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agramfort/subscriptions","organizations_url":"https://api.github.com/users/agramfort/orgs","repos_url":"https://api.github.com/users/agramfort/repos","events_url":"https://api.github.com/users/agramfort/events{/privacy}","received_events_url":"https://api.github.com/users/agramfort/received_events","type":"User","site_admin":false},"body":"this can be obtained from the labels no? If so I would not add this param to constructor but no objection to add some in the API of RejectLog to get this info","created_at":"2018-09-17T10:35:22Z","updated_at":"2018-09-17T10:35:22Z","html_url":"https://github.com/autoreject/autoreject/pull/117#discussion_r218020640","pull_request_url":"https://api.github.com/repos/autoreject/autoreject/pulls/117","author_association":"COLLABORATOR","_links":{"self":{"href":"https://api.github.com/repos/autoreject/autoreject/pulls/comments/218020640"},"html":{"href":"https://github.com/autoreject/autoreject/pull/117#discussion_r218020640"},"pull_request":{"href":"https://api.github.com/repos/autoreject/autoreject/pulls/117"}}},"pull_request":{"url":"https://api.github.com/repos/autoreject/autoreject/pulls/117","id":215933182,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE1OTMzMTgy","html_url":"https://github.com/autoreject/autoreject/pull/117","diff_url":"https://github.com/autoreject/autoreject/pull/117.diff","patch_url":"https://github.com/autoreject/autoreject/pull/117.patch","issue_url":"https://api.github.com/repos/autoreject/autoreject/issues/117","number":117,"state":"open","locked":false,"title":"Add bad sensors count per epochs to reject log","user":{"login":"fraimondo","id":4493699,"node_id":"MDQ6VXNlcjQ0OTM2OTk=","avatar_url":"https://avatars0.githubusercontent.com/u/4493699?v=4","gravatar_id":"","url":"https://api.github.com/users/fraimondo","html_url":"https://github.com/fraimondo","followers_url":"https://api.github.com/users/fraimondo/followers","following_url":"https://api.github.com/users/fraimondo/following{/other_user}","gists_url":"https://api.github.com/users/fraimondo/gists{/gist_id}","starred_url":"https://api.github.com/users/fraimondo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fraimondo/subscriptions","organizations_url":"https://api.github.com/users/fraimondo/orgs","repos_url":"https://api.github.com/users/fraimondo/repos","events_url":"https://api.github.com/users/fraimondo/events{/privacy}","received_events_url":"https://api.github.com/users/fraimondo/received_events","type":"User","site_admin":false},"body":"Hi Team!\r\n\r\nI added an option to keep the number of bad channels detected for each epoch in the log. Basically I use this to check the overall quality of the data after cleaning and give the user an idea on how many electrodes were rejected for each epoch, with the possibility to visualise or do stats.\r\n\r\nI hope you find it useful.","created_at":"2018-09-17T10:21:05Z","updated_at":"2018-09-17T10:35:22Z","closed_at":null,"merged_at":null,"merge_commit_sha":"32e857f3297fe1948b7d71d9ded66c4238ed3e02","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/autoreject/autoreject/pulls/117/commits","review_comments_url":"https://api.github.com/repos/autoreject/autoreject/pulls/117/comments","review_comment_url":"https://api.github.com/repos/autoreject/autoreject/pulls/comments{/number}","comments_url":"https://api.github.com/repos/autoreject/autoreject/issues/117/comments","statuses_url":"https://api.github.com/repos/autoreject/autoreject/statuses/17823a9dabbb40795e3a1a32aa0e6b66eff28e15","head":{"label":"fraimondo:bad_sensors_counts","ref":"bad_sensors_counts","sha":"17823a9dabbb40795e3a1a32aa0e6b66eff28e15","user":{"login":"fraimondo","id":4493699,"node_id":"MDQ6VXNlcjQ0OTM2OTk=","avatar_url":"https://avatars0.githubusercontent.com/u/4493699?v=4","gravatar_id":"","url":"https://api.github.com/users/fraimondo","html_url":"https://github.com/fraimondo","followers_url":"https://api.github.com/users/fraimondo/followers","following_url":"https://api.github.com/users/fraimondo/following{/other_user}","gists_url":"https://api.github.com/users/fraimondo/gists{/gist_id}","starred_url":"https://api.github.com/users/fraimondo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fraimondo/subscriptions","organizations_url":"https://api.github.com/users/fraimondo/orgs","repos_url":"https://api.github.com/users/fraimondo/repos","events_url":"https://api.github.com/users/fraimondo/events{/privacy}","received_events_url":"https://api.github.com/users/fraimondo/received_events","type":"User","site_admin":false},"repo":{"id":78221008,"node_id":"MDEwOlJlcG9zaXRvcnk3ODIyMTAwOA==","name":"autoreject","full_name":"fraimondo/autoreject","private":false,"owner":{"login":"fraimondo","id":4493699,"node_id":"MDQ6VXNlcjQ0OTM2OTk=","avatar_url":"https://avatars0.githubusercontent.com/u/4493699?v=4","gravatar_id":"","url":"https://api.github.com/users/fraimondo","html_url":"https://github.com/fraimondo","followers_url":"https://api.github.com/users/fraimondo/followers","following_url":"https://api.github.com/users/fraimondo/following{/other_user}","gists_url":"https://api.github.com/users/fraimondo/gists{/gist_id}","starred_url":"https://api.github.com/users/fraimondo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fraimondo/subscriptions","organizations_url":"https://api.github.com/users/fraimondo/orgs","repos_url":"https://api.github.com/users/fraimondo/repos","events_url":"https://api.github.com/users/fraimondo/events{/privacy}","received_events_url":"https://api.github.com/users/fraimondo/received_events","type":"User","site_admin":false},"html_url":"https://github.com/fraimondo/autoreject","description":"Automated rejection and repair of bad trials/sensors in M/EEG","fork":true,"url":"https://api.github.com/repos/fraimondo/autoreject","forks_url":"https://api.github.com/repos/fraimondo/autoreject/forks","keys_url":"https://api.github.com/repos/fraimondo/autoreject/keys{/key_id}","collaborators_url":"https://api.github.com/repos/fraimondo/autoreject/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/fraimondo/autoreject/teams","hooks_url":"https://api.github.com/repos/fraimondo/autoreject/hooks","issue_events_url":"https://api.github.com/repos/fraimondo/autoreject/issues/events{/number}","events_url":"https://api.github.com/repos/fraimondo/autoreject/events","assignees_url":"https://api.github.com/repos/fraimondo/autoreject/assignees{/user}","branches_url":"https://api.github.com/repos/fraimondo/autoreject/branches{/branch}","tags_url":"https://api.github.com/repos/fraimondo/autoreject/tags","blobs_url":"https://api.github.com/repos/fraimondo/autoreject/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/fraimondo/autoreject/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/fraimondo/autoreject/git/refs{/sha}","trees_url":"https://api.github.com/repos/fraimondo/autoreject/git/trees{/sha}","statuses_url":"https://api.github.com/repos/fraimondo/autoreject/statuses/{sha}","languages_url":"https://api.github.com/repos/fraimondo/autoreject/languages","stargazers_url":"https://api.github.com/repos/fraimondo/autoreject/stargazers","contributors_url":"https://api.github.com/repos/fraimondo/autoreject/contributors","subscribers_url":"https://api.github.com/repos/fraimondo/autoreject/subscribers","subscription_url":"https://api.github.com/repos/fraimondo/autoreject/subscription","commits_url":"https://api.github.com/repos/fraimondo/autoreject/commits{/sha}","git_commits_url":"https://api.github.com/repos/fraimondo/autoreject/git/commits{/sha}","comments_url":"https://api.github.com/repos/fraimondo/autoreject/comments{/number}","issue_comment_url":"https://api.github.com/repos/fraimondo/autoreject/issues/comments{/number}","contents_url":"https://api.github.com/repos/fraimondo/autoreject/contents/{+path}","compare_url":"https://api.github.com/repos/fraimondo/autoreject/compare/{base}...{head}","merges_url":"https://api.github.com/repos/fraimondo/autoreject/merges","archive_url":"https://api.github.com/repos/fraimondo/autoreject/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/fraimondo/autoreject/downloads","issues_url":"https://api.github.com/repos/fraimondo/autoreject/issues{/number}","pulls_url":"https://api.github.com/repos/fraimondo/autoreject/pulls{/number}","milestones_url":"https://api.github.com/repos/fraimondo/autoreject/milestones{/number}","notifications_url":"https://api.github.com/repos/fraimondo/autoreject/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/fraimondo/autoreject/labels{/name}","releases_url":"https://api.github.com/repos/fraimondo/autoreject/releases{/id}","deployments_url":"https://api.github.com/repos/fraimondo/autoreject/deployments","created_at":"2017-01-06T16:31:01Z","updated_at":"2017-01-06T16:31:03Z","pushed_at":"2018-09-17T10:15:45Z","git_url":"git://github.com/fraimondo/autoreject.git","ssh_url":"git@github.com:fraimondo/autoreject.git","clone_url":"https://github.com/fraimondo/autoreject.git","svn_url":"https://github.com/fraimondo/autoreject","homepage":"http://autoreject.github.io/","size":373,"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":{"key":"bsd-3-clause","name":"BSD 3-Clause \"New\" or \"Revised\" License","spdx_id":"BSD-3-Clause","url":"https://api.github.com/licenses/bsd-3-clause","node_id":"MDc6TGljZW5zZTU="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"autoreject:master","ref":"master","sha":"71133bb04a3a8f87b292dba095d4911e7f1c44eb","user":{"login":"autoreject","id":19533006,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE5NTMzMDA2","avatar_url":"https://avatars0.githubusercontent.com/u/19533006?v=4","gravatar_id":"","url":"https://api.github.com/users/autoreject","html_url":"https://github.com/autoreject","followers_url":"https://api.github.com/users/autoreject/followers","following_url":"https://api.github.com/users/autoreject/following{/other_user}","gists_url":"https://api.github.com/users/autoreject/gists{/gist_id}","starred_url":"https://api.github.com/users/autoreject/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/autoreject/subscriptions","organizations_url":"https://api.github.com/users/autoreject/orgs","repos_url":"https://api.github.com/users/autoreject/repos","events_url":"https://api.github.com/users/autoreject/events{/privacy}","received_events_url":"https://api.github.com/users/autoreject/received_events","type":"Organization","site_admin":false},"repo":{"id":59485140,"node_id":"MDEwOlJlcG9zaXRvcnk1OTQ4NTE0MA==","name":"autoreject","full_name":"autoreject/autoreject","private":false,"owner":{"login":"autoreject","id":19533006,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE5NTMzMDA2","avatar_url":"https://avatars0.githubusercontent.com/u/19533006?v=4","gravatar_id":"","url":"https://api.github.com/users/autoreject","html_url":"https://github.com/autoreject","followers_url":"https://api.github.com/users/autoreject/followers","following_url":"https://api.github.com/users/autoreject/following{/other_user}","gists_url":"https://api.github.com/users/autoreject/gists{/gist_id}","starred_url":"https://api.github.com/users/autoreject/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/autoreject/subscriptions","organizations_url":"https://api.github.com/users/autoreject/orgs","repos_url":"https://api.github.com/users/autoreject/repos","events_url":"https://api.github.com/users/autoreject/events{/privacy}","received_events_url":"https://api.github.com/users/autoreject/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/autoreject/autoreject","description":"Automated rejection and repair of bad trials/sensors in M/EEG","fork":false,"url":"https://api.github.com/repos/autoreject/autoreject","forks_url":"https://api.github.com/repos/autoreject/autoreject/forks","keys_url":"https://api.github.com/repos/autoreject/autoreject/keys{/key_id}","collaborators_url":"https://api.github.com/repos/autoreject/autoreject/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/autoreject/autoreject/teams","hooks_url":"https://api.github.com/repos/autoreject/autoreject/hooks","issue_events_url":"https://api.github.com/repos/autoreject/autoreject/issues/events{/number}","events_url":"https://api.github.com/repos/autoreject/autoreject/events","assignees_url":"https://api.github.com/repos/autoreject/autoreject/assignees{/user}","branches_url":"https://api.github.com/repos/autoreject/autoreject/branches{/branch}","tags_url":"https://api.github.com/repos/autoreject/autoreject/tags","blobs_url":"https://api.github.com/repos/autoreject/autoreject/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/autoreject/autoreject/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/autoreject/autoreject/git/refs{/sha}","trees_url":"https://api.github.com/repos/autoreject/autoreject/git/trees{/sha}","statuses_url":"https://api.github.com/repos/autoreject/autoreject/statuses/{sha}","languages_url":"https://api.github.com/repos/autoreject/autoreject/languages","stargazers_url":"https://api.github.com/repos/autoreject/autoreject/stargazers","contributors_url":"https://api.github.com/repos/autoreject/autoreject/contributors","subscribers_url":"https://api.github.com/repos/autoreject/autoreject/subscribers","subscription_url":"https://api.github.com/repos/autoreject/autoreject/subscription","commits_url":"https://api.github.com/repos/autoreject/autoreject/commits{/sha}","git_commits_url":"https://api.github.com/repos/autoreject/autoreject/git/commits{/sha}","comments_url":"https://api.github.com/repos/autoreject/autoreject/comments{/number}","issue_comment_url":"https://api.github.com/repos/autoreject/autoreject/issues/comments{/number}","contents_url":"https://api.github.com/repos/autoreject/autoreject/contents/{+path}","compare_url":"https://api.github.com/repos/autoreject/autoreject/compare/{base}...{head}","merges_url":"https://api.github.com/repos/autoreject/autoreject/merges","archive_url":"https://api.github.com/repos/autoreject/autoreject/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/autoreject/autoreject/downloads","issues_url":"https://api.github.com/repos/autoreject/autoreject/issues{/number}","pulls_url":"https://api.github.com/repos/autoreject/autoreject/pulls{/number}","milestones_url":"https://api.github.com/repos/autoreject/autoreject/milestones{/number}","notifications_url":"https://api.github.com/repos/autoreject/autoreject/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/autoreject/autoreject/labels{/name}","releases_url":"https://api.github.com/repos/autoreject/autoreject/releases{/id}","deployments_url":"https://api.github.com/repos/autoreject/autoreject/deployments","created_at":"2016-05-23T13:28:23Z","updated_at":"2018-09-17T07:36:59Z","pushed_at":"2018-09-17T10:21:06Z","git_url":"git://github.com/autoreject/autoreject.git","ssh_url":"git@github.com:autoreject/autoreject.git","clone_url":"https://github.com/autoreject/autoreject.git","svn_url":"https://github.com/autoreject/autoreject","homepage":"http://autoreject.github.io/","size":396,"stargazers_count":40,"watchers_count":40,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":26,"mirror_url":null,"archived":false,"open_issues_count":19,"license":{"key":"bsd-3-clause","name":"BSD 3-Clause \"New\" or \"Revised\" License","spdx_id":"BSD-3-Clause","url":"https://api.github.com/licenses/bsd-3-clause","node_id":"MDc6TGljZW5zZTU="},"forks":26,"open_issues":19,"watchers":40,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/autoreject/autoreject/pulls/117"},"html":{"href":"https://github.com/autoreject/autoreject/pull/117"},"issue":{"href":"https://api.github.com/repos/autoreject/autoreject/issues/117"},"comments":{"href":"https://api.github.com/repos/autoreject/autoreject/issues/117/comments"},"review_comments":{"href":"https://api.github.com/repos/autoreject/autoreject/pulls/117/comments"},"review_comment":{"href":"https://api.github.com/repos/autoreject/autoreject/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/autoreject/autoreject/pulls/117/commits"},"statuses":{"href":"https://api.github.com/repos/autoreject/autoreject/statuses/17823a9dabbb40795e3a1a32aa0e6b66eff28e15"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"agramfort"}}