search

developmentseed / asdi-examples

A place for notebook examples related to the Amazon Sustainability Data Initiative (ASDI)
2 stars 1 forks source link

Feat/amazonia 1 #12

Closed wildintellect closed 11 months ago

wildintellect commented 12 months ago

New notebook demonstrating how to query and access data from the Amazonia-1 WFI sensor with stackstac.

review-notebook-app[bot] commented 12 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

wildintellect commented 12 months ago

I realized I forgot to update the readme. Working on that. Also need to look into RGB adjustments with colormap for more True color rendering.

review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:17Z ----------------------------------------------------------------

Some commas and a little clarity in the second sections

---

With the AWS ASDI STAC API, we can search and load various geospatial datasets. Below, we'll work with the Amazonia-1 Satellite WFI Camera, Level-4 Orthorectified data sourced from Cloud Optimized GeoTIFFs (COGs) hosted on AWS. In this example, we'll be reading in 4-band imagery within a specified bounding box.

First, we need to install any necessary packages. Please note that the following code block only needs to be run if you're setting up a new workspace. You may need to restart the kernel to use updated packages. The code block only needs to be run once for each AWS Sagemaker Studio profile.


review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:18Z ----------------------------------------------------------------

performantly* or maybe change it to "efficiently"


wildintellect commented on 2023-09-26T22:59:34Z ----------------------------------------------------------------

Hmm, I don't think this example uses rio-tiler must be left over from the prior example.

review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:19Z ----------------------------------------------------------------

Remove import from individual cells since we're importing everything up above?


wildintellect commented on 2023-09-26T23:00:09Z ----------------------------------------------------------------

that is best practices, I'll do that.

review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:21Z ----------------------------------------------------------------

Title edit

---

Loading an OpenStreetMap (OSM) Descriptive Map of the Same Area


jjfrench commented on 2023-10-12T21:26:04Z ----------------------------------------------------------------

I think this is the last change I would make. The other conversations seem to be resolved.

wildintellect commented on 2023-10-12T22:20:31Z ----------------------------------------------------------------

Ah TitleCase instead of a sentence ... I might want to change the title entirely to something like Overview Map for Context what do you think of this title?

jjfrench commented on 2023-10-13T16:46:32Z ----------------------------------------------------------------

I like that 👍🏻

review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:22Z ----------------------------------------------------------------

Directly import of geometry or box?


wildintellect commented on 2023-10-11T23:46:28Z ----------------------------------------------------------------

if this wasn't a notebook I'd probably agree. going to leave as is for now. came directly from other examples online and more clearly shows the shapely library.

review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:23Z ----------------------------------------------------------------

resulted in -> returned

Let's*

lowercase "start"

lowercase "item" or maybe "STAC item"?


wildintellect commented on 2023-10-11T23:48:14Z ----------------------------------------------------------------

STAC-Item?

review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:24Z ----------------------------------------------------------------

Remove TODO comment


review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:25Z ----------------------------------------------------------------

Capitalize "Raster"

Loading DEM Raster into an Xarray DataArray


wildintellect commented on 2023-10-11T23:58:17Z ----------------------------------------------------------------

also it's not a DEM ...

review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:26Z ----------------------------------------------------------------

Let's*

right when it's needed -> as it's needed

It turns out that clouds are a significant issue, so we need to find a time without clouds. With the following code block, you will see that only 5 of the 15 times are good data.


review-notebook-app[bot] commented 11 months ago

View / edit / reply to this conversation on ReviewNB

jjfrench commented on 2023-09-26T16:02:27Z ----------------------------------------------------------------

lowercase "Bands"

For the Amazonia WFI sensor, you can see the labeled bands while looking at the asset properties on the STAC item.


wildintellect commented 11 months ago

Hmm, I don't think this example uses rio-tiler must be left over from the prior example.

View entire conversation on ReviewNB

wildintellect commented 11 months ago

that is best practices, I'll do that.

View entire conversation on ReviewNB

wildintellect commented 11 months ago

if this wasn't a notebook I'd probably agree. going to leave as is for now. came directly from other examples online and more clearly shows the shapely library.

View entire conversation on ReviewNB

wildintellect commented 11 months ago

STAC-Item?

View entire conversation on ReviewNB

wildintellect commented 11 months ago

also it's not a DEM ...

View entire conversation on ReviewNB

jjfrench commented 11 months ago

I think this is the last change I would make. The other conversations seem to be resolved.

View entire conversation on ReviewNB

wildintellect commented 11 months ago

Ah TitleCase instead of a sentence ... I might want to change the title entirely to something like Overview Map for Context what do you think of this title?

View entire conversation on ReviewNB

jjfrench commented 11 months ago

I like that 👍🏻

View entire conversation on ReviewNB

  • © Githubissues.
  • Githubissues is a development platform for aggregating issues.