-
Notifications
You must be signed in to change notification settings - Fork 10
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
Support for Google Cloud Storage URLs (like WeatherBench2 data) #51
base: dev
Are you sure you want to change the base?
Conversation
Thanks for you PR, I'm a bit worry about adding another dependency. Could we make |
Yes! P.S. I realized I can use the public HTTPS URL, which is already supported: https://storage.googleapis.com/weatherbench2/datasets/era5/1959-2023_01_10-wb13-6h-1440x721_with_derived_variables.zarr/ |
Co-authored-by: Vincent Sarago <[email protected]>
@vincentsarago I kind of need this for a project. Anything I can do to get this merged? |
@thijsvanwinden the branch is outated so we would need to update it and also make sure the test/lint passes I also feel we should have tests added Sadly I do not have bandwidth to take care of this right now |
What I am changing
gs://weatherbench2/datasets/era5/1959-2023_01_10-wb13-6h-1440x721_with_derived_variables.zarr
(see https://weatherbench2.readthedocs.io/en/latest/data-guide.html)How I did it
gs://...
inreader.py
How you can test it
gs://weatherbench2/datasets/era5/1959-2023_01_10-wb13-6h-1440x721_with_derived_variables.zarr
) and choosing a variable with atime
dimension from https://weatherbench2.readthedocs.io/en/latest/data-guide.html2m_temperature
data with http://127.0.0.1:8000/map?url=gs%3A%2F%2Fweatherbench2%2Fdatasets%2Fera5%2F1959-2023_01_10-wb13-6h-1440x721_with_derived_variables.zarr&variable=2m_temperature&consolidated=true&rescale=208%2C310&colormap_name=coolwarm should produceRelated Issues