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

Uavsar tutorial sections #63

Merged
merged 61 commits into from
Jul 1, 2022

Conversation

ZachHoppinen
Copy link
Member

Split tutorial into sections

"metadata": {
"tags": [
"nbsphinx-gallery",
"nbsphinx-thumbnail"
]
},
"metadata": {},
Copy link
Member

Choose a reason for hiding this comment

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

not sure why the metadata changed in all the database notebooks

Copy link
Member Author

Choose a reason for hiding this comment

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

I think when I was cleaning notebooks I ran the clean *.ipynb command in the wrong directory. I reverted all these files back to the commit before I did that.

@scottyhq
Copy link
Member

Great! I'd prefer not to add the data folder here since it's 50+MB. I made a copy of those files in /home/jovyan/shared/uavsar-tutorial-data, but you could also create a separate repository (like snowex/uavsar-tutorial-data) and add them there (then the first step in your tutorial could be cd /tmp; git clone https://github.com/snowex/uavsar-tutorial-data.git

@ZachHoppinen
Copy link
Member Author

Hey @scottyhq created that github repo and am cloning the data to tmp now.

@scottyhq
Copy link
Member

@scottyhq
Copy link
Member

this looks great! Happy to merge whenever @ZachKeskinen

adding %config InlineBackend.figure_format='retina' to your imports generally makes the tick labels and titles look better on most modern screens

@ZachHoppinen
Copy link
Member Author

Hey @scottyhq can we pull this in?

Copy link
Member

@scottyhq scottyhq left a comment

Choose a reason for hiding this comment

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

thanks @ZachKeskinen !

@scottyhq scottyhq merged commit fcd1aa4 into snowex-hackweek:main Jul 1, 2022
@ZachHoppinen ZachHoppinen deleted the uavsar_tutorial_sections branch July 1, 2022 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
preview Trigger a preview deploy to a test instance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants