html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/pydata/xarray/pull/5731#issuecomment-978081711,https://api.github.com/repos/pydata/xarray/issues/5731,978081711,IC_kwDOAMm_X846TFev,2448579,2021-11-24T17:25:17Z,2021-11-24T17:25:17Z,MEMBER,Thanks @Gijom ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,976790237
https://github.com/pydata/xarray/pull/5731#issuecomment-977464974,https://api.github.com/repos/pydata/xarray/issues/5731,977464974,IC_kwDOAMm_X846Qu6O,2448579,2021-11-24T03:18:36Z,2021-11-24T03:18:36Z,MEMBER,"> so it probably makes sense to merge it now and save the other bug fix for a later issue?
It was easy .Should be good to go if tests pass.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,976790237
https://github.com/pydata/xarray/pull/5731#issuecomment-909335676,https://api.github.com/repos/pydata/xarray/issues/5731,909335676,IC_kwDOAMm_X842M1x8,2448579,2021-08-31T15:19:57Z,2021-08-31T15:19:57Z,MEMBER,"Ouch here's the problem: `da_a` has dtype int if it has no NaNs. this dtype could change but need not change after the masking.
Importantly `skipna=False` by default for `int` dtypes (since they can't represent NaNs).
The fix is to specify `skipna=True` explicitly in the `.mean` calls.
Alternatively we could stick an `.astype(float)` in `_get_valid_values`","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,976790237
https://github.com/pydata/xarray/pull/5731#issuecomment-909314125,https://api.github.com/repos/pydata/xarray/issues/5731,909314125,IC_kwDOAMm_X842MwhN,2448579,2021-08-31T14:54:51Z,2021-08-31T14:54:51Z,MEMBER,"oh never mind: I can reproduce with
```
da_a = xr.DataArray([[1, 2], [2, 1]], dims=[""x"", ""time""])
da_b = xr.DataArray([[1, 2], [1, np.nan]], dims=[""x"", ""time""])
dim = None
xr.testing.assert_equal(
xr.corr(da_a, da_b, dim=dim), xr.corr(da_a.chunk(), da_b.chunk(), dim=dim)
)
xr.testing.assert_equal(
xr.corr(da_a, da_b, dim=dim), xr.corr(da_a, da_b.chunk(), dim=dim)
)
```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,976790237
https://github.com/pydata/xarray/pull/5731#issuecomment-909311207,https://api.github.com/repos/pydata/xarray/issues/5731,909311207,IC_kwDOAMm_X842Mvzn,2448579,2021-08-31T14:51:43Z,2021-08-31T14:52:06Z,MEMBER,"Thanks @Gijom, I can't reproduce. Can you post the output of `xr.show_versions()`?
Here's mine
```
INSTALLED VERSIONS
------------------
commit: None
python: 3.8.10 | packaged by conda-forge | (default, May 10 2021, 22:58:09)
[Clang 11.1.0 ]
python-bits: 64
OS: Darwin
OS-release: 19.6.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: ('en_US', 'UTF-8')
libhdf5: 1.10.6
libnetcdf: 4.7.4
xarray: 0.19.0
pandas: 1.3.1
numpy: 1.21.1
scipy: 1.5.3
netCDF4: 1.5.6
pydap: None
h5netcdf: 0.11.0
h5py: 3.3.0
Nio: None
zarr: 2.8.3
cftime: 1.5.0
nc_time_axis: 1.3.1
PseudoNetCDF: None
rasterio: None
cfgrib: None
iris: 3.0.4
bottleneck: 1.3.2
dask: 2021.07.2
distributed: 2021.07.2
matplotlib: 3.4.2
cartopy: 0.19.0.post1
seaborn: 0.11.1
numbagg: None
pint: 0.17
setuptools: 49.6.0.post20210108
pip: 21.2.3
conda: 4.10.3
pytest: 6.2.4
IPython: 7.26.0
sphinx: 4.1.2
```
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,976790237
https://github.com/pydata/xarray/pull/5731#issuecomment-909297283,https://api.github.com/repos/pydata/xarray/issues/5731,909297283,IC_kwDOAMm_X842MsaD,2448579,2021-08-31T14:36:19Z,2021-08-31T14:36:19Z,MEMBER,Thanks @Gijom can you post the error for those tests please.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,976790237
https://github.com/pydata/xarray/pull/5731#issuecomment-903840548,https://api.github.com/repos/pydata/xarray/issues/5731,903840548,IC_kwDOAMm_X84134Mk,2448579,2021-08-23T14:46:28Z,2021-08-23T14:46:47Z,MEMBER,"Thanks @Gijom!
> he already existing test function test_corr does not have a @requires_dask decorator. But it did not fail as it should have. Not sure why.
I can't tell either. It would be good to track this down.
> I am not sure of the whats-new.rst format and I thus did not add it. Should I just add a bullet item on the top of the file ?
Yes just copy an existing entry and modify it.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,976790237