fix: Add NOTICE to MANIFEST

BastianZim commented 2 years ago

Issue #, if available:

Description of changes:

The license requires the notice to be distributed as well. This is done with this PR.

Testing done:

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[ ] I have used the regional endpoint when creating S3 and/or STS clients (if appropriate)
[ ] I have updated any necessary documentation, including READMEs

Tests

[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have checked that my tests are not configured for a specific region or account (if appropriate)

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

sagemaker-bot commented 2 years ago

AWS CodeBuild CI Report

CodeBuild project: sagemaker-inference-toolkit-pr
Commit ID: d5a884f955f621075733fe42bbc9848505783643
Result: FAILED
Build Logs (available for 30 days)

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

BastianZim commented 2 years ago

The error doesn't seem related to this?

BastianZim commented 2 years ago

Gentle ping here @ajaykarpur as you also merged some other PRs.

sagemaker-bot commented 2 years ago

AWS CodeBuild CI Report

CodeBuild project: sagemaker-inference-toolkit-pr
Commit ID: 78af17f8b83a571605c67034bece467f922a5973
Result: FAILED
Build Logs (available for 30 days)

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

BastianZim commented 2 years ago

Failure is due to Step 1/20 : FROM awsdeeplearningteam/mxnet-model-server:base-cpu-py3.6 toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit Should be unrelated.

BastianZim commented 2 years ago

Gentle ping here @maaquib

BastianZim commented 2 years ago

Thanks, @nskool but it's still failing. Unfortunately, I cannot see the new logs so no idea what's causing this.

sagemaker-bot commented 2 years ago

AWS CodeBuild CI Report

CodeBuild project: sagemaker-inference-toolkit-pr
Commit ID: 14ff2be6261e323014ff1276f59fb0ac358e13b1
Result: FAILED
Build Logs (available for 30 days)

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

sagemaker-bot commented 2 years ago

AWS CodeBuild CI Report

CodeBuild project: sagemaker-inference-toolkit-pr
Commit ID: 9c5a993142e7a038de3af4b57ddb351dd65f019b
Result: SUCCEEDED
Build Logs (available for 30 days)

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

BastianZim commented 2 years ago

Succeeded finally

BastianZim commented 2 years ago

@nskool Would you mind merging this?

sagemaker-bot commented 2 years ago

AWS CodeBuild CI Report

CodeBuild project: sagemaker-inference-toolkit-pr
Commit ID: 3216c250b2e64a25ba6206e6044ba171bd088ee3
Result: SUCCEEDED
Build Logs (available for 30 days)

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

aws / sagemaker-inference-toolkit