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

[WIP] Notebook - general-preprocessing-rainfall_noaa #96

Merged
merged 9 commits into from
Jul 16, 2022

Conversation

acocac
Copy link
Member

@acocac acocac commented Jun 14, 2022

Before review

  • Work in progress
  • Ready for review
  • Need help!

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • New notebook and/or content
  • Documentation notebook/content update
  • Other (please describe):
  • Feature change (upgrade version)

Checklist:

  • My code follows the code style of this project: Python
  • I have read the CONTRIBUTING doc
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests to cover my changes
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage)

Additional information

@netlify
Copy link

netlify bot commented Jun 14, 2022

Deploy Preview for the-environmental-ds-book canceled.

Name Link
🔨 Latest commit f201b2a
🔍 Latest deploy log https://app.netlify.com/sites/the-environmental-ds-book/deploys/62d2d44ddb47b300088b2361

@acocac
Copy link
Member Author

acocac commented Jun 22, 2022

@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.

@timo0thy
Copy link
Collaborator

Looks all good to me, thanks!

@NHomer-Edi
Copy link
Collaborator

👍

@acocac
Copy link
Member Author

acocac commented Jul 13, 2022

@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.

@NHomer-Edi
Copy link
Collaborator

@acocac The Netflify preview and Binder work well for me - no issues to report.

Copy link
Collaborator

@timo0thy timo0thy left a 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!

@acocac acocac merged commit 485851d into master Jul 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority notebook Add/update notebook preprocessing Preprocessing Notebooks ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[NBI] Concatenating a gridded rainfall dataset into a time series
3 participants