Skip to content
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

CRU-TS 4.06 #220

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

larsbuntemeyer
Copy link

@larsbuntemeyer larsbuntemeyer commented Nov 14, 2022

CRU-TS 4.06 high resolution gridded dataset (1910-2021). It's blocked by

...but works with v0.9.0. Requires os.environ['CEDA_USERNAME'] and os.environ['CEDA_PASSWORD'] to be set!

@larsbuntemeyer larsbuntemeyer changed the title CRU-TS 4.05 CRU-TS 4.06 Nov 15, 2022
@larsbuntemeyer
Copy link
Author

larsbuntemeyer commented Nov 15, 2022

I am not sure, if the encoding works correct. For example, data that was originally integer (like stn which is number of stations) results in float64. float32 data that has frequency units results in float64, too. Original units of ground frost frequency frs was days, now it's nanoseconds:

grafik

@cisaacstern
Copy link
Member

@larsbuntemeyer apologies for the lag time in engaging on this PR. Looks like the question you have above relates to internals of zarr encoding, and may require some upstream investigation in zarr and/or xarray.

For the moment, I'll see if we're able to get to the same place you were in your notebook, by running a test of this recipe on Pangeo Forge Cloud.

@cisaacstern
Copy link
Member

/run cru-ts-4.06

@larsbuntemeyer
Copy link
Author

@cisaacstern Thanks for looking into this!

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.

2 participants