Skip to content

bugfix: remove sagemaker upgrade line; pin to v1 range #1566

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

Merged
merged 1 commit into from
Sep 25, 2020

Conversation

aaronmarkham
Copy link
Contributor

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

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Contributor

@TEChopra1000 TEChopra1000 left a comment

Choose a reason for hiding this comment

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

Seeing this error when running:

---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-2-0f073dcf364f> in <module>
      8 
      9 # Specify training container
---> 10 from sagemaker.amazon.amazon_estimator import get_image_uri
     11 container = get_image_uri(boto3.Session().region_name, 'linear-learner')
     12 

ImportError: cannot import name 'get_image_uri'


From the python sdk update:

Image URI Functions (e.g. get_image_uri)
The following functions have been removed in favor of sagemaker.image_uris.retrieve():
sagemaker.amazon_estimator.get_image_uri()
...
For more information about usage, see sagemaker.image_uris.retrieve().

@aaronmarkham aaronmarkham changed the title bugfix: remove sagemaker upgrade line bugfix: remove sagemaker upgrade line; pin to v1 range Sep 25, 2020
@aaronmarkham aaronmarkham merged commit 09b92ef into aws:master Sep 25, 2020
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.

2 participants