xarray-contrib / xwrf

A lightweight interface for working with the Weather Research and Forecasting (WRF) model output in Xarray.
https://xwrf.readthedocs.io/
Apache License 2.0
56 stars 16 forks source link

Update of tutorials for v0.0.2 #89

Closed lpilz closed 1 year ago

lpilz commented 1 year ago

Change Summary

Added a tutorial for using xgcm with dask-data.

Related issue number

Closes #69

Checklist

lpilz commented 1 year ago

I'm not sure if this is ready to deploy, as the last cell in the tutorial errored with a KeyboardInterrupt. @andersy005 should we just let it run again or merge and pray? :D

andersy005 commented 1 year ago

The latest build appears to have succeeded (the past failures may have been a glitch). However, the plots don't get updated when the widget value is updated.

Screen Shot 2022-09-08 at 3 21 42 PM
lpilz commented 1 year ago

Thanks! Do you know how to debug this? Could it be just due to insufficient compute power of the server?

lpilz commented 1 year ago

I noticed, that the widget also doesn't update for the last blog post, so it's a real bug...

jthielen commented 1 year ago

Thanks! Do you know how to debug this? Could it be just due to insufficient compute power of the server?

I noticed, that the widget also doesn't update for the last blog post, so it's a real bug...

I unfortunatley don't have any insight into debugging this either...so, in the interest of streamlining getting this in, would you be okay with creating a static (rather than interactive) output from hvplot?

lpilz commented 1 year ago

I'll overhaul the tutorial PR once #93 is merged...

lpilz commented 1 year ago

Yes, I'm having the same problem. At least for the commit 88958fa, I don't get updated readthedocs either.

lpilz commented 1 year ago

For 0d12e00, it works for me though.

lpilz commented 1 year ago

Unfortunately, for some reason _wind_speed seems to drop its lat/long coordinates during the vertical interpolation, which it didn't do before... Any clue what's going on there?

lpilz commented 1 year ago

Nevermind, it's a DataArray-issue again. I'm reverting to just using the whole Dataset....

lpilz commented 1 year ago

Ready for review again :)

lpilz commented 1 year ago

Yes, I guess it is. Afaik in the commit before that one, everything worked well.

lpilz commented 1 year ago

BTW: I think this is the last PR required for v0.0.2, so we're just in time with that one! :)