-
Notifications
You must be signed in to change notification settings - Fork 70
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
Convert tutorial to use intake #502
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Do we need to keep the deprecated |
I don't think so. Happy for you to make the executive decision to delete. |
I'm so happy to delete deprecated stuff. It brings a sensation of liberation. |
Yep I’m all for deleting too! I think we now have intake working with all the mom6 configs and most of the cj50 exps are in intake now too. |
Possibly the only useful thing remaining associated with the cookbook is the explorer tools that I don’t think we have a replacement for yet with intake. |
@AndyHoggANU this looks great! I tweaked the language a bit and added |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
Merging - @charles-turner-1 don't worry about the review, but feel free to check it out on Monday. |
This PR updates the COSIMA Cookbook Beginner Tutorial and addresses #464.
The tutorial is now intake-ified and cleaned up a little, but other suggestions very welcome.
I've also retained the old tutorial under a different name, but maybe we should delete it altogether (as we really don't want new users adopting the old cookbook).