Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Replace s3://snowex-data from tutorials with zenodo dataset #160

Merged
merged 12 commits into from
Oct 21, 2021

Conversation

scottyhq
Copy link
Member

closes #15

@scottyhq scottyhq added the preview create netlify website preview label Sep 23, 2021
@github-actions
Copy link

Binder 👈 Launch a binder notebook on this branch

@github-actions
Copy link

github-actions bot commented Sep 23, 2021

@@ -2,7 +2,7 @@
"cells": [
{
"cell_type": "markdown",
"id": "minimal-cuisine",
"id": "urban-steal",
Copy link
Member

Choose a reason for hiding this comment

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

I wish that the ID's would not be renamed every time someone opens a notebook - maybe some day ....

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, this is actually due to a bug that was fixed for jlab>=3.0.13 (jupyterlab/jupyterlab#10018) https://github.com/jupyterlab/jupyterlab/releases/tag/v3.0.13

book/tutorials/sar/sentinel1.ipynb Outdated Show resolved Hide resolved
@jomey
Copy link
Member

jomey commented Oct 13, 2021

@scottyhq - Is there anything you want me review on this again?

@scottyhq
Copy link
Member Author

@scottyhq - Is there anything you want me review on this again?

Sorry, left this hanging, going to merge now and hopefully things still render...

@scottyhq scottyhq merged commit 59d4438 into snowex-hackweek:main Oct 21, 2021
@scottyhq scottyhq deleted the zenodo-data branch October 21, 2021 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview create netlify website preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Link repository to zenodo DOI
2 participants