{"action":"created","comment":{"url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/comments/220280025","pull_request_review_id":158652003,"id":220280025,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDIyMDI4MDAyNQ==","diff_hunk":"@@ -0,0 +1,69 @@\n+# Copyright 2017-2018 Amazon.com, Inc. or its affiliates. All Rights Reserved.\n+#\n+# Licensed under the Apache License, Version 2.0 (the \"License\"). You\n+# may not use this file except in compliance with the License. A copy of\n+# the License is located at\n+#\n+#     http://aws.amazon.com/apache2.0/\n+#\n+# or in the \"license\" file accompanying this file. This file is\n+# distributed on an \"AS IS\" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF\n+# ANY KIND, either express or implied. See the License for the specific\n+# language governing permissions and limitations under the License.\n+from __future__ import absolute_import\n+\n+import os\n+import pytest\n+\n+from sagemaker.tensorflow import TensorFlow\n+from test.integration.docker_utils import Container\n+\n+\n+@pytest.fixture\n+def py_full_version(py_version):\n+    if py_version == '2':\n+        return '2.7'\n+    else:\n+        return '3.6'\n+\n+\n+def test_py_versions(docker_image, processor, py_full_version):\n+    with Container(docker_image, processor) as c:","path":"test/integration/test_training.py","position":31,"original_position":31,"commit_id":"c9704230600f0a0336c591cffa21ab0a5c6beec2","original_commit_id":"c9704230600f0a0336c591cffa21ab0a5c6beec2","user":{"login":"mvsusp","id":142074,"node_id":"MDQ6VXNlcjE0MjA3NA==","avatar_url":"https://avatars0.githubusercontent.com/u/142074?v=4","gravatar_id":"","url":"https://api.github.com/users/mvsusp","html_url":"https://github.com/mvsusp","followers_url":"https://api.github.com/users/mvsusp/followers","following_url":"https://api.github.com/users/mvsusp/following{/other_user}","gists_url":"https://api.github.com/users/mvsusp/gists{/gist_id}","starred_url":"https://api.github.com/users/mvsusp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mvsusp/subscriptions","organizations_url":"https://api.github.com/users/mvsusp/orgs","repos_url":"https://api.github.com/users/mvsusp/repos","events_url":"https://api.github.com/users/mvsusp/events{/privacy}","received_events_url":"https://api.github.com/users/mvsusp/received_events","type":"User","site_admin":false},"body":"Good call.","created_at":"2018-09-25T17:17:04Z","updated_at":"2018-09-25T17:17:04Z","html_url":"https://github.com/aws/sagemaker-tensorflow-container/pull/78#discussion_r220280025","pull_request_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/78","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/comments/220280025"},"html":{"href":"https://github.com/aws/sagemaker-tensorflow-container/pull/78#discussion_r220280025"},"pull_request":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/78"}},"in_reply_to_id":220235214},"pull_request":{"url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/78","id":217850855,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE3ODUwODU1","html_url":"https://github.com/aws/sagemaker-tensorflow-container/pull/78","diff_url":"https://github.com/aws/sagemaker-tensorflow-container/pull/78.diff","patch_url":"https://github.com/aws/sagemaker-tensorflow-container/pull/78.patch","issue_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/issues/78","number":78,"state":"open","locked":false,"title":"Scriptmode single machine training implementation","user":{"login":"icywang86rui","id":24926915,"node_id":"MDQ6VXNlcjI0OTI2OTE1","avatar_url":"https://avatars0.githubusercontent.com/u/24926915?v=4","gravatar_id":"","url":"https://api.github.com/users/icywang86rui","html_url":"https://github.com/icywang86rui","followers_url":"https://api.github.com/users/icywang86rui/followers","following_url":"https://api.github.com/users/icywang86rui/following{/other_user}","gists_url":"https://api.github.com/users/icywang86rui/gists{/gist_id}","starred_url":"https://api.github.com/users/icywang86rui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icywang86rui/subscriptions","organizations_url":"https://api.github.com/users/icywang86rui/orgs","repos_url":"https://api.github.com/users/icywang86rui/repos","events_url":"https://api.github.com/users/icywang86rui/events{/privacy}","received_events_url":"https://api.github.com/users/icywang86rui/received_events","type":"User","site_admin":false},"body":"* Migrate to sagemaker-containers 2.1\r\n* Remove serving related packages and code from container\r\n* Add py3 container\r\n* Add integ and unit tests for script mode\r\n* Remove non-asci characters from  README\r\n\r\n*Issue #, if available:*\r\n\r\n*Description of changes:*\r\nI have run all the new tests manually. Creating the TC PR build config right now. I will add the build validation to this PR tomorrow.\r\n\r\nBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.\r\n","created_at":"2018-09-25T04:15:42Z","updated_at":"2018-09-25T17:17:04Z","closed_at":null,"merged_at":null,"merge_commit_sha":"6dc21bd4a1014436add4a6b0d6945e0b35f30056","assignee":null,"assignees":[],"requested_reviewers":[{"login":"laurenyu","id":6631887,"node_id":"MDQ6VXNlcjY2MzE4ODc=","avatar_url":"https://avatars2.githubusercontent.com/u/6631887?v=4","gravatar_id":"","url":"https://api.github.com/users/laurenyu","html_url":"https://github.com/laurenyu","followers_url":"https://api.github.com/users/laurenyu/followers","following_url":"https://api.github.com/users/laurenyu/following{/other_user}","gists_url":"https://api.github.com/users/laurenyu/gists{/gist_id}","starred_url":"https://api.github.com/users/laurenyu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/laurenyu/subscriptions","organizations_url":"https://api.github.com/users/laurenyu/orgs","repos_url":"https://api.github.com/users/laurenyu/repos","events_url":"https://api.github.com/users/laurenyu/events{/privacy}","received_events_url":"https://api.github.com/users/laurenyu/received_events","type":"User","site_admin":false}],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/78/commits","review_comments_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/78/comments","review_comment_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/comments{/number}","comments_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/issues/78/comments","statuses_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/statuses/c9704230600f0a0336c591cffa21ab0a5c6beec2","head":{"label":"icywang86rui:script-mode","ref":"script-mode","sha":"c9704230600f0a0336c591cffa21ab0a5c6beec2","user":{"login":"icywang86rui","id":24926915,"node_id":"MDQ6VXNlcjI0OTI2OTE1","avatar_url":"https://avatars0.githubusercontent.com/u/24926915?v=4","gravatar_id":"","url":"https://api.github.com/users/icywang86rui","html_url":"https://github.com/icywang86rui","followers_url":"https://api.github.com/users/icywang86rui/followers","following_url":"https://api.github.com/users/icywang86rui/following{/other_user}","gists_url":"https://api.github.com/users/icywang86rui/gists{/gist_id}","starred_url":"https://api.github.com/users/icywang86rui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icywang86rui/subscriptions","organizations_url":"https://api.github.com/users/icywang86rui/orgs","repos_url":"https://api.github.com/users/icywang86rui/repos","events_url":"https://api.github.com/users/icywang86rui/events{/privacy}","received_events_url":"https://api.github.com/users/icywang86rui/received_events","type":"User","site_admin":false},"repo":{"id":147874768,"node_id":"MDEwOlJlcG9zaXRvcnkxNDc4NzQ3Njg=","name":"sagemaker-tensorflow-container","full_name":"icywang86rui/sagemaker-tensorflow-container","private":false,"owner":{"login":"icywang86rui","id":24926915,"node_id":"MDQ6VXNlcjI0OTI2OTE1","avatar_url":"https://avatars0.githubusercontent.com/u/24926915?v=4","gravatar_id":"","url":"https://api.github.com/users/icywang86rui","html_url":"https://github.com/icywang86rui","followers_url":"https://api.github.com/users/icywang86rui/followers","following_url":"https://api.github.com/users/icywang86rui/following{/other_user}","gists_url":"https://api.github.com/users/icywang86rui/gists{/gist_id}","starred_url":"https://api.github.com/users/icywang86rui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/icywang86rui/subscriptions","organizations_url":"https://api.github.com/users/icywang86rui/orgs","repos_url":"https://api.github.com/users/icywang86rui/repos","events_url":"https://api.github.com/users/icywang86rui/events{/privacy}","received_events_url":"https://api.github.com/users/icywang86rui/received_events","type":"User","site_admin":false},"html_url":"https://github.com/icywang86rui/sagemaker-tensorflow-container","description":"This support code is used for making the TensorFlow framework run on Amazon SageMaker.","fork":true,"url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container","forks_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/forks","keys_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/keys{/key_id}","collaborators_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/teams","hooks_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/hooks","issue_events_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/issues/events{/number}","events_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/events","assignees_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/assignees{/user}","branches_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/branches{/branch}","tags_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/tags","blobs_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/git/refs{/sha}","trees_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/git/trees{/sha}","statuses_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/statuses/{sha}","languages_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/languages","stargazers_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/stargazers","contributors_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/contributors","subscribers_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/subscribers","subscription_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/subscription","commits_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/commits{/sha}","git_commits_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/git/commits{/sha}","comments_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/comments{/number}","issue_comment_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/issues/comments{/number}","contents_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/contents/{+path}","compare_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/compare/{base}...{head}","merges_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/merges","archive_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/downloads","issues_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/issues{/number}","pulls_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/pulls{/number}","milestones_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/milestones{/number}","notifications_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/labels{/name}","releases_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/releases{/id}","deployments_url":"https://api.github.com/repos/icywang86rui/sagemaker-tensorflow-container/deployments","created_at":"2018-09-07T21:06:01Z","updated_at":"2018-09-07T21:06:04Z","pushed_at":"2018-09-25T04:11:09Z","git_url":"git://github.com/icywang86rui/sagemaker-tensorflow-container.git","ssh_url":"git@github.com:icywang86rui/sagemaker-tensorflow-container.git","clone_url":"https://github.com/icywang86rui/sagemaker-tensorflow-container.git","svn_url":"https://github.com/icywang86rui/sagemaker-tensorflow-container","homepage":null,"size":12775,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":false,"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":"aws:script-mode","ref":"script-mode","sha":"8acf51de4a98a4f70de1003535674a8933347825","user":{"login":"aws","id":2232217,"node_id":"MDEyOk9yZ2FuaXphdGlvbjIyMzIyMTc=","avatar_url":"https://avatars3.githubusercontent.com/u/2232217?v=4","gravatar_id":"","url":"https://api.github.com/users/aws","html_url":"https://github.com/aws","followers_url":"https://api.github.com/users/aws/followers","following_url":"https://api.github.com/users/aws/following{/other_user}","gists_url":"https://api.github.com/users/aws/gists{/gist_id}","starred_url":"https://api.github.com/users/aws/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aws/subscriptions","organizations_url":"https://api.github.com/users/aws/orgs","repos_url":"https://api.github.com/users/aws/repos","events_url":"https://api.github.com/users/aws/events{/privacy}","received_events_url":"https://api.github.com/users/aws/received_events","type":"Organization","site_admin":false},"repo":{"id":118646184,"node_id":"MDEwOlJlcG9zaXRvcnkxMTg2NDYxODQ=","name":"sagemaker-tensorflow-container","full_name":"aws/sagemaker-tensorflow-container","private":false,"owner":{"login":"aws","id":2232217,"node_id":"MDEyOk9yZ2FuaXphdGlvbjIyMzIyMTc=","avatar_url":"https://avatars3.githubusercontent.com/u/2232217?v=4","gravatar_id":"","url":"https://api.github.com/users/aws","html_url":"https://github.com/aws","followers_url":"https://api.github.com/users/aws/followers","following_url":"https://api.github.com/users/aws/following{/other_user}","gists_url":"https://api.github.com/users/aws/gists{/gist_id}","starred_url":"https://api.github.com/users/aws/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aws/subscriptions","organizations_url":"https://api.github.com/users/aws/orgs","repos_url":"https://api.github.com/users/aws/repos","events_url":"https://api.github.com/users/aws/events{/privacy}","received_events_url":"https://api.github.com/users/aws/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/aws/sagemaker-tensorflow-container","description":"This support code is used for making the TensorFlow framework run on Amazon SageMaker.","fork":false,"url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container","forks_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/forks","keys_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/keys{/key_id}","collaborators_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/teams","hooks_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/hooks","issue_events_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/issues/events{/number}","events_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/events","assignees_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/assignees{/user}","branches_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/branches{/branch}","tags_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/tags","blobs_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/git/refs{/sha}","trees_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/git/trees{/sha}","statuses_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/statuses/{sha}","languages_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/languages","stargazers_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/stargazers","contributors_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/contributors","subscribers_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/subscribers","subscription_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/subscription","commits_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/commits{/sha}","git_commits_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/git/commits{/sha}","comments_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/comments{/number}","issue_comment_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/issues/comments{/number}","contents_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/contents/{+path}","compare_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/compare/{base}...{head}","merges_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/merges","archive_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/downloads","issues_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/issues{/number}","pulls_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls{/number}","milestones_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/milestones{/number}","notifications_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/labels{/name}","releases_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/releases{/id}","deployments_url":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/deployments","created_at":"2018-01-23T17:41:21Z","updated_at":"2018-09-25T12:31:20Z","pushed_at":"2018-09-25T04:15:43Z","git_url":"git://github.com/aws/sagemaker-tensorflow-container.git","ssh_url":"git@github.com:aws/sagemaker-tensorflow-container.git","clone_url":"https://github.com/aws/sagemaker-tensorflow-container.git","svn_url":"https://github.com/aws/sagemaker-tensorflow-container","homepage":null,"size":12146,"stargazers_count":74,"watchers_count":74,"language":"Python","has_issues":true,"has_projects":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":33,"mirror_url":null,"archived":false,"open_issues_count":10,"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":33,"open_issues":10,"watchers":74,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/78"},"html":{"href":"https://github.com/aws/sagemaker-tensorflow-container/pull/78"},"issue":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/issues/78"},"comments":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/issues/78/comments"},"review_comments":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/78/comments"},"review_comment":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/pulls/78/commits"},"statuses":{"href":"https://api.github.com/repos/aws/sagemaker-tensorflow-container/statuses/c9704230600f0a0336c591cffa21ab0a5c6beec2"}},"author_association":"NONE"}}
{"actor":{"display_login":"mvsusp"}}