Add ECCC HRDPS forecast quickstart notebook (review draft) - #54
Draft
aldenks wants to merge 3 commits into
Draft
Conversation
Review draft: carries six static question candidates and three animations so there is something to choose between, to be cut to two and one before publishing. Two draft notes mark the surplus and the recommended keeps. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>
aldenks
force-pushed
the
eccc-hrdps-forecast-notebook
branch
from
August 28, 2026 20:06
583da54 to
d4e16bb
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Quickstart for the new
eccc-hrdps-forecastdataset: ECCC's 2.5 km convection-permitting continental model, 6-hourly inits, hourly to 48 hours. Written against the draft "Writing quickstart notebooks" guidelines, which are held back for their own PR.Draft, not ready to merge: it carries surplus so there is something to choose between.
Not ready to merge
Six static question candidates and three animations, to be cut to two and one. Two draft notes (after the title, and after the community challenge) mark the surplus and must be deleted along with whatever is cut. Recommended keeps:
Each candidate is independent: delete its heading cell, its code cell, and any note cell that follows. An AST check confirms no cross-candidate name dependencies, so any subset runs.
Dataset notes worth knowing
y/xaregrid_latitude/grid_longitudein degrees, so there is no 1D lat/lon to.sel()on. Three helpers carry the notebook: a cartopyRotatedPoleCRS built fromspatial_ref, a nearest-cell search over the 2D coordinate arrays, and a lat/lon box slicer.ingested_forecast_lengthis all-NaT for every init time, and as a scalar coord it lands in every auto-generated plot title asNaT, which reads as broken software. The notebook drops it in the open cell; worth fixing upstream instead.-1and-1000source markers over a third of the domain at a given hour, varying with the weather; the structure note says to mask below-0.1.Events
All verified inside the archive window (2026-07-09 onward) and domain: the Bald Range fire near Summerland (7-8 Aug), Thunder Bay 36 in Wabakimi (12-14 Jul), the Prairie heat ridge and its two nocturnal MCSs (11-13 Jul), the ECCC Maritimes rainfall warning (23-24 Aug), and a Lake Superior lake breeze (13 Jul). Sources are named in prose rather than linked, since the 2026 warnings and fire dashboards are live-state pages.
Open question for review
Q5 (Maritimes rain) is the only section that checked the model against something external — ECCC warned 40-60 mm with locals to 100, the forecast gave a 44 mm median and 83 mm at the 90th percentile. The guidelines' "don't compute a number only so the prose can quote it" removed that comparison, so the section now shows an accumulation without saying whether it agrees. That may be right for a quickstart, but it puts "how good is this model" out of scope by construction, and is worth deciding deliberately.
Also flagged: the mm/h conversion on Q7's colorbar partially reverses a unit-conversion cut made on the HRRR notebook. The guidelines were narrowed to "convert only where the stored unit is one nobody reads in" —
kg m-2 s-1earns mm/h,m s-1does not. One-line revert if that lands differently.Runs end to end, 29 cells, 8.2 MB, no errors.
🤖 Generated with Claude Code