Skip to content
This repository was archived by the owner on Jun 15, 2023. It is now read-only.

Add DeepAR ap-northeast-3 information. #219

Merged
merged 3 commits into from
Oct 14, 2022
Merged

Conversation

jaheba
Copy link

@jaheba jaheba commented May 30, 2022

Issue #, if available:

Description of changes: Adding missing repository information for DeepAR algorithm in ap-northeast-3.

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

@jaheba
Copy link
Author

jaheba commented May 30, 2022

PR for SageMaker-SDK: aws/sagemaker-python-sdk#3120


| Registry path | Version | Job types \(image scope\) |
| --- | --- | --- |
| 867004704886\.dkr\.ecr\.ap\-northeast\-2\.amazonaws\.com/forecasting\-deepar:<tag> | 1 | inference, training |

Choose a reason for hiding this comment

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

Should this be ap-northeast-3 (not 2)?

Copy link
Author

Choose a reason for hiding this comment

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

Thanks! Updated the PR.

@aws-rbs
Copy link
Contributor

aws-rbs commented Jun 2, 2022

Talked to @julkroll about this update.
She has a script that updates this file from the content in https://github.com/aws/sagemaker-python-sdk/blob/master/src/sagemaker/image_uri_config/forecasting-deepar.json
So we do not manually update this information in the Developer Guide.
Then the script-updated content in the Developer Guide gets pushed to the GitHub version.

TBD: We confirm that this image is actually supported by SageMaker
jgasthaus can you confirm that this is the case?

@easyj2j easyj2j assigned easyj2j and unassigned aws-rbs Jul 5, 2022
@judyheflin judyheflin merged commit c52c416 into awsdocs:master Oct 14, 2022
@judyheflin
Copy link
Contributor

Thanks for bringing this to our attention! This image is supported and is public on the SageMaker developer guide.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants