Skip to content

doc: pytorch serving eia #52

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 6 commits into from
Closed

doc: pytorch serving eia #52

wants to merge 6 commits into from

Conversation

chuyang-deng
Copy link
Contributor

Issue #, if available:

Description of changes:
add pytorch serving eia related doc

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@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

README.rst Outdated
* For information on how Amazon Elastic Inference works, see `How EI Works <https://docs.aws.amazon.com/sagemaker/latest/dg/ei.html#ei-how-it-works>`__.
* For more information in regards to using Amazon Elastic Inference in SageMaker, see `Amazon SageMaker Elastic Inference <https://docs.aws.amazon.com/sagemaker/latest/dg/ei.html>`__.

Building the SageMaker Elastic Inference MXNet Serving container

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo "MXNet" => "PyTorch"

@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

@@ -25,6 +25,7 @@ Table of Contents

#. `Getting Started <#getting-started>`__
#. `Building your Image <#building-your-image>`__
#. `Amazon Elastic Inference with PyTorch in SageMaker <#elastic-inference>`__

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#elastic-inference doesn't work as a link. It should be #amazon-elastic-inference-with-pytorch-in-sagemaker


Building the SageMaker Elastic Inference PyTorch Serving container
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Amazon Elastic Inference is designed to be used with AWS enhanced versions of TensorFlow serving, Apache MXNet or PyTorch serving.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"PyTorch serving"?

`Amazon Elastic Inference <https://aws.amazon.com/machine-learning/elastic-inference/>`__ allows you to to attach
low-cost GPU-powered acceleration to Amazon EC2 and Amazon SageMaker instances to reduce the cost of running deep
learning inference by up to 75%. Currently, Amazon Elastic Inference supports TensorFlow, Apache MXNet, PyTorch,
and ONNX models, with more frameworks coming soon.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"with more frameworks coming soon"...if this was originally intended to refer to PyTorch, maybe it should be removed now.

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Amazon Elastic Inference is designed to be used with AWS enhanced versions of TensorFlow serving, Apache MXNet or PyTorch serving.
The SageMaker PyTorch containers with Amazon Elastic Inference support were built utilizing the
same instructions listed `above <https://github.com/aws/sagemaker-pytorch-serving-container#building-your-image>`__ with the

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Should this link just be above <#building-your-image> like in table of contents?

@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

@chuyang-deng chuyang-deng changed the title [DO NOT MERGE] doc: pytorch serving eia doc: pytorch serving eia Mar 10, 2020
@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

@chuyang-deng
Copy link
Contributor Author

closing for #55

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.

3 participants