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/6797#issuecomment-1192103529,https://api.github.com/repos/pydata/xarray/issues/6797,1192103529,IC_kwDOAMm_X85HDg5p,2448579,2022-07-22T02:03:43Z,2022-07-22T02:03:43Z,MEMBER,Thanks @maxrjones I added a sparse variable test that fails on main.,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1306903264
https://github.com/pydata/xarray/pull/6797#issuecomment-1189861071,https://api.github.com/repos/pydata/xarray/issues/6797,1189861071,IC_kwDOAMm_X85G69bP,35968931,2022-07-20T05:57:58Z,2022-07-20T05:57:58Z,MEMBER,Great first contribution @maxrjones ! #6804 should auto-merge in a sec so you can then fetch and merge in `main` to write your tests :smile: ,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1306903264
https://github.com/pydata/xarray/pull/6797#issuecomment-1189527359,https://api.github.com/repos/pydata/xarray/issues/6797,1189527359,IC_kwDOAMm_X85G5r8_,2448579,2022-07-19T20:37:09Z,2022-07-19T20:37:09Z,MEMBER,Ah great idea!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1306903264
https://github.com/pydata/xarray/pull/6797#issuecomment-1189473091,https://api.github.com/repos/pydata/xarray/issues/6797,1189473091,IC_kwDOAMm_X85G5etD,14077947,2022-07-19T19:31:31Z,2022-07-19T19:31:31Z,CONTRIBUTOR,"> Thanks @maxrjones can you push any WIP tests you might have for this?
If it's alright I will wait for https://github.com/pydata/xarray/pull/6804 to be merged first because that will greatly simplify testing the changes. After that PR, the following could be added to `xarray/tests/test_array_api.py` to test both cases:
```
def test_properties(arrays) -> None:
np_arr, xp_arr = arrays
assert np_arr.nbytes == 48
assert xp_arr.nbytes == 48
```
","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1306903264
https://github.com/pydata/xarray/pull/6797#issuecomment-1189154297,https://api.github.com/repos/pydata/xarray/issues/6797,1189154297,IC_kwDOAMm_X85G4Q35,2448579,2022-07-19T14:52:11Z,2022-07-19T14:52:11Z,MEMBER,Thanks @maxrjones can you push any WIP tests you might have for this?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1306903264
https://github.com/pydata/xarray/pull/6797#issuecomment-1186293504,https://api.github.com/repos/pydata/xarray/issues/6797,1186293504,IC_kwDOAMm_X85GtWcA,2448579,2022-07-16T21:09:23Z,2022-07-16T21:09:23Z,MEMBER,"> I should add a docstring as suggested
This would be great!
Adding docstrings for the other properties would be a great (separate) PR!","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1306903264
https://github.com/pydata/xarray/pull/6797#issuecomment-1186293138,https://api.github.com/repos/pydata/xarray/issues/6797,1186293138,IC_kwDOAMm_X85GtWWS,14077947,2022-07-16T21:08:11Z,2022-07-16T21:08:11Z,CONTRIBUTOR,Please let me know if I should add a docstring as suggested in https://github.com/pydata/xarray/issues/6565#issuecomment-1115545322. I didn't yet because most of the properties do not have docstrings.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1306903264