-
Notifications
You must be signed in to change notification settings - Fork 25
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
Update sagemaker docs #701
Conversation
The sagemaker page is currently not part of the index @GeorgesLorre, can you add it? |
Can you also remove the "Coming Soon" from the README? |
Can you also add the sagemaker runner to |
I also see that for the other runners we are documenting the |
02e7ec2
to
3495f75
Compare
docs/runners/sagemaker.md
Outdated
|
||
#### Assigning custom resources to the pipeline | ||
|
||
You can specify different kinds of `instance_type` while running your pipeline, you can see the available options [here](https://docs.aws.amazon.com/sagemaker/latest/dg/notebooks-available-instance-types.html). You may need to request a [quota increase](https://docs.aws.amazon.com/general/latest/gr/aws_service_limits.html) for certain `instance_type`'s. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this need to be done with the CLI, or do we support a Resources
instance for Sagemaker as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added it: #710
3495f75
to
fdcd768
Compare
No description provided.