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/2706#issuecomment-504184558,https://api.github.com/repos/pydata/xarray/issues/2706,504184558,MDEyOklzc3VlQ29tbWVudDUwNDE4NDU1OA==,4711805,2019-06-20T21:11:57Z,2019-06-20T21:11:57Z,CONTRIBUTOR,"You're right @shikharsg, the `chunk_dim` argument can be removed. I was not very happy with the complexity it brought as well.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-498199759,https://api.github.com/repos/pydata/xarray/issues/2706,498199759,MDEyOklzc3VlQ29tbWVudDQ5ODE5OTc1OQ==,4711805,2019-06-03T10:19:15Z,2019-06-03T10:19:15Z,CONTRIBUTOR,It would be great if you could fix it. @jendrikjoe can give you the permission to push to the branch in his fork.,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-498166896,https://api.github.com/repos/pydata/xarray/issues/2706,498166896,MDEyOklzc3VlQ29tbWVudDQ5ODE2Njg5Ng==,4711805,2019-06-03T08:41:01Z,2019-06-03T08:41:01Z,CONTRIBUTOR,"Thanks @shikharsg for looking into that. This PR and master have diverged quite a bit, I will need to merge the changes, I will let you know.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-494340006,https://api.github.com/repos/pydata/xarray/issues/2706,494340006,MDEyOklzc3VlQ29tbWVudDQ5NDM0MDAwNg==,4711805,2019-05-21T10:46:20Z,2019-05-21T10:46:20Z,CONTRIBUTOR,"No problem @rabernat, and thanks a lot for your time in reviewing this PR.
I added a test for `chunk_dim`. Please let me know if this is clearer, and if I should explain further in the docs.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-485340727,https://api.github.com/repos/pydata/xarray/issues/2706,485340727,MDEyOklzc3VlQ29tbWVudDQ4NTM0MDcyNw==,4711805,2019-04-22T06:38:04Z,2019-04-22T06:38:04Z,CONTRIBUTOR,I added a `chunk_dim` parameter which allows to rechunk the appended coordinate. I think it is ready for a final review now.,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-484551274,https://api.github.com/repos/pydata/xarray/issues/2706,484551274,MDEyOklzc3VlQ29tbWVudDQ4NDU1MTI3NA==,4711805,2019-04-18T15:14:24Z,2019-04-18T15:14:24Z,CONTRIBUTOR,"I don't think it's ready yet. I think I should address the chunking issue of the appended dimension, as explained in https://medium.com/pangeo/continuously-extending-zarr-datasets-c54fbad3967d. For instance if we append along a time dimension, the time coordinate (which is a 1-D array) will have very small chunks, instead of maybe only one.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-479932785,https://api.github.com/repos/pydata/xarray/issues/2706,479932785,MDEyOklzc3VlQ29tbWVudDQ3OTkzMjc4NQ==,4711805,2019-04-04T14:57:45Z,2019-04-04T14:57:45Z,CONTRIBUTOR,"@rabernat you're right, I took your suggestion into account in my last commit. I also rewrote the test.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-479800472,https://api.github.com/repos/pydata/xarray/issues/2706,479800472,MDEyOklzc3VlQ29tbWVudDQ3OTgwMDQ3Mg==,4711805,2019-04-04T08:24:01Z,2019-04-04T08:24:01Z,CONTRIBUTOR,"Thanks @jendrikjoe, I just pushed to your fork: to make sure that the encoding of the appended variables is compatible with the target store, we explicitly put the target store encodings in the appended variable.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-479788744,https://api.github.com/repos/pydata/xarray/issues/2706,479788744,MDEyOklzc3VlQ29tbWVudDQ3OTc4ODc0NA==,4711805,2019-04-04T07:46:52Z,2019-04-04T07:46:52Z,CONTRIBUTOR,Or should I open a new PR?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-479480023,https://api.github.com/repos/pydata/xarray/issues/2706,479480023,MDEyOklzc3VlQ29tbWVudDQ3OTQ4MDAyMw==,4711805,2019-04-03T13:04:24Z,2019-04-03T13:04:24Z,CONTRIBUTOR,@jendrikjoe I think you need to give me the permission to push to the branch in your fork.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-478365516,https://api.github.com/repos/pydata/xarray/issues/2706,478365516,MDEyOklzc3VlQ29tbWVudDQ3ODM2NTUxNg==,4711805,2019-03-31T18:22:53Z,2019-03-31T18:22:53Z,CONTRIBUTOR,May I try and take this work over?,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-475558223,https://api.github.com/repos/pydata/xarray/issues/2706,475558223,MDEyOklzc3VlQ29tbWVudDQ3NTU1ODIyMw==,4711805,2019-03-22T09:53:43Z,2019-03-22T09:53:43Z,CONTRIBUTOR,"Hi @jendrikjoe, do you plan to work on this PR again in the future? I think it would be a great contribution to xarray.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-459667067,https://api.github.com/repos/pydata/xarray/issues/2706,459667067,MDEyOklzc3VlQ29tbWVudDQ1OTY2NzA2Nw==,4711805,2019-02-01T09:51:55Z,2019-02-01T09:51:55Z,CONTRIBUTOR,"When we use this feature e.g. to store data that is produced every day, we might start with a data set that has a small size on the time dimension, and thus the chunks will be chosen according to this initial shape. When we append to this data set, will the chunks be kept as in the initial zarr archive? If so, we might end up with a lot of small chunks on the time dimension, where ideally we would have chosen only one chunk.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-458866421,https://api.github.com/repos/pydata/xarray/issues/2706,458866421,MDEyOklzc3VlQ29tbWVudDQ1ODg2NjQyMQ==,4711805,2019-01-30T09:05:53Z,2019-01-30T09:05:53Z,CONTRIBUTOR,"zarr stores the reference in the `.zattrs` file:
```
{
""_ARRAY_DIMENSIONS"": [
""time""
],
""calendar"": ""proleptic_gregorian"",
""units"": ""days since 2000-01-01 00:00:00""
}
```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-458730247,https://api.github.com/repos/pydata/xarray/issues/2706,458730247,MDEyOklzc3VlQ29tbWVudDQ1ODczMDI0Nw==,4711805,2019-01-29T22:19:07Z,2019-01-29T22:19:07Z,CONTRIBUTOR,"To make it work, time dimensions would have to be treated separately because zarr doesn't encode absolute time values but deltas relative to a reference (see https://github.com/davidbrochart/pangeo_upload/blob/master/py/trmm2pangeo.py#L108).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148
https://github.com/pydata/xarray/pull/2706#issuecomment-458720920,https://api.github.com/repos/pydata/xarray/issues/2706,458720920,MDEyOklzc3VlQ29tbWVudDQ1ODcyMDkyMA==,4711805,2019-01-29T21:50:14Z,2019-01-29T21:50:14Z,CONTRIBUTOR,"Hi @jendrikjoe,
Thanks for your PR, I am very interested in it because this is something I was hacking around (see [here](https://github.com/davidbrochart/pangeo_upload/blob/master/py/trmm2pangeo.py)). In my particular case, I want to append along a time dimension, but it looks like your PR currently doesn't support it. In the following example `ds2` should have a time dimension ranging from 2000-01-01 to 2000-01-06:
```python
import xarray as xr
import pandas as pd
ds0 = xr.Dataset({'temperature': (['time'], [50, 51, 52])}, coords={'time': pd.date_range('2000-01-01', periods=3)})
ds1 = xr.Dataset({'temperature': (['time'], [53, 54, 55])}, coords={'time': pd.date_range('2000-01-04', periods=3)})
ds0.to_zarr('temp')
ds1.to_zarr('temp', mode='a', append_dim='time')
ds2 = xr.open_zarr('temp')
```
But it's not the case:
```
ds2.time
array(['2000-01-01T00:00:00.000000000', '2000-01-02T00:00:00.000000000',
'2000-01-03T00:00:00.000000000', '2000-01-01T00:00:00.000000000',
'2000-01-02T00:00:00.000000000', '2000-01-03T00:00:00.000000000'],
dtype='datetime64[ns]')
Coordinates:
* time (time) datetime64[ns] 2000-01-01 2000-01-02 ... 2000-01-03
```
Maybe it's not intended to work with time dimensions yet?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,402908148