C2SM / iconarray

R&D Python package for processing ICON data with xarray
https://c2sm.github.io/iconarray/
BSD 3-Clause "New" or "Revised" License
10 stars 2 forks source link

open_dataset improvements #55

Closed victoria-cherkas closed 1 year ago

victoria-cherkas commented 1 year ago
victoria-cherkas commented 1 year ago

launch jenkins

victoria-cherkas commented 1 year ago

launch jenkins

victoria-cherkas commented 1 year ago

launch jenkins

victoria-cherkas commented 1 year ago

https://github.com/C2SM/icon-vis/pull/36 PR for icon-vis using this branch of iconarray.

https://jenkins-mch.cscs.ch/job/IconVis/job/Iconvis_PR/10/

victoria-cherkas commented 1 year ago

for this file grib= '/scratch/vcherkas/iconarray/data/example_data/grib/lfff00010000_edgeplots':

Elapsed time GRIB (unfiltered): ds = iconarray.open_dataset(grib) New open_dataset(): 0:02:13 Old open_dataset(): 0:02:39

So not to worry about performance loss, it is clearly just a large file (that should probably have been cropped before uploading to the example data server).

clairemerker commented 1 year ago

Great! I think we should make it a task to reduce the test data size. We can for example crop the data to a smaller domain

victoria-cherkas commented 1 year ago

Great! I think we should make it a task to reduce the test data size. We can for example crop the data to a smaller domain

https://github.com/C2SM/iconarray/issues/57 Just added issue for it :)

victoria-cherkas commented 1 year ago

Ok thanks for the review @clairemerker we can wait for additional test, thanks for that

clairemerker commented 1 year ago

launch jenkins

clairemerker commented 1 year ago

launch jenkins

clairemerker commented 1 year ago

@victoria-cherkas did you already test this PR with icon-vis?

victoria-cherkas commented 1 year ago

@victoria-cherkas did you already test this PR with icon-vis?

Yes its passing on icon-vis :)