Skip to content

Conversation

andypbarrett
Copy link
Collaborator

Created notebook for icesat2 applications demo

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

github-actions bot commented Aug 5, 2025

Binder 👈 Launch a binder notebook on this branch for commit 6dcb645

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 16b4685

Binder 👈 Launch a binder notebook on this branch for commit 3821acd

@mikala-nsidc
Copy link
Collaborator

everything ran well for me, except the search by RGT cell. I got the following error:

IndexError Traceback (most recent call last)
Cell In[16], line 1
----> 1 g = [g for g in granules if g["umm"]['GranuleUR'].split('_')[2].startswith('0531')]
2 g
IndexError: list index out of range

not sure why, because in the list it looked like that granule was at index 2 in my list.
Another thing I noticed is that i think i'm still seeing ECS data with my own login credentials. but other users wouldn't see that.
Good to merge in my opinion!
Thanks for doing this! I know it always ends up taking longer than expected. It will be a great resource for users and much appreciated!

Copy link

review-notebook-app bot commented Aug 6, 2025

View / edit / reply to this conversation on ReviewNB

betolink commented on 2025-08-06T13:10:48Z
----------------------------------------------------------------

I'd say this would be filtering not searching. We can search for aa particular RGT using the granule UR like in this notebook https://github.com/ICESAT-2HackWeek/h5cloud/blob/main/notebooks/01_data-selection.ipynb


@betolink betolink merged commit e4d47cc into main Aug 6, 2025
3 of 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