roocs / clisops

Climate Simulation Operations
https://clisops.readthedocs.io/en/latest/
Other
21 stars 9 forks source link

Subset by point #193

Closed cehbrecht closed 3 years ago

cehbrecht commented 3 years ago

Pull Request Checklist:

Feature

yes

cehbrecht commented 3 years ago

@agstephens @Zeitsperre I have updated the notebook example for subset. Should we add shortcut imports for the parameters?

from roocs_utils.parameter.param_utils import time_interval

as 

from clisops.ops import time_interval
cehbrecht commented 3 years ago

@agstephens added a fix for subset_bbox when combing area and time_components parameter: https://github.com/roocs/clisops/commit/ff69b7b9a71f95d955c3e7624d4ae3ceb4140c04

cehbrecht commented 3 years ago

@Zeitsperre when we have merged this PR we want to make a 0.7.0 release, probably next week on Tuesday. Ok?

agstephens commented 3 years ago

@agstephens @Zeitsperre I have updated the notebook example for subset. Should we add shortcut imports for the parameters?

from roocs_utils.parameter.param_utils import time_interval

as 

from clisops.ops import time_interval

@cehbrecht Yes, that sounds good.

agstephens commented 3 years ago

@agstephens added a fix for subset_bbox when combing area and time_components parameter: ff69b7b

Great stuff. @cehbrecht . Thanks for spotting this.