rapidsai / deployment

RAPIDS Deployment Documentation
https://docs.rapids.ai/deployment/stable/
9 stars 28 forks source link

simplify SageMaker docs #408

Closed jameslamb closed 1 month ago

jameslamb commented 1 month ago

Contributes to #402.

Contributes to #382 (by removing an image with a reference to the no-longer-supported rapidsai/core Docker image).

The main purpose of this PR is to reduce duplication in the SageMaker example at https://docs.rapids.ai/deployment/nightly/cloud/aws/sagemaker/. If you read that page from top to bottom, today it feels like 2 very-similar pages that were concatenated together. Details like starting a SageMaker Notebook instance and setting up a lifecycle configuration are repeated.

It also contains an incomplete example of how to use SageMaker Estimators. This proposes replacing that and instead encouraging the use of https://docs.rapids.ai/deployment/nightly/examples/rapids-sagemaker-higgs/notebook/, which contains an end-to-end complete example of using SageMaker Estimators.

review-notebook-app[bot] commented 1 month ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB