Closed wildintellect closed 1 year ago
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
View / edit / reply to this conversation on ReviewNB
jjfrench commented on 2023-04-17T15:15:24Z ----------------------------------------------------------------
In addition to additional steps for handling large data volumes, we also need to handle the fact that data requesters need to pay for the Sentinel-1 imagery. To do this, we can use a context manager and the rasterio.Env
function to specify that we will pay for the data transfer cost.
"In addition to additional steps for handling large data volumes," -> "Additionally,"
You will need an AWS account and have a method to obtain security credentials with an ID and Key for use with that account. The s3 bucket is requester-pays, and as long as requests are within the same region, requesters only pay for GET requests, no cross region data transfer or egress costs. To discern if a dataset is requester-pays, consult the dataset page on AWS Open Data, like this description for example: https://registry.opendata.aws/sentinel-1/ In this example Sentinel 1 is in the eu-central-1 and SageMaker Studio Lab is currently in us-east-2, so this notebook should be used for limited testing.
"The s3 bucket is requester-pays" -> "The Sentinel-1 bucket is requester-pays"
"no cross region data transfer or..." -> "no cross-region data transfer or..."
View / edit / reply to this conversation on ReviewNB
jjfrench commented on 2023-04-17T15:15:26Z ----------------------------------------------------------------
Line #1. # Read a example GRD file's level 2 overviews
# Read an example GRD file's level 2 overviews
or
# Read a GRD file's level 2 overviews
Just a couple of minor text suggestions in the notebook, otherwise lgtm
Needed to make several changes to the notebook for compatibility with SageMaker Studio Lab #4
TODO: