Skip to content

Conversation

@robbibt
Copy link
Member

@robbibt robbibt commented Dec 10, 2025

  • Notebook updated to follow updated DEA Notebooks template format
  • For STAC loading:
    • odc-stac load() and configure_s3_access imported
    • pystac_client Client imported
    • Environment setup includes catalog, stac_client and configure_s3_access()
    • Cell that searched a collection for items with pystac-client
    • Cell that loads discovered items into an xarray with odc-stac
  • Notebook no longer requires datacube
  • Notebook no longer does sys.path.insert(1, '../Tools/')

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -4,27 +4,59 @@
"cell_type": "markdown",
Copy link
Collaborator

@jennaguffogg jennaguffogg Dec 10, 2025

Choose a reason for hiding this comment

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

Line #2.    import odc.aws

Well that's cool - I didn't know odc had an aws module.

Is it worth having links to the docs pages for these modules in the notebook? Not necessarily here in the load cell, but where they are used later down in the notebook.

AWS — Open Data Cube 1.9.12.dev19+g3f6a0c018 documentation


Reply via ReviewNB

@@ -4,27 +4,59 @@
"cell_type": "markdown",
Copy link
Collaborator

@jennaguffogg jennaguffogg Dec 10, 2025

Choose a reason for hiding this comment

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

Use blue call-out box from the template to make this really jump out to the user maybe?


Reply via ReviewNB

@@ -4,27 +4,59 @@
"cell_type": "markdown",
Copy link
Collaborator

@jennaguffogg jennaguffogg Dec 10, 2025

Choose a reason for hiding this comment

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

Unsure what is happening with the formatting here:

# Get list of all URLs for the 'nbart_blue' band asset = 'nbart_blue' urls = [i.assets["nbart_blue"].href for i in items]

Is the "get list" part meant to be a subtitle and then the code is meant to be inside `?


Reply via ReviewNB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants