rigoudyg / climaf

CliMAF - a Climate Model Analysis Framework - doc at : http://climaf.readthedocs.org/
Other
18 stars 7 forks source link

Automatically check that the period is covered when using ds() #165

Open jservonnat opened 4 years ago

jservonnat commented 4 years ago

I will come back later on this issue myself, I just needed to write it somewhere it would not be forgotten. A user had the case of a missing decade in his archive, and ds() did not mention anything. We tried .check() but we had an error that was not related with the hole in the period. And .explore('resolve') did not return any error neither.

senesis commented 4 years ago

I also noticed that check() has bugs. I agree this topic is instrumental Nevertheless, I advocate for upward compatibility : check should be explicitly asked for; we could maybe let the user choose when he wants to activate it, e.g. by a toggle command which would act on all next ds() commands

Did you play on your case with explore() and period=*, such as in this doc example ? `

Let us use a kind of dataset which data files are temporally splitted,

and allow for various models, and use a wildcard for period

so=ds(project="CMIP6", model='CNRM', experiment="piControl", realization="r1i1p1f2", table="Omon", variable="so", period="")

What is the overall period covered by the union of all datafiles (but not necessarily by a single model!)

so.explore('choices', operation='union') { 'period': [1850-2349], 'model': ['CNRM-ESM2-1', 'CNRM-CM6-1'] .....} `