Skip to content

ECDI notebooks#578

Merged
mickwelli merged 7 commits intomainfrom
drought-index
May 27, 2025
Merged

ECDI notebooks#578
mickwelli merged 7 commits intomainfrom
drought-index

Conversation

@vikineema
Copy link
Collaborator

Proposed changes

  • Add the Enhanced Combined Drought Index notebooks

Checklist (replace [ ] with [x] to check off)

  • Remove any unused Python packages from Load packages
  • Remove any unused/empty code cells
  • Remove any guidance cells (e.g. General advice)
  • Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended)
  • Include relevant tags in the first notebook cell and re-use tags if possible
  • Ensure appropriate colour schemes have been used to maximise accessibility for vision impairment. Test your images or learn more with Coblis or TPGI
  • Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated

Closes issues (optional)

  • Closes Issue #000

@vikineema vikineema marked this pull request as ready for review April 9, 2025 10:46
Copy link
Contributor

@mickwelli mickwelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @vikineema for the great notebooks and @nanaboamah89 for the markdown. I've used Amhara, a regional state in Ethiopia. The Landsat notebooks run in 20-30 mins, which I think is acceptable. It gives users a framework to apply to larger areas.

@mickwelli mickwelli merged commit 8cf1d23 into main May 27, 2025
1 check passed
@mickwelli mickwelli deleted the drought-index branch September 23, 2025 09:16
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