Skip to content

Conversation

@mikala-nsidc
Copy link
Collaborator

@frizatch would you mind taking a quick look?

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions
Copy link

github-actions bot commented Jul 16, 2025

Binder 👈 Launch a binder notebook on this branch for commit 69e36d0

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 35d41b5

Binder 👈 Launch a binder notebook on this branch for commit 7b5f4a1

…ECS. Edited readme and notebook with note that this was for a workshop and would not be further updated
Copy link
Collaborator

@frizatch frizatch left a comment

Choose a reason for hiding this comment

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

This looks good! I would just add a dash in "on premises" and remove the ".html" from the second link.

"nbconvert_exporter": "python",
"pygments_lexer": "ipython3",
"version": "3.10.14"
"version": "3.9.18"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This one actually downgraded from 3.10.14 to 3.9.18? Others went from 3.9.16 to 3.9.18 (?)

@@ -0,0 +1,558 @@
#!/usr/bin/env python
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm assuming this is a copy and paste of a functional script to create a new file so all is well!

"metadata": {},
"source": [
"Then we use a spatial filter to search for ATL10 granules that intersect our area of interest. This is the same area we used in our cs2eo query above."
"Locate the ATL10 urls in the ATL10 download script provided by cs2eo (ATL10_V6_download_script.py). Copy the cloudfront urls from the script into a list, as below. "
Copy link
Collaborator

Choose a reason for hiding this comment

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

This shift looks to cloudfront URLs seems fine, but I'm not being exacting.

@mikala-nsidc mikala-nsidc merged commit c133812 into main Jul 17, 2025
4 checks passed
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