-
Notifications
You must be signed in to change notification settings - Fork 20
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
[WIP] Notebook - general-preprocessing-rainfall_noaa #96
Conversation
✅ Deploy Preview for the-environmental-ds-book canceled.
|
@NHomer-Edi @timo0thy can you double check if the version in the Netlify preview is the latest one of the round 1? Just add thumbs-up if the preview is ok. Then, I'll add some suggestions for a 2nd round in the notebook repository. |
Looks all good to me, thanks! |
👍 |
@timo0thy @NHomer-Edi I've finished the post-print stage (i.e. styling, adding badges, minor changes) of the notebook. May I ask to double check the Netlify preview and inspect if the Binder and links of the the notebook work for you. Note, the Rohub is incomplete, but I hope to have it ready before the release of the notebook Monday next week. |
@acocac The Netflify preview and Binder work well for me - no issues to report. |
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.
Sorry for taking a day to get back to you, but it works perfect on both Netlify and Binder. Thanks!
Before review
Description
Add notebook in preprocessing NOAA timeseries for rainfall analysis at global scale and Borneo (see notebook idea in #53).
Motivation and Context
New notebook for the preprocessing theme.
How has this been tested?
GitHub actions deployed first in the forked templated by the main author, @timo0thy, then forked repo was transferred to the EnvDS book organisation (see https://github.com/Environmental-DS-Book/general-preprocessing-rainfall_noaa).
Types of changes
Checklist:
Additional information