{"action":"created","comment":{"url":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/comments/115078896","pull_request_review_id":36599324,"id":115078896,"diff_hunk":"@@ -150,18 +149,20 @@ def create_test_model(fset, model_type='RandomForestClassifier'):\n \n \n @contextmanager\n-def create_test_prediction(dataset, model):\n+def create_test_prediction(dataset, featureset, model):\n     \"\"\"Create and yield test prediction, then delete.\n \n     Params\n     ------\n     dataset : `models.Dataset` instance\n-        The dataset on which prediction will be performed.\n+        Dummy dataset used to create prediction instance.\n+    featureset : `models.Featureset` instance\n+        The featureset on which prediction will be performed.\n     model : `models.Model` instance\n         The model to use to create prediction.\n \n     \"\"\"\n-    fset, data = featurize.load_featureset(model.featureset.file.uri)\n+    fset, data = featurize.load_featureset(featureset.file.uri)","path":"cesium_app/tests/fixtures.py","position":43,"original_position":43,"commit_id":"c84206247bdca3e673ed7720875ef003cfd056b1","original_commit_id":"c84206247bdca3e673ed7720875ef003cfd056b1","user":{"login":"acrellin","id":7230285,"avatar_url":"https://avatars3.githubusercontent.com/u/7230285?v=3","gravatar_id":"","url":"https://api.github.com/users/acrellin","html_url":"https://github.com/acrellin","followers_url":"https://api.github.com/users/acrellin/followers","following_url":"https://api.github.com/users/acrellin/following{/other_user}","gists_url":"https://api.github.com/users/acrellin/gists{/gist_id}","starred_url":"https://api.github.com/users/acrellin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/acrellin/subscriptions","organizations_url":"https://api.github.com/users/acrellin/orgs","repos_url":"https://api.github.com/users/acrellin/repos","events_url":"https://api.github.com/users/acrellin/events{/privacy}","received_events_url":"https://api.github.com/users/acrellin/received_events","type":"User","site_admin":false},"body":"No, that's the whole point of this change - to test w/ unlabeled data.","created_at":"2017-05-05T20:12:57Z","updated_at":"2017-05-05T20:12:57Z","html_url":"https://github.com/cesium-ml/cesium_web/pull/202#discussion_r115078896","pull_request_url":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/202","_links":{"self":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/comments/115078896"},"html":{"href":"https://github.com/cesium-ml/cesium_web/pull/202#discussion_r115078896"},"pull_request":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/202"}}},"pull_request":{"url":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/202","id":119094075,"html_url":"https://github.com/cesium-ml/cesium_web/pull/202","diff_url":"https://github.com/cesium-ml/cesium_web/pull/202.diff","patch_url":"https://github.com/cesium-ml/cesium_web/pull/202.patch","issue_url":"https://api.github.com/repos/cesium-ml/cesium_web/issues/202","number":202,"state":"open","locked":false,"title":"Allow for downloading of unlabeled prediction data","user":{"login":"acrellin","id":7230285,"avatar_url":"https://avatars3.githubusercontent.com/u/7230285?v=3","gravatar_id":"","url":"https://api.github.com/users/acrellin","html_url":"https://github.com/acrellin","followers_url":"https://api.github.com/users/acrellin/followers","following_url":"https://api.github.com/users/acrellin/following{/other_user}","gists_url":"https://api.github.com/users/acrellin/gists{/gist_id}","starred_url":"https://api.github.com/users/acrellin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/acrellin/subscriptions","organizations_url":"https://api.github.com/users/acrellin/orgs","repos_url":"https://api.github.com/users/acrellin/repos","events_url":"https://api.github.com/users/acrellin/events{/privacy}","received_events_url":"https://api.github.com/users/acrellin/received_events","type":"User","site_admin":false},"body":"- Fixes bug resulting in error if attempt to download unlabeled prediction results was made.\r\n- Adds corresponding test.\r\n- Updates test fixtures.","created_at":"2017-05-05T00:43:10Z","updated_at":"2017-05-05T20:12:57Z","closed_at":null,"merged_at":null,"merge_commit_sha":"58f61923ef2e9b90d06edeb8050a85cadca508cd","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/202/commits","review_comments_url":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/202/comments","review_comment_url":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/comments{/number}","comments_url":"https://api.github.com/repos/cesium-ml/cesium_web/issues/202/comments","statuses_url":"https://api.github.com/repos/cesium-ml/cesium_web/statuses/c84206247bdca3e673ed7720875ef003cfd056b1","head":{"label":"acrellin:download_unlabeled_pred_results","ref":"download_unlabeled_pred_results","sha":"c84206247bdca3e673ed7720875ef003cfd056b1","user":{"login":"acrellin","id":7230285,"avatar_url":"https://avatars3.githubusercontent.com/u/7230285?v=3","gravatar_id":"","url":"https://api.github.com/users/acrellin","html_url":"https://github.com/acrellin","followers_url":"https://api.github.com/users/acrellin/followers","following_url":"https://api.github.com/users/acrellin/following{/other_user}","gists_url":"https://api.github.com/users/acrellin/gists{/gist_id}","starred_url":"https://api.github.com/users/acrellin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/acrellin/subscriptions","organizations_url":"https://api.github.com/users/acrellin/orgs","repos_url":"https://api.github.com/users/acrellin/repos","events_url":"https://api.github.com/users/acrellin/events{/privacy}","received_events_url":"https://api.github.com/users/acrellin/received_events","type":"User","site_admin":false},"repo":{"id":61676520,"name":"cesium_web","full_name":"acrellin/cesium_web","owner":{"login":"acrellin","id":7230285,"avatar_url":"https://avatars3.githubusercontent.com/u/7230285?v=3","gravatar_id":"","url":"https://api.github.com/users/acrellin","html_url":"https://github.com/acrellin","followers_url":"https://api.github.com/users/acrellin/followers","following_url":"https://api.github.com/users/acrellin/following{/other_user}","gists_url":"https://api.github.com/users/acrellin/gists{/gist_id}","starred_url":"https://api.github.com/users/acrellin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/acrellin/subscriptions","organizations_url":"https://api.github.com/users/acrellin/orgs","repos_url":"https://api.github.com/users/acrellin/repos","events_url":"https://api.github.com/users/acrellin/events{/privacy}","received_events_url":"https://api.github.com/users/acrellin/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/acrellin/cesium_web","description":"Cesium Web Frontend","fork":true,"url":"https://api.github.com/repos/acrellin/cesium_web","forks_url":"https://api.github.com/repos/acrellin/cesium_web/forks","keys_url":"https://api.github.com/repos/acrellin/cesium_web/keys{/key_id}","collaborators_url":"https://api.github.com/repos/acrellin/cesium_web/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/acrellin/cesium_web/teams","hooks_url":"https://api.github.com/repos/acrellin/cesium_web/hooks","issue_events_url":"https://api.github.com/repos/acrellin/cesium_web/issues/events{/number}","events_url":"https://api.github.com/repos/acrellin/cesium_web/events","assignees_url":"https://api.github.com/repos/acrellin/cesium_web/assignees{/user}","branches_url":"https://api.github.com/repos/acrellin/cesium_web/branches{/branch}","tags_url":"https://api.github.com/repos/acrellin/cesium_web/tags","blobs_url":"https://api.github.com/repos/acrellin/cesium_web/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/acrellin/cesium_web/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/acrellin/cesium_web/git/refs{/sha}","trees_url":"https://api.github.com/repos/acrellin/cesium_web/git/trees{/sha}","statuses_url":"https://api.github.com/repos/acrellin/cesium_web/statuses/{sha}","languages_url":"https://api.github.com/repos/acrellin/cesium_web/languages","stargazers_url":"https://api.github.com/repos/acrellin/cesium_web/stargazers","contributors_url":"https://api.github.com/repos/acrellin/cesium_web/contributors","subscribers_url":"https://api.github.com/repos/acrellin/cesium_web/subscribers","subscription_url":"https://api.github.com/repos/acrellin/cesium_web/subscription","commits_url":"https://api.github.com/repos/acrellin/cesium_web/commits{/sha}","git_commits_url":"https://api.github.com/repos/acrellin/cesium_web/git/commits{/sha}","comments_url":"https://api.github.com/repos/acrellin/cesium_web/comments{/number}","issue_comment_url":"https://api.github.com/repos/acrellin/cesium_web/issues/comments{/number}","contents_url":"https://api.github.com/repos/acrellin/cesium_web/contents/{+path}","compare_url":"https://api.github.com/repos/acrellin/cesium_web/compare/{base}...{head}","merges_url":"https://api.github.com/repos/acrellin/cesium_web/merges","archive_url":"https://api.github.com/repos/acrellin/cesium_web/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/acrellin/cesium_web/downloads","issues_url":"https://api.github.com/repos/acrellin/cesium_web/issues{/number}","pulls_url":"https://api.github.com/repos/acrellin/cesium_web/pulls{/number}","milestones_url":"https://api.github.com/repos/acrellin/cesium_web/milestones{/number}","notifications_url":"https://api.github.com/repos/acrellin/cesium_web/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/acrellin/cesium_web/labels{/name}","releases_url":"https://api.github.com/repos/acrellin/cesium_web/releases{/id}","deployments_url":"https://api.github.com/repos/acrellin/cesium_web/deployments","created_at":"2016-06-22T00:30:28Z","updated_at":"2016-11-22T19:51:53Z","pushed_at":"2017-05-05T19:14:52Z","git_url":"git://github.com/acrellin/cesium_web.git","ssh_url":"git@github.com:acrellin/cesium_web.git","clone_url":"https://github.com/acrellin/cesium_web.git","svn_url":"https://github.com/acrellin/cesium_web","homepage":null,"size":995,"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,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"cesium-ml:master","ref":"master","sha":"69486819aeb5315fbc72fe62354e29e7c807a40d","user":{"login":"cesium-ml","id":17838960,"avatar_url":"https://avatars1.githubusercontent.com/u/17838960?v=3","gravatar_id":"","url":"https://api.github.com/users/cesium-ml","html_url":"https://github.com/cesium-ml","followers_url":"https://api.github.com/users/cesium-ml/followers","following_url":"https://api.github.com/users/cesium-ml/following{/other_user}","gists_url":"https://api.github.com/users/cesium-ml/gists{/gist_id}","starred_url":"https://api.github.com/users/cesium-ml/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cesium-ml/subscriptions","organizations_url":"https://api.github.com/users/cesium-ml/orgs","repos_url":"https://api.github.com/users/cesium-ml/repos","events_url":"https://api.github.com/users/cesium-ml/events{/privacy}","received_events_url":"https://api.github.com/users/cesium-ml/received_events","type":"Organization","site_admin":false},"repo":{"id":61668340,"name":"cesium_web","full_name":"cesium-ml/cesium_web","owner":{"login":"cesium-ml","id":17838960,"avatar_url":"https://avatars1.githubusercontent.com/u/17838960?v=3","gravatar_id":"","url":"https://api.github.com/users/cesium-ml","html_url":"https://github.com/cesium-ml","followers_url":"https://api.github.com/users/cesium-ml/followers","following_url":"https://api.github.com/users/cesium-ml/following{/other_user}","gists_url":"https://api.github.com/users/cesium-ml/gists{/gist_id}","starred_url":"https://api.github.com/users/cesium-ml/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cesium-ml/subscriptions","organizations_url":"https://api.github.com/users/cesium-ml/orgs","repos_url":"https://api.github.com/users/cesium-ml/repos","events_url":"https://api.github.com/users/cesium-ml/events{/privacy}","received_events_url":"https://api.github.com/users/cesium-ml/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/cesium-ml/cesium_web","description":"Cesium Web Frontend","fork":false,"url":"https://api.github.com/repos/cesium-ml/cesium_web","forks_url":"https://api.github.com/repos/cesium-ml/cesium_web/forks","keys_url":"https://api.github.com/repos/cesium-ml/cesium_web/keys{/key_id}","collaborators_url":"https://api.github.com/repos/cesium-ml/cesium_web/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/cesium-ml/cesium_web/teams","hooks_url":"https://api.github.com/repos/cesium-ml/cesium_web/hooks","issue_events_url":"https://api.github.com/repos/cesium-ml/cesium_web/issues/events{/number}","events_url":"https://api.github.com/repos/cesium-ml/cesium_web/events","assignees_url":"https://api.github.com/repos/cesium-ml/cesium_web/assignees{/user}","branches_url":"https://api.github.com/repos/cesium-ml/cesium_web/branches{/branch}","tags_url":"https://api.github.com/repos/cesium-ml/cesium_web/tags","blobs_url":"https://api.github.com/repos/cesium-ml/cesium_web/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/cesium-ml/cesium_web/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/cesium-ml/cesium_web/git/refs{/sha}","trees_url":"https://api.github.com/repos/cesium-ml/cesium_web/git/trees{/sha}","statuses_url":"https://api.github.com/repos/cesium-ml/cesium_web/statuses/{sha}","languages_url":"https://api.github.com/repos/cesium-ml/cesium_web/languages","stargazers_url":"https://api.github.com/repos/cesium-ml/cesium_web/stargazers","contributors_url":"https://api.github.com/repos/cesium-ml/cesium_web/contributors","subscribers_url":"https://api.github.com/repos/cesium-ml/cesium_web/subscribers","subscription_url":"https://api.github.com/repos/cesium-ml/cesium_web/subscription","commits_url":"https://api.github.com/repos/cesium-ml/cesium_web/commits{/sha}","git_commits_url":"https://api.github.com/repos/cesium-ml/cesium_web/git/commits{/sha}","comments_url":"https://api.github.com/repos/cesium-ml/cesium_web/comments{/number}","issue_comment_url":"https://api.github.com/repos/cesium-ml/cesium_web/issues/comments{/number}","contents_url":"https://api.github.com/repos/cesium-ml/cesium_web/contents/{+path}","compare_url":"https://api.github.com/repos/cesium-ml/cesium_web/compare/{base}...{head}","merges_url":"https://api.github.com/repos/cesium-ml/cesium_web/merges","archive_url":"https://api.github.com/repos/cesium-ml/cesium_web/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/cesium-ml/cesium_web/downloads","issues_url":"https://api.github.com/repos/cesium-ml/cesium_web/issues{/number}","pulls_url":"https://api.github.com/repos/cesium-ml/cesium_web/pulls{/number}","milestones_url":"https://api.github.com/repos/cesium-ml/cesium_web/milestones{/number}","notifications_url":"https://api.github.com/repos/cesium-ml/cesium_web/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/cesium-ml/cesium_web/labels{/name}","releases_url":"https://api.github.com/repos/cesium-ml/cesium_web/releases{/id}","deployments_url":"https://api.github.com/repos/cesium-ml/cesium_web/deployments","created_at":"2016-06-21T21:27:14Z","updated_at":"2017-03-23T16:14:34Z","pushed_at":"2017-05-05T19:14:53Z","git_url":"git://github.com/cesium-ml/cesium_web.git","ssh_url":"git@github.com:cesium-ml/cesium_web.git","clone_url":"https://github.com/cesium-ml/cesium_web.git","svn_url":"https://github.com/cesium-ml/cesium_web","homepage":null,"size":995,"stargazers_count":5,"watchers_count":5,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":14,"mirror_url":null,"open_issues_count":31,"forks":14,"open_issues":31,"watchers":5,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/202"},"html":{"href":"https://github.com/cesium-ml/cesium_web/pull/202"},"issue":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/issues/202"},"comments":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/issues/202/comments"},"review_comments":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/202/comments"},"review_comment":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/pulls/202/commits"},"statuses":{"href":"https://api.github.com/repos/cesium-ml/cesium_web/statuses/c84206247bdca3e673ed7720875ef003cfd056b1"}}}}
{"actor":{"display_login":"acrellin"}}