Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Is2 cryo2 #87

Merged
merged 3 commits into from
Oct 8, 2024
Merged

Is2 cryo2 #87

merged 3 commits into from
Oct 8, 2024

Conversation

mikala-nsidc
Copy link
Collaborator

I removed the non-coincident ICESat2-CryoSat2 notebook from the repo and also updated ICESat2-CryoSat2-Coincident.ipynb by removing the last code cell that was hanging when trying to plot too many points.
@asteiker @betolink ready for review if you have a chance.

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 Oct 4, 2024

Binder 👈 Launch a binder notebook on this branch for commit fe6d369

I will automatically update this comment whenever this PR is modified

Binder 👈 Launch a binder notebook on this branch for commit 068b81a

@mikala-nsidc
Copy link
Collaborator Author

forgot to clear cell output earlier before pushing, so did that just now.

@asteiker asteiker self-requested a review October 8, 2024 01:39
Copy link
Member

@asteiker asteiker left a comment

Choose a reason for hiding this comment

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

Thank you for these updates! I noticed we also need to update the REadme link to point to the coincident file but i'll do this after this merge.

@asteiker asteiker merged commit 1971a97 into main Oct 8, 2024
4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants