-
Notifications
You must be signed in to change notification settings - Fork 9
Add science use case comparing AORC and CONUS404 #13
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
base: main
Are you sure you want to change the base?
Conversation
… be consistent with other notebooks.
…d CONUS404 datasets
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @igarousi! This is such a nice notebook. I made a few minor edits that I pushed to this PR branch. Tony mentioned that I may want to steer away from abstracting too much code inside functions so I focused more on readability versus modularity for this initial review.
I do think adding an animated GIF of the rainfall over a time period with something like GeoGIF (https://github.com/gjoseph92/geogif; this package supports GIF rendering from xarray and dask array objects) could be helpful and would be something nice to include as a graphical abstract and/or a visual to add to the newsletter. I'll bring this up in our compute team meeting to see if this is something we might want to add.
…ategory and degree from the Zarr format on HydroShare
This PR includes a first draft of a science use case exploring two high-resolution meteorological forcing datasets: the USGS/NCAR CONUS404 reanalysis and NOAA’s Analysis of Record for Calibration (AORC).