fix advanced_functionality/causal-inference/causal-inference-containe…

…r.ipynb (aws#3544) * fix advanced_functionality/causal-inference/causal-inference-container.ipynb * fix login command * fix login * fix login * fix login Co-authored-by: EC2 Default User <[email protected]>
hchings · Aug 16, 2022 · aabd058 · aabd058
1 parent f355543
commit aabd058
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/advanced_functionality/causal-inference/causal-inference-container.ipynb b/advanced_functionality/causal-inference/causal-inference-container.ipynb
@@ -108,7 +108,7 @@
     "fi\n",
     "\n",
     "# Get the login command from ECR and execute it directly\n",
-    "aws ecr get-login-password --region ${region}|docker login --username AWS --password-stdin ${fullname}\n",
+    "$(aws ecr get-login --region $region --registry-ids $account --no-include-email)\n",
     "\n",
     "# Build the docker image locally with the image name and then push it to ECR\n",
     "# with the full name.\n",
@@ -176,6 +176,8 @@
    "metadata": {},
    "outputs": [],
    "source": [
+    "! mkdir data\n",
+    "\n",
     "# S3 bucket where the training data is located.\n",
     "data_bucket = f\"sagemaker-sample-files\"\n",
     "data_prefix = \"datasets/tabular/uci_heart_failure/\"\n",