{"action":"created","comment":{"url":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/comments/203557213","pull_request_review_id":138471730,"id":203557213,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIwMzU1NzIxMw==","diff_hunk":"@@ -341,9 +340,13 @@ def run(self):\n                                                  key_parts[\"t_index\"])\n \n             try:\n+                print(\"x_tile_size: \" + str(self.config.config_data['ingest_job'][\"tile_size\"][\"x\"]))\n+                print(\"x_tile_size: \" + str(self.config.config_data['ingest_job'][\"tile_size\"][\"x\"]))","path":"ingestclient/core/engine.py","position":13,"original_position":13,"commit_id":"40f065fb58c78398ee4ca203c0effc2fff2563b9","original_commit_id":"40f065fb58c78398ee4ca203c0effc2fff2563b9","user":{"login":"movestill","id":7428355,"node_id":"MDQ6VXNlcjc0MjgzNTU=","avatar_url":"https://avatars3.githubusercontent.com/u/7428355?v=4","gravatar_id":"","url":"https://api.github.com/users/movestill","html_url":"https://github.com/movestill","followers_url":"https://api.github.com/users/movestill/followers","following_url":"https://api.github.com/users/movestill/following{/other_user}","gists_url":"https://api.github.com/users/movestill/gists{/gist_id}","starred_url":"https://api.github.com/users/movestill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/movestill/subscriptions","organizations_url":"https://api.github.com/users/movestill/orgs","repos_url":"https://api.github.com/users/movestill/repos","events_url":"https://api.github.com/users/movestill/events{/privacy}","received_events_url":"https://api.github.com/users/movestill/received_events","type":"User","site_admin":false},"body":"You're print x_tile_size twice.","created_at":"2018-07-18T22:59:44Z","updated_at":"2018-07-18T22:59:45Z","html_url":"https://github.com/jhuapl-boss/ingest-client/pull/11#discussion_r203557213","pull_request_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/11","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/comments/203557213"},"html":{"href":"https://github.com/jhuapl-boss/ingest-client/pull/11#discussion_r203557213"},"pull_request":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/11"}}},"pull_request":{"url":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/11","id":202368836,"node_id":"MDExOlB1bGxSZXF1ZXN0MjAyMzY4ODM2","html_url":"https://github.com/jhuapl-boss/ingest-client/pull/11","diff_url":"https://github.com/jhuapl-boss/ingest-client/pull/11.diff","patch_url":"https://github.com/jhuapl-boss/ingest-client/pull/11.patch","issue_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues/11","number":11,"state":"open","locked":false,"title":"Tile Sizes added to S3 Put MetaData","user":{"login":"sandyhider","id":5942883,"node_id":"MDQ6VXNlcjU5NDI4ODM=","avatar_url":"https://avatars0.githubusercontent.com/u/5942883?v=4","gravatar_id":"","url":"https://api.github.com/users/sandyhider","html_url":"https://github.com/sandyhider","followers_url":"https://api.github.com/users/sandyhider/followers","following_url":"https://api.github.com/users/sandyhider/following{/other_user}","gists_url":"https://api.github.com/users/sandyhider/gists{/gist_id}","starred_url":"https://api.github.com/users/sandyhider/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sandyhider/subscriptions","organizations_url":"https://api.github.com/users/sandyhider/orgs","repos_url":"https://api.github.com/users/sandyhider/repos","events_url":"https://api.github.com/users/sandyhider/events{/privacy}","received_events_url":"https://api.github.com/users/sandyhider/received_events","type":"User","site_admin":false},"body":"These changes are  needed to allow ingest_lambda correctly handle corrupt tiles.  With out them it does not know how to shape the ndarray given a corrupt tile.\r\n\r\nadded x_tile and y_tile to engine.py s3 put metadata, which is needed by the ingest_lambda on the backend to handle corrupt tiles.\r\n\r\nFixed a misleading argument tag in debug.py\r\nUpdated version and change log to reflect the changes","created_at":"2018-07-18T20:22:04Z","updated_at":"2018-07-18T22:59:45Z","closed_at":null,"merged_at":null,"merge_commit_sha":"c11667eeac95ca32dd8470d82e8b90872ba05065","assignee":null,"assignees":[],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/11/commits","review_comments_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/11/comments","review_comment_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/comments{/number}","comments_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues/11/comments","statuses_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/statuses/40f065fb58c78398ee4ca203c0effc2fff2563b9","head":{"label":"jhuapl-boss:MICRONS-1336","ref":"MICRONS-1336","sha":"40f065fb58c78398ee4ca203c0effc2fff2563b9","user":{"login":"jhuapl-boss","id":18535655,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE4NTM1NjU1","avatar_url":"https://avatars2.githubusercontent.com/u/18535655?v=4","gravatar_id":"","url":"https://api.github.com/users/jhuapl-boss","html_url":"https://github.com/jhuapl-boss","followers_url":"https://api.github.com/users/jhuapl-boss/followers","following_url":"https://api.github.com/users/jhuapl-boss/following{/other_user}","gists_url":"https://api.github.com/users/jhuapl-boss/gists{/gist_id}","starred_url":"https://api.github.com/users/jhuapl-boss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhuapl-boss/subscriptions","organizations_url":"https://api.github.com/users/jhuapl-boss/orgs","repos_url":"https://api.github.com/users/jhuapl-boss/repos","events_url":"https://api.github.com/users/jhuapl-boss/events{/privacy}","received_events_url":"https://api.github.com/users/jhuapl-boss/received_events","type":"Organization","site_admin":false},"repo":{"id":65855357,"node_id":"MDEwOlJlcG9zaXRvcnk2NTg1NTM1Nw==","name":"ingest-client","full_name":"jhuapl-boss/ingest-client","owner":{"login":"jhuapl-boss","id":18535655,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE4NTM1NjU1","avatar_url":"https://avatars2.githubusercontent.com/u/18535655?v=4","gravatar_id":"","url":"https://api.github.com/users/jhuapl-boss","html_url":"https://github.com/jhuapl-boss","followers_url":"https://api.github.com/users/jhuapl-boss/followers","following_url":"https://api.github.com/users/jhuapl-boss/following{/other_user}","gists_url":"https://api.github.com/users/jhuapl-boss/gists{/gist_id}","starred_url":"https://api.github.com/users/jhuapl-boss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhuapl-boss/subscriptions","organizations_url":"https://api.github.com/users/jhuapl-boss/orgs","repos_url":"https://api.github.com/users/jhuapl-boss/repos","events_url":"https://api.github.com/users/jhuapl-boss/events{/privacy}","received_events_url":"https://api.github.com/users/jhuapl-boss/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/jhuapl-boss/ingest-client","description":"A Python command line application for performing distributed ingest of image data into the Boss","fork":false,"url":"https://api.github.com/repos/jhuapl-boss/ingest-client","forks_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/forks","keys_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/teams","hooks_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/hooks","issue_events_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues/events{/number}","events_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/events","assignees_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/assignees{/user}","branches_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/branches{/branch}","tags_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/tags","blobs_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/refs{/sha}","trees_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/statuses/{sha}","languages_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/languages","stargazers_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/stargazers","contributors_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/contributors","subscribers_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/subscribers","subscription_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/subscription","commits_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/commits{/sha}","git_commits_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/commits{/sha}","comments_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/comments{/number}","issue_comment_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues/comments{/number}","contents_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/contents/{+path}","compare_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/merges","archive_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/downloads","issues_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues{/number}","pulls_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls{/number}","milestones_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/milestones{/number}","notifications_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/labels{/name}","releases_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/releases{/id}","deployments_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/deployments","created_at":"2016-08-16T21:36:54Z","updated_at":"2018-07-03T16:17:58Z","pushed_at":"2018-07-18T20:22:04Z","git_url":"git://github.com/jhuapl-boss/ingest-client.git","ssh_url":"git@github.com:jhuapl-boss/ingest-client.git","clone_url":"https://github.com/jhuapl-boss/ingest-client.git","svn_url":"https://github.com/jhuapl-boss/ingest-client","homepage":"","size":3136,"stargazers_count":4,"watchers_count":4,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":9,"mirror_url":null,"archived":false,"open_issues_count":3,"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":9,"open_issues":3,"watchers":4,"default_branch":"master"}},"base":{"label":"jhuapl-boss:master","ref":"master","sha":"64b054b392117cc7863f77e6a44deec27939e3ef","user":{"login":"jhuapl-boss","id":18535655,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE4NTM1NjU1","avatar_url":"https://avatars2.githubusercontent.com/u/18535655?v=4","gravatar_id":"","url":"https://api.github.com/users/jhuapl-boss","html_url":"https://github.com/jhuapl-boss","followers_url":"https://api.github.com/users/jhuapl-boss/followers","following_url":"https://api.github.com/users/jhuapl-boss/following{/other_user}","gists_url":"https://api.github.com/users/jhuapl-boss/gists{/gist_id}","starred_url":"https://api.github.com/users/jhuapl-boss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhuapl-boss/subscriptions","organizations_url":"https://api.github.com/users/jhuapl-boss/orgs","repos_url":"https://api.github.com/users/jhuapl-boss/repos","events_url":"https://api.github.com/users/jhuapl-boss/events{/privacy}","received_events_url":"https://api.github.com/users/jhuapl-boss/received_events","type":"Organization","site_admin":false},"repo":{"id":65855357,"node_id":"MDEwOlJlcG9zaXRvcnk2NTg1NTM1Nw==","name":"ingest-client","full_name":"jhuapl-boss/ingest-client","owner":{"login":"jhuapl-boss","id":18535655,"node_id":"MDEyOk9yZ2FuaXphdGlvbjE4NTM1NjU1","avatar_url":"https://avatars2.githubusercontent.com/u/18535655?v=4","gravatar_id":"","url":"https://api.github.com/users/jhuapl-boss","html_url":"https://github.com/jhuapl-boss","followers_url":"https://api.github.com/users/jhuapl-boss/followers","following_url":"https://api.github.com/users/jhuapl-boss/following{/other_user}","gists_url":"https://api.github.com/users/jhuapl-boss/gists{/gist_id}","starred_url":"https://api.github.com/users/jhuapl-boss/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhuapl-boss/subscriptions","organizations_url":"https://api.github.com/users/jhuapl-boss/orgs","repos_url":"https://api.github.com/users/jhuapl-boss/repos","events_url":"https://api.github.com/users/jhuapl-boss/events{/privacy}","received_events_url":"https://api.github.com/users/jhuapl-boss/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/jhuapl-boss/ingest-client","description":"A Python command line application for performing distributed ingest of image data into the Boss","fork":false,"url":"https://api.github.com/repos/jhuapl-boss/ingest-client","forks_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/forks","keys_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/teams","hooks_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/hooks","issue_events_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues/events{/number}","events_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/events","assignees_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/assignees{/user}","branches_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/branches{/branch}","tags_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/tags","blobs_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/refs{/sha}","trees_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/statuses/{sha}","languages_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/languages","stargazers_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/stargazers","contributors_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/contributors","subscribers_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/subscribers","subscription_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/subscription","commits_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/commits{/sha}","git_commits_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/git/commits{/sha}","comments_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/comments{/number}","issue_comment_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues/comments{/number}","contents_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/contents/{+path}","compare_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/merges","archive_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/downloads","issues_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues{/number}","pulls_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls{/number}","milestones_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/milestones{/number}","notifications_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/labels{/name}","releases_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/releases{/id}","deployments_url":"https://api.github.com/repos/jhuapl-boss/ingest-client/deployments","created_at":"2016-08-16T21:36:54Z","updated_at":"2018-07-03T16:17:58Z","pushed_at":"2018-07-18T20:22:04Z","git_url":"git://github.com/jhuapl-boss/ingest-client.git","ssh_url":"git@github.com:jhuapl-boss/ingest-client.git","clone_url":"https://github.com/jhuapl-boss/ingest-client.git","svn_url":"https://github.com/jhuapl-boss/ingest-client","homepage":"","size":3136,"stargazers_count":4,"watchers_count":4,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":9,"mirror_url":null,"archived":false,"open_issues_count":3,"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":9,"open_issues":3,"watchers":4,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/11"},"html":{"href":"https://github.com/jhuapl-boss/ingest-client/pull/11"},"issue":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues/11"},"comments":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/issues/11/comments"},"review_comments":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/11/comments"},"review_comment":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/pulls/11/commits"},"statuses":{"href":"https://api.github.com/repos/jhuapl-boss/ingest-client/statuses/40f065fb58c78398ee4ca203c0effc2fff2563b9"}},"author_association":"MEMBER"}}
{"actor":{"display_login":"movestill"}}