roocs / clisops

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

added xclim subset module #4

Closed cehbrecht closed 4 years ago

cehbrecht commented 4 years ago

Changes:

cehbrecht commented 4 years ago

@ellesmith88 I have moved test_subset.py to test_operations.

cehbrecht commented 4 years ago

@ellesmith88 I have tested like this:

$ git submodule update --init 
$ conda env create -f environment.yml
$ conda activate clisops
$ python setup.py develop
$ pip install -r requirements_dev.txt
$ pytest tests/

All tests passed ...

I will try to fix travis with a new PR (see #2)

ellesmith88 commented 4 years ago

@cehbrecht Thanks, I realised I was doing something wrong with the testing. I'm happy with it.

cehbrecht commented 4 years ago

@ellesmith88 merged. Thanks for review :)