bradyrx / esmtools

:octopus: a toolbox for Earth System Model analysis :octopus:
https://esmtools.readthedocs.io/en/latest/
MIT License
27 stars 4 forks source link

Update xr version #92

Closed bradyrx closed 4 years ago

bradyrx commented 4 years ago

Description

This updates to the xarray v0.16.0 requirement. This allows us to use xr.infer_freq directly. I avoided using xr.corr() since it doesn't really add any value over us using xskillscore.pearson_r. The xarray version does not have an option to return a p value, so we'ld literally only change our code from xs.pearson_r to xr.corr. And we'd need xs.pearson_r_p_value anyways so we wouldn't drop the requirement.