Skip to content

Climate zones#66

Merged
Stephen Haddad (stevehadd) merged 13 commits into
mainfrom
climate_zones
Jul 9, 2026
Merged

Climate zones#66
Stephen Haddad (stevehadd) merged 13 commits into
mainfrom
climate_zones

Conversation

@stevehadd

Copy link
Copy Markdown
Collaborator

This pull request ports some notebooks from the AI4Climate hackathon repo in preparation for a tutorial. It contains

  • notebooks for data exploration, training, inference and evaluation
  • A readme
  • A data prep notebook

Could you please review the following:

  • check that the notebooks run on JupyterHub on spice
  • Check that you understand the explanatory text and that the instruction on how to run the notebooks are clear.
  • Any suggestion for improvement to the notebooks, including additional ideas for extenstion actvities or additional things to demo to improve the learning journey.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a “Climate Zones” tutorial as a set of Jupyter notebooks (EDA → training → inference/visualisation → evaluation) plus a data-prep notebook and a README to guide running the tutorial (intended for JupyterHub on Spice).

Changes:

  • Added tutorial README with a notebook “pathway” and environment setup notes.
  • Added 4 core tutorial notebooks (exploration, training, inference/visualisation, evaluation) and 1 supporting data-prep notebook.
  • Introduced a shared configuration dependency (config.json) across all notebooks.

Reviewed changes

Copilot reviewed 1 out of 6 changed files in this pull request and generated 21 comments.

Show a summary per file
File Description
ml_examples/climate_zones/README.md Tutorial entry point and run instructions; contains several typos and one broken link.
ml_examples/climate_zones/ClimateZones_DataExploration.ipynb EDA notebook; depends on missing config.json.
ml_examples/climate_zones/ClimateZones_TrainingPipeline.ipynb Training notebook; depends on missing config.json, contains a future-year copyright and template placeholder text.
ml_examples/climate_zones/ClimateZones_InferenceVisualisation.ipynb Inference/visualisation notebook; depends on missing config.json, contains template placeholder text.
ml_examples/climate_zones/ClimateZones_Evaluation.ipynb Evaluation notebook; depends on missing config.json.
ml_examples/climate_zones/ClimateZones_DataPrep.ipynb Data-prep notebook; depends on missing config.json and contains several concrete run-breaking/logic issues (path handling, subgroup mapping, multiprocessing, hard-coded paths).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ml_examples/climate_zones/README.md
Comment thread ml_examples/climate_zones/README.md
Comment thread ml_examples/climate_zones/README.md Outdated
Comment thread ml_examples/climate_zones/README.md Outdated
Comment thread ml_examples/climate_zones/README.md Outdated
Comment thread ml_examples/climate_zones/ClimateZones_DataPrep.ipynb
Comment thread ml_examples/climate_zones/ClimateZones_TrainingPipeline.ipynb
Comment thread ml_examples/climate_zones/ClimateZones_InferenceVisualisation.ipynb Outdated
Comment thread ml_examples/climate_zones/README.md Outdated
Comment thread ml_examples/climate_zones/ClimateZones_DataPrep.ipynb
@stevehadd

Copy link
Copy Markdown
Collaborator Author

mo-BrandonBirch could you approve this PR when yopu are happy so I can merge?

@mo-BrandonBirch mo-BrandonBirch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Changes made based on feedback provided, everything runs

@stevehadd Stephen Haddad (stevehadd) merged commit 3014f52 into main Jul 9, 2026
1 check passed
@stevehadd Stephen Haddad (stevehadd) deleted the climate_zones branch July 9, 2026 13:36
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.

3 participants