aws-samples / aws-panorama-samples

This repository has samples that demonstrate various aspects of AWS Panorama device and the Panorama SDK
https://docs.aws.amazon.com/panorama/
MIT No Attribution
82 stars 58 forks source link

car_tracker_tutorial.ipynb failed with S3 404 error due to the wrong path `--model-s3-uri` of `panorama-cli add-raw-model` #87

Closed hightensan closed 1 year ago

hightensan commented 1 year ago

In the car_tracker_tutorial.ipynb sample notebook, the below cell is failed with S3 404 error.

!cd ./car_tracker_app && panorama-cli add-raw-model \
    --model-asset-name {model_asset_name} \
    --model-s3-uri s3://{S3_BUCKET}/{app_name}/{ML_MODEL_FNAME}.tar.gz \
    --descriptor-path {model_descriptor_path}  \
    --packages-path {model_package_path}

The cause is --model-s3-uri is the wrong path, which does not contain {model_node_name}.

SageMaker Com pilation job's output is specified as s3://{S3_BUCKET}/{app_name}/{model_node_name}/, so the path should contain /{model_node_name}/ prefix.

$ aws sagemaker describe-compilation-job --compilation-job-name Comp-Job20230714012911AM --region us-east-1 | jq '.OutputConfig'
{
  "S3OutputLocation": "s3://test-aws-panorama-samples-bucket-us-east-1/car_tracker_app/model_node/",
  "TargetPlatform": {
    "Os": "LINUX",
    "Arch": "X86_64"
  }
}

Screenshot 2023-07-14 at 11 02 18

shimomut commented 1 year ago

Thank you for your issue report, @hightensan , I will confirm and get back here.

shimomut commented 1 year ago

Merged the PR. Thank you!