Skip to content
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

Asynchronous fit #4

Closed
defreez opened this issue Dec 4, 2017 · 4 comments · Fixed by #59
Closed

Asynchronous fit #4

defreez opened this issue Dec 4, 2017 · 4 comments · Fixed by #59

Comments

@defreez
Copy link

defreez commented Dec 4, 2017

My first use for SageMaker involved async submission of KMeans jobs. EstimatorBase.fit raises NotImplemented('Asynchronous fit not available') if the wait parameter is set to false.

Is fit() necessarily synchronous, or is it just not implemented yet?

@winstonaws
Copy link
Contributor

Hey, thanks for trying out SageMaker!

We're going to support async fit, it just didn't make the cut for launch. This is on our backlog, and we'll give you an update when we have an ETA.

@defreez
Copy link
Author

defreez commented Dec 5, 2017

That's great to hear. Thanks!

@iquintero
Copy link
Contributor

This is already implemented in master. However its not released to pypi yet. I will reopen this issue to track it and close it once this feature lands in pypi. Anyone interested in using it now can install the sdk from source.

@iquintero iquintero reopened this Feb 1, 2018
@iquintero
Copy link
Contributor

This was just released in v1.1.0 to PyPI

laurenyu pushed a commit to laurenyu/sagemaker-python-sdk that referenced this issue May 31, 2018
Add integ tests for pytorch prediction.
apacker pushed a commit to apacker/sagemaker-python-sdk that referenced this issue Nov 15, 2018
athewsey pushed a commit to athewsey/sagemaker-python-sdk that referenced this issue May 19, 2021
Swap s3_prefix for code_location and separate code/entrypoint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants