Skip to content

docs: show funder logos near top of README#121

Merged
peetmate merged 11 commits into
mainfrom
docs/readme-logos-top
Jul 15, 2026
Merged

docs: show funder logos near top of README#121
peetmate merged 11 commits into
mainfrom
docs/readme-logos-top

Conversation

@peetmate

@peetmate peetmate commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Adds the McKnight Foundation and CGIAR Climate Action logos with a one-line funding credit directly under the README intro, linking to the full Funding & Acknowledgements section at the bottom. Follow-up to #116.

🤖 Generated with Claude Code

peetmate and others added 4 commits July 9, 2026 11:39
McKnight and CGIAR Climate Action logos with a one-line funding credit
right under the intro, linking down to the full acknowledgements.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@peetmate peetmate enabled auto-merge (squash) July 9, 2026 18:54
Rwema25 and others added 5 commits July 10, 2026 09:30
Companion to the use-as-a-package guide: runs the credential-free
examples (NASA POWER, GHCN stations) end-to-end in the browser with
zero install. Earth Engine section gated behind an opt-in flag.
Badge links added to the guide and README.
Surface the companion notebook where new users land, not just in
the package guide.
Adds credential-free station-vs-grid validation (precipitation only:
the Nairobi GHCN station fails the completeness guard on temperature)
and expands the Earth Engine section: AgERA5 fetch, maize seasonal
statistics, hazards, focal-vs-baseline comparison, source comparison,
and NEX-GDDP 2050 projections. Colab Secrets tip for the project id.

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

Nice — the Colab on-ramp is a great addition, and it goes well beyond "funder logos" (the title undersells it). I checked it against the code:

  • Logo images (docs/assets/logos/mcknight.jpg, cgiar-climate-action.png) already exist in the repo, so the README renders fine. ✅
  • API usage is accurate: validation["confidence_summary"] and stats["ltm_season_summary"] are real return keys; the station cells work now that the GHCN verbose fix (#124) is merged; and the notebook wisely restricts compare_station_to_grids to precipitation (the nearest Nairobi station is too sparse on temperature for the completeness guard). ✅
  • The Earth Engine section is correctly gated behind RUN_EARTH_ENGINE, so a fresh Colab run stays credential-free through sections 1–4. ✅

One thing worth a maintainer decision (not blocking): the new examples/climate_toolkit_colab.ipynb overlaps examples/package_demo.ipynb (added in #124) — we’d have two credential-free example notebooks. Yours is the more comprehensive, Colab-hosted, all-seven-functions walkthrough; package_demo.ipynb is a local demo. Might be worth making the Colab one canonical and dropping/repointing package_demo.ipynb, to avoid two demos drifting apart.

Minor: I didn’t execute the notebook end-to-end (the EE cells are gated off), but the credential-free calls match usage I’ve verified works on main.

@Sammyjoseph999 Sammyjoseph999 requested a review from bjyberg July 13, 2026 12:12
@peetmate peetmate merged commit c6e30e5 into main Jul 15, 2026
2 checks passed
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.

4 participants