Results 138 comments of James Adams

Look into [Azure Pipelines](https://azure.microsoft.com/en-us/services/devops/pipelines/) which also has a lot of good reviews.

CircleCI is now enabled, as of PR #358

Yes, the time values should be of the form "\ since \" as described [here](https://www.unidata.ucar.edu/software/netcdf/time/recs.html). Off the top of my head I think we can use anything that xarray supports...

Yes, agreed, it's mentioned in the README that this is Python 3 but we can be more explicit. We can include a badge of some sort on the README showing...

It looks like this feature is included in the [`__spi__.py`](https://github.com/monocongo/climate_indices/blob/master/src/climate_indices/__spi__.py) script. Hopefully, this may at least steer you in the right direction until we can get this code rolled into...

@JulianGiles This is excellent work, thanks a lot for this contribution! I think generalizing this to also cover non-gridded datasets such as climate divisions won't be too much extra work,...

https://github.com/rmcgibbo/python-appveyor-conda-example

Hi @Yassminaa When we first developed the SPEI it was necessary to add an offset to make sure we always have a positive value, so we went with 1000. There...

This sounds like a good idea to me, please have at it! Thanks in advance for your contributions.