{"action":"created","comment":{"url":"https://api.github.com/repos/OpenMined/PySyft/pulls/comments/241971303","pull_request_review_id":185378069,"id":241971303,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDI0MTk3MTMwMw==","diff_hunk":"@@ -607,10 +607,11 @@ def get_worker(self, id_or_worker):\n             if id_or_worker in self._known_workers:\n                 return self._known_workers[id_or_worker]\n             else:\n-                logging.warning(\n-                    \"Worker\", self.id, \"couldnt recognize worker\", id_or_worker\n+                raise TypeError(","path":"syft/core/workers/base.py","position":6,"original_position":6,"commit_id":"d93477dcd66850743d7689fc431cd04238d22f3d","original_commit_id":"d93477dcd66850743d7689fc431cd04238d22f3d","user":{"login":"mari-linhares","id":8157164,"node_id":"MDQ6VXNlcjgxNTcxNjQ=","avatar_url":"https://avatars3.githubusercontent.com/u/8157164?v=4","gravatar_id":"","url":"https://api.github.com/users/mari-linhares","html_url":"https://github.com/mari-linhares","followers_url":"https://api.github.com/users/mari-linhares/followers","following_url":"https://api.github.com/users/mari-linhares/following{/other_user}","gists_url":"https://api.github.com/users/mari-linhares/gists{/gist_id}","starred_url":"https://api.github.com/users/mari-linhares/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mari-linhares/subscriptions","organizations_url":"https://api.github.com/users/mari-linhares/orgs","repos_url":"https://api.github.com/users/mari-linhares/repos","events_url":"https://api.github.com/users/mari-linhares/events{/privacy}","received_events_url":"https://api.github.com/users/mari-linhares/received_events","type":"User","site_admin":false},"body":"To be honest, I'm not particularly convinced of this as well. In some places a \"`Exception` is raised, but I don't think this is the best approach. Maybe raise a `RuntimeWarning`  (https://docs.python.org/2/library/exceptions.html#exceptions.RuntimeWarning) ?","created_at":"2018-12-16T08:11:22Z","updated_at":"2018-12-16T08:11:22Z","html_url":"https://github.com/OpenMined/PySyft/pull/1779#discussion_r241971303","pull_request_url":"https://api.github.com/repos/OpenMined/PySyft/pulls/1779","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/OpenMined/PySyft/pulls/comments/241971303"},"html":{"href":"https://github.com/OpenMined/PySyft/pull/1779#discussion_r241971303"},"pull_request":{"href":"https://api.github.com/repos/OpenMined/PySyft/pulls/1779"}},"in_reply_to_id":241968184},"pull_request":{"url":"https://api.github.com/repos/OpenMined/PySyft/pulls/1779","id":238939990,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM4OTM5OTkw","html_url":"https://github.com/OpenMined/PySyft/pull/1779","diff_url":"https://github.com/OpenMined/PySyft/pull/1779.diff","patch_url":"https://github.com/OpenMined/PySyft/pull/1779.patch","issue_url":"https://api.github.com/repos/OpenMined/PySyft/issues/1779","number":1779,"state":"open","locked":false,"title":"Fix #1760","user":{"login":"mari-linhares","id":8157164,"node_id":"MDQ6VXNlcjgxNTcxNjQ=","avatar_url":"https://avatars3.githubusercontent.com/u/8157164?v=4","gravatar_id":"","url":"https://api.github.com/users/mari-linhares","html_url":"https://github.com/mari-linhares","followers_url":"https://api.github.com/users/mari-linhares/followers","following_url":"https://api.github.com/users/mari-linhares/following{/other_user}","gists_url":"https://api.github.com/users/mari-linhares/gists{/gist_id}","starred_url":"https://api.github.com/users/mari-linhares/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mari-linhares/subscriptions","organizations_url":"https://api.github.com/users/mari-linhares/orgs","repos_url":"https://api.github.com/users/mari-linhares/repos","events_url":"https://api.github.com/users/mari-linhares/events{/privacy}","received_events_url":"https://api.github.com/users/mari-linhares/received_events","type":"User","site_admin":false},"body":"","created_at":"2018-12-16T00:34:01Z","updated_at":"2018-12-16T08:11:22Z","closed_at":null,"merged_at":null,"merge_commit_sha":"3c6ca10c61c48d1c51e364de82dc8754dae06d02","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/OpenMined/PySyft/pulls/1779/commits","review_comments_url":"https://api.github.com/repos/OpenMined/PySyft/pulls/1779/comments","review_comment_url":"https://api.github.com/repos/OpenMined/PySyft/pulls/comments{/number}","comments_url":"https://api.github.com/repos/OpenMined/PySyft/issues/1779/comments","statuses_url":"https://api.github.com/repos/OpenMined/PySyft/statuses/d93477dcd66850743d7689fc431cd04238d22f3d","head":{"label":"mari-linhares:issue_1760","ref":"issue_1760","sha":"d93477dcd66850743d7689fc431cd04238d22f3d","user":{"login":"mari-linhares","id":8157164,"node_id":"MDQ6VXNlcjgxNTcxNjQ=","avatar_url":"https://avatars3.githubusercontent.com/u/8157164?v=4","gravatar_id":"","url":"https://api.github.com/users/mari-linhares","html_url":"https://github.com/mari-linhares","followers_url":"https://api.github.com/users/mari-linhares/followers","following_url":"https://api.github.com/users/mari-linhares/following{/other_user}","gists_url":"https://api.github.com/users/mari-linhares/gists{/gist_id}","starred_url":"https://api.github.com/users/mari-linhares/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mari-linhares/subscriptions","organizations_url":"https://api.github.com/users/mari-linhares/orgs","repos_url":"https://api.github.com/users/mari-linhares/repos","events_url":"https://api.github.com/users/mari-linhares/events{/privacy}","received_events_url":"https://api.github.com/users/mari-linhares/received_events","type":"User","site_admin":false},"repo":{"id":159087166,"node_id":"MDEwOlJlcG9zaXRvcnkxNTkwODcxNjY=","name":"PySyft","full_name":"mari-linhares/PySyft","private":false,"owner":{"login":"mari-linhares","id":8157164,"node_id":"MDQ6VXNlcjgxNTcxNjQ=","avatar_url":"https://avatars3.githubusercontent.com/u/8157164?v=4","gravatar_id":"","url":"https://api.github.com/users/mari-linhares","html_url":"https://github.com/mari-linhares","followers_url":"https://api.github.com/users/mari-linhares/followers","following_url":"https://api.github.com/users/mari-linhares/following{/other_user}","gists_url":"https://api.github.com/users/mari-linhares/gists{/gist_id}","starred_url":"https://api.github.com/users/mari-linhares/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mari-linhares/subscriptions","organizations_url":"https://api.github.com/users/mari-linhares/orgs","repos_url":"https://api.github.com/users/mari-linhares/repos","events_url":"https://api.github.com/users/mari-linhares/events{/privacy}","received_events_url":"https://api.github.com/users/mari-linhares/received_events","type":"User","site_admin":false},"html_url":"https://github.com/mari-linhares/PySyft","description":"A library for encrypted, privacy preserving deep learning","fork":true,"url":"https://api.github.com/repos/mari-linhares/PySyft","forks_url":"https://api.github.com/repos/mari-linhares/PySyft/forks","keys_url":"https://api.github.com/repos/mari-linhares/PySyft/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mari-linhares/PySyft/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mari-linhares/PySyft/teams","hooks_url":"https://api.github.com/repos/mari-linhares/PySyft/hooks","issue_events_url":"https://api.github.com/repos/mari-linhares/PySyft/issues/events{/number}","events_url":"https://api.github.com/repos/mari-linhares/PySyft/events","assignees_url":"https://api.github.com/repos/mari-linhares/PySyft/assignees{/user}","branches_url":"https://api.github.com/repos/mari-linhares/PySyft/branches{/branch}","tags_url":"https://api.github.com/repos/mari-linhares/PySyft/tags","blobs_url":"https://api.github.com/repos/mari-linhares/PySyft/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mari-linhares/PySyft/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mari-linhares/PySyft/git/refs{/sha}","trees_url":"https://api.github.com/repos/mari-linhares/PySyft/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mari-linhares/PySyft/statuses/{sha}","languages_url":"https://api.github.com/repos/mari-linhares/PySyft/languages","stargazers_url":"https://api.github.com/repos/mari-linhares/PySyft/stargazers","contributors_url":"https://api.github.com/repos/mari-linhares/PySyft/contributors","subscribers_url":"https://api.github.com/repos/mari-linhares/PySyft/subscribers","subscription_url":"https://api.github.com/repos/mari-linhares/PySyft/subscription","commits_url":"https://api.github.com/repos/mari-linhares/PySyft/commits{/sha}","git_commits_url":"https://api.github.com/repos/mari-linhares/PySyft/git/commits{/sha}","comments_url":"https://api.github.com/repos/mari-linhares/PySyft/comments{/number}","issue_comment_url":"https://api.github.com/repos/mari-linhares/PySyft/issues/comments{/number}","contents_url":"https://api.github.com/repos/mari-linhares/PySyft/contents/{+path}","compare_url":"https://api.github.com/repos/mari-linhares/PySyft/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mari-linhares/PySyft/merges","archive_url":"https://api.github.com/repos/mari-linhares/PySyft/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mari-linhares/PySyft/downloads","issues_url":"https://api.github.com/repos/mari-linhares/PySyft/issues{/number}","pulls_url":"https://api.github.com/repos/mari-linhares/PySyft/pulls{/number}","milestones_url":"https://api.github.com/repos/mari-linhares/PySyft/milestones{/number}","notifications_url":"https://api.github.com/repos/mari-linhares/PySyft/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mari-linhares/PySyft/labels{/name}","releases_url":"https://api.github.com/repos/mari-linhares/PySyft/releases{/id}","deployments_url":"https://api.github.com/repos/mari-linhares/PySyft/deployments","created_at":"2018-11-26T00:10:39Z","updated_at":"2018-12-15T23:04:14Z","pushed_at":"2018-12-16T00:32:46Z","git_url":"git://github.com/mari-linhares/PySyft.git","ssh_url":"git@github.com:mari-linhares/PySyft.git","clone_url":"https://github.com/mari-linhares/PySyft.git","svn_url":"https://github.com/mari-linhares/PySyft","homepage":"","size":26365,"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":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0","node_id":"MDc6TGljZW5zZTI="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"OpenMined:master","ref":"master","sha":"da0b2e0ae0f53effdbcc53c22053023a62da4671","user":{"login":"OpenMined","id":30185530,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMwMTg1NTMw","avatar_url":"https://avatars3.githubusercontent.com/u/30185530?v=4","gravatar_id":"","url":"https://api.github.com/users/OpenMined","html_url":"https://github.com/OpenMined","followers_url":"https://api.github.com/users/OpenMined/followers","following_url":"https://api.github.com/users/OpenMined/following{/other_user}","gists_url":"https://api.github.com/users/OpenMined/gists{/gist_id}","starred_url":"https://api.github.com/users/OpenMined/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OpenMined/subscriptions","organizations_url":"https://api.github.com/users/OpenMined/orgs","repos_url":"https://api.github.com/users/OpenMined/repos","events_url":"https://api.github.com/users/OpenMined/events{/privacy}","received_events_url":"https://api.github.com/users/OpenMined/received_events","type":"Organization","site_admin":false},"repo":{"id":97641933,"node_id":"MDEwOlJlcG9zaXRvcnk5NzY0MTkzMw==","name":"PySyft","full_name":"OpenMined/PySyft","private":false,"owner":{"login":"OpenMined","id":30185530,"node_id":"MDEyOk9yZ2FuaXphdGlvbjMwMTg1NTMw","avatar_url":"https://avatars3.githubusercontent.com/u/30185530?v=4","gravatar_id":"","url":"https://api.github.com/users/OpenMined","html_url":"https://github.com/OpenMined","followers_url":"https://api.github.com/users/OpenMined/followers","following_url":"https://api.github.com/users/OpenMined/following{/other_user}","gists_url":"https://api.github.com/users/OpenMined/gists{/gist_id}","starred_url":"https://api.github.com/users/OpenMined/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/OpenMined/subscriptions","organizations_url":"https://api.github.com/users/OpenMined/orgs","repos_url":"https://api.github.com/users/OpenMined/repos","events_url":"https://api.github.com/users/OpenMined/events{/privacy}","received_events_url":"https://api.github.com/users/OpenMined/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/OpenMined/PySyft","description":"A library for encrypted, privacy preserving deep learning","fork":false,"url":"https://api.github.com/repos/OpenMined/PySyft","forks_url":"https://api.github.com/repos/OpenMined/PySyft/forks","keys_url":"https://api.github.com/repos/OpenMined/PySyft/keys{/key_id}","collaborators_url":"https://api.github.com/repos/OpenMined/PySyft/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/OpenMined/PySyft/teams","hooks_url":"https://api.github.com/repos/OpenMined/PySyft/hooks","issue_events_url":"https://api.github.com/repos/OpenMined/PySyft/issues/events{/number}","events_url":"https://api.github.com/repos/OpenMined/PySyft/events","assignees_url":"https://api.github.com/repos/OpenMined/PySyft/assignees{/user}","branches_url":"https://api.github.com/repos/OpenMined/PySyft/branches{/branch}","tags_url":"https://api.github.com/repos/OpenMined/PySyft/tags","blobs_url":"https://api.github.com/repos/OpenMined/PySyft/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/OpenMined/PySyft/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/OpenMined/PySyft/git/refs{/sha}","trees_url":"https://api.github.com/repos/OpenMined/PySyft/git/trees{/sha}","statuses_url":"https://api.github.com/repos/OpenMined/PySyft/statuses/{sha}","languages_url":"https://api.github.com/repos/OpenMined/PySyft/languages","stargazers_url":"https://api.github.com/repos/OpenMined/PySyft/stargazers","contributors_url":"https://api.github.com/repos/OpenMined/PySyft/contributors","subscribers_url":"https://api.github.com/repos/OpenMined/PySyft/subscribers","subscription_url":"https://api.github.com/repos/OpenMined/PySyft/subscription","commits_url":"https://api.github.com/repos/OpenMined/PySyft/commits{/sha}","git_commits_url":"https://api.github.com/repos/OpenMined/PySyft/git/commits{/sha}","comments_url":"https://api.github.com/repos/OpenMined/PySyft/comments{/number}","issue_comment_url":"https://api.github.com/repos/OpenMined/PySyft/issues/comments{/number}","contents_url":"https://api.github.com/repos/OpenMined/PySyft/contents/{+path}","compare_url":"https://api.github.com/repos/OpenMined/PySyft/compare/{base}...{head}","merges_url":"https://api.github.com/repos/OpenMined/PySyft/merges","archive_url":"https://api.github.com/repos/OpenMined/PySyft/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/OpenMined/PySyft/downloads","issues_url":"https://api.github.com/repos/OpenMined/PySyft/issues{/number}","pulls_url":"https://api.github.com/repos/OpenMined/PySyft/pulls{/number}","milestones_url":"https://api.github.com/repos/OpenMined/PySyft/milestones{/number}","notifications_url":"https://api.github.com/repos/OpenMined/PySyft/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/OpenMined/PySyft/labels{/name}","releases_url":"https://api.github.com/repos/OpenMined/PySyft/releases{/id}","deployments_url":"https://api.github.com/repos/OpenMined/PySyft/deployments","created_at":"2017-07-18T20:41:16Z","updated_at":"2018-12-16T06:38:10Z","pushed_at":"2018-12-16T05:15:54Z","git_url":"git://github.com/OpenMined/PySyft.git","ssh_url":"git@github.com:OpenMined/PySyft.git","clone_url":"https://github.com/OpenMined/PySyft.git","svn_url":"https://github.com/OpenMined/PySyft","homepage":"","size":26344,"stargazers_count":2435,"watchers_count":2435,"language":"Python","has_issues":true,"has_projects":false,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":565,"mirror_url":null,"archived":false,"open_issues_count":65,"license":{"key":"apache-2.0","name":"Apache License 2.0","spdx_id":"Apache-2.0","url":"https://api.github.com/licenses/apache-2.0","node_id":"MDc6TGljZW5zZTI="},"forks":565,"open_issues":65,"watchers":2435,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/OpenMined/PySyft/pulls/1779"},"html":{"href":"https://github.com/OpenMined/PySyft/pull/1779"},"issue":{"href":"https://api.github.com/repos/OpenMined/PySyft/issues/1779"},"comments":{"href":"https://api.github.com/repos/OpenMined/PySyft/issues/1779/comments"},"review_comments":{"href":"https://api.github.com/repos/OpenMined/PySyft/pulls/1779/comments"},"review_comment":{"href":"https://api.github.com/repos/OpenMined/PySyft/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/OpenMined/PySyft/pulls/1779/commits"},"statuses":{"href":"https://api.github.com/repos/OpenMined/PySyft/statuses/d93477dcd66850743d7689fc431cd04238d22f3d"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"mari-linhares"}}