Skip to content
This repository has been archived by the owner on Apr 11, 2019. It is now read-only.

Initial commit of winds extremes nb #179

Merged
merged 6 commits into from
Nov 26, 2014
Merged

Initial commit of winds extremes nb #179

merged 6 commits into from
Nov 26, 2014

Conversation

Bobfrat
Copy link
Contributor

@Bobfrat Bobfrat commented Sep 22, 2014

@ocefpaf ocefpaf mentioned this pull request Nov 18, 2014
Closed
@Bobfrat
Copy link
Contributor Author

Bobfrat commented Nov 26, 2014

@birdage can you review/merge? It just ran no problem on my machine.

@ocefpaf
Copy link
Member

ocefpaf commented Nov 26, 2014

@Bobfrat this PR needs a to be rebased before merging. Are you familiar with that procedure?

If you find rebasing too cumbersome you can close this PR and re-submit this notebook in a new one.

@Bobfrat
Copy link
Contributor Author

Bobfrat commented Nov 26, 2014

@ocefpaf I rebased but still failing Travis CI. Is that expected? It's failing on cd test?

@ocefpaf
Copy link
Member

ocefpaf commented Nov 26, 2014

That is fine. We can merge it now. The test directory was moved to "housekeeping." I will prepare a PR to look for the test folder at new directory.

(BTW Travis CI would have failed anyways. There are a few notebooks that do not comply with the test in test_system-test_rules.py.)

Bobfrat added a commit that referenced this pull request Nov 26, 2014
@Bobfrat
Merge pull request #179 from Bobfrat/extreme_winds
f2755d0 
Initial commit of winds extremes nb
@Bobfrat Bobfrat merged commit f2755d0 into ioos:master Nov 26, 2014
@Bobfrat Bobfrat deleted the extreme_winds branch November 26, 2014 16:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
System Administration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Bobfrat @ocefpaf @jkupiec