Skip to content

Conversation

@serreaaron
Copy link

Description

This notebook shows how to load in data for the Dust Deposition Story on VEDA notebooks.

What type of example is this?

Choose one of the following and delete the rest (read more)

Quickstart: Accessing the Data Directly
Quickstart: Using the Raster API
Tutorial
Dataset

Checklist

The following checklist ensures that our notebooks are internally consistent (read more)

  • The first cell contains the rendering information with author and data
  • The notebook has a Run this Notebook section (with filename in link)
  • The notebook has an Approach section
  • All the packages that are imported in the notebooks are used within the notebook
  • Any complex geometries are accessed from remote storage
  • Each code cell comes after a markdown cell containing explantory text
  • All cells in the notebook book have been run in order with a fresh kernel (cell numbers should be 1, 2, 3, 4, ...)
  • If the example type is Dataset the filename is <collection_id>.ipynb

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@smk0033
Copy link
Collaborator

smk0033 commented Dec 3, 2025

Is this good to add to Science Support's board for review? cc @acblackford

@acblackford
Copy link
Contributor

Yes, this is ready for review! @smk0033

@HarshiniGirish
Copy link

1.Under # Calling in packages and finding the chlorophyll-a dataset

The text in the following cell refers to a comparison “between April 13, 2025 and July 1, 2025,” but the two item IDs currently point to one date in May 2024 and one in April 2025.
Please either update the narrative to match these actual dates, or adjust the item IDs so they correspond to the dates mentioned in the text.

  1. Under # Creating the slider map

Fix the small typo in the comment header “Creatiing” to “Creating.”

Since this is a pure tile-based map, no legend is shown by default.
Consider adding a brief markdown sentence directly below the map noting that the color ramp represents chlorophyll concentration (e.g., “The color ramp indicates chlorophyll-a concentration, with higher values shown in brighter colors.”).

  1. Under # find all core files in the current directory

This cleanup cell uses both glob and os. While glob is imported earlier in the notebook, os is not imported anywhere.
To make this cell self-contained and robust, please ensure that both glob and os are explicitly imported in this cell (or at the top of the notebook), so it does not depend on any previous cells having been executed.

  1. Under the “Clean Up (Optional)” markdown heading

Add a clarifying line such as: “This step is only needed if the kernel has crashed and left behind core dump files (named like core.*); otherwise, you can safely skip it.”

apart from these minor changes the notebook looks good to go. Thank you @serreaaron

@acblackford
Copy link
Contributor

@HarshiniGirish comments have been addressed. Thank you for the review!

@HarshiniGirish
Copy link

Thank you @acblackford. Looks ready!

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.

5 participants