Skip to content

Conversation

@laurenyu
Copy link
Contributor

@laurenyu laurenyu commented Mar 2, 2020

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@laurenyu laurenyu force-pushed the python-sdk-in-dlc branch from 3fe2f65 to fdb339f Compare March 3, 2020 19:49
@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@YYStreet YYStreet requested review from a user and YYStreet March 3, 2020 21:25
@YYStreet YYStreet changed the title change: install SageMaker Python SDK into Python 3 images change: install SageMaker Python SDK into Python 3 images - TF 1.5.12 Mar 3, 2020
@YYStreet YYStreet changed the title change: install SageMaker Python SDK into Python 3 images - TF 1.5.12 change: install SageMaker Python SDK into Python 3 images - TF 1.15.2 Mar 3, 2020
@YYStreet
Copy link
Contributor

YYStreet commented Mar 3, 2020

Similar issues reported when adding smdebug to Tensorflow 1.15.2
#298
The error logs look nearly the same

@laurenyu
Copy link
Contributor Author

laurenyu commented Mar 3, 2020

that's because the buildspec isn't right. see #297. that's why we haven't required the PR build for merges yet

@YYStreet
Copy link
Contributor

YYStreet commented Mar 3, 2020

that's because the buildspec isn't right. see #297. that's why we haven't required the PR build for merges yet

Thanks!

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot
Copy link
Collaborator

AWS CodeBuild CI Report

  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@ghost ghost merged commit bfc9c41 into aws:master Mar 9, 2020
@laurenyu laurenyu deleted the python-sdk-in-dlc branch March 9, 2020 18:05
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants