Skip to content

WIP: first pass at notebooks/geoclaw/dtopotools_examples.ipynb#38

Merged
rjleveque merged 11 commits into
clawpack:masterfrom
rjleveque:dtopotools_notebook
Sep 1, 2014
Merged

WIP: first pass at notebooks/geoclaw/dtopotools_examples.ipynb#38
rjleveque merged 11 commits into
clawpack:masterfrom
rjleveque:dtopotools_notebook

Conversation

@rjleveque
Copy link
Copy Markdown
Member

@mandli
Copy link
Copy Markdown
Member

mandli commented Jul 23, 2014

This looks great! Do you think we should do something similar for the topotools? Also, should we turn these examples into tests?

@rjleveque
Copy link
Copy Markdown
Member Author

Yes, I was planning to do something similar for topotools, or feel free to work on it! More dtopo tests coming, based in part on this.

@mandli
Copy link
Copy Markdown
Member

mandli commented Jul 23, 2014

There's the more general GeoClaw app-setup notebook I started which I have not touched in a bit. Since one of the more interesting aspects of that was the setup of topography we could probably steal from that. The relevant code is here.

@rjleveque
Copy link
Copy Markdown
Member Author

The dtopotools notebook has been improved and I added a topotools notebook, visible at http://nbviewer.ipython.org/url/faculty.washington.edu/rjl/notebooks/topotools_examples.ipynb

@mandli
Copy link
Copy Markdown
Member

mandli commented Aug 14, 2014

Just a note, this depends on clawpack/visclaw#114 to work (last couple of cells will not work otherwise). We should merge that in sooner rather than later probably.

@rjleveque
Copy link
Copy Markdown
Member Author

clawpack/visclaw#114 has been merged.

I added a topotools notebook. nbviewer views are now linked from the documentation pages found at
http://clawpack.github.io/doc/topotools.html

rjleveque added a commit that referenced this pull request Sep 1, 2014
Merging  notebooks/geoclaw/dtopotools_examples.ipynb and also topotools_examples.ipynb and Okada.ipynb
@rjleveque rjleveque merged commit 86afdd8 into clawpack:master Sep 1, 2014
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.

2 participants