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/issues/825#issuecomment-210923242,https://api.github.com/repos/pydata/xarray/issues/825,210923242,MDEyOklzc3VlQ29tbWVudDIxMDkyMzI0Mg==,12307589,2016-04-16T23:29:06Z,2016-04-16T23:29:06Z,CONTRIBUTOR,"It turns out the bug was line 323 of groupby.py, _concat_shortcut silently copies the metadata of the array doing the concatenation to the result. I've removed that line and now the tests are passing.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,148765426
https://github.com/pydata/xarray/issues/825#issuecomment-210711180,https://api.github.com/repos/pydata/xarray/issues/825,210711180,MDEyOklzc3VlQ29tbWVudDIxMDcxMTE4MA==,12307589,2016-04-16T01:58:53Z,2016-04-16T02:00:28Z,CONTRIBUTOR,"It turns out that in addition, first and last in ops don't accept keep_attrs as a keyword argument, so right now they _always_ preserve attributes. A side effect of this is that the keep_attrs arguments passed around by _first_and_last and whatnot in groupby actually don't do anything (though their default value, True, reflects what happens).
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,148765426
https://github.com/pydata/xarray/issues/825#issuecomment-210701860,https://api.github.com/repos/pydata/xarray/issues/825,210701860,MDEyOklzc3VlQ29tbWVudDIxMDcwMTg2MA==,12307589,2016-04-16T01:06:27Z,2016-04-16T01:06:27Z,CONTRIBUTOR,"@shoyer the default keep_attrs isn't the problem here, the issue is that there is currently no keep_attrs option at all for resampling.
I've implemented a solution, but now test TestDataset.test_resample_and_first is failing. This is because for how=""first"" and how=""last"", attributes _are_ currently kept (keep_attrs=True). This may break some code if resample is given a default of keep_attrs=False. Using a default of keep_attrs=True for how in ('first', 'last') results in the test passing.
Alternatively I could make it so the default behavior is to not pass any keep_attrs value on to the grouper function, which would keep the current defaults of those groupers. The code would be a bit uglier but it's not hard, and it would prevent breaking scripts. What do we want for the default behavior?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,148765426
https://github.com/pydata/xarray/issues/825#issuecomment-210695712,https://api.github.com/repos/pydata/xarray/issues/825,210695712,MDEyOklzc3VlQ29tbWVudDIxMDY5NTcxMg==,12307589,2016-04-16T00:21:44Z,2016-04-16T00:21:44Z,CONTRIBUTOR,"@pwolfram I use xarray within a wrapper for my own work, and have already written this transfer-attributes functionality into that for my short-term solution. But it makes sense to have the same keep_attrs flag that many other xarray functions have.
@jhamman I'll try to put the PR together.
","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,148765426
https://github.com/pydata/xarray/issues/825#issuecomment-210685190,https://api.github.com/repos/pydata/xarray/issues/825,210685190,MDEyOklzc3VlQ29tbWVudDIxMDY4NTE5MA==,4295853,2016-04-15T23:36:54Z,2016-04-15T23:36:54Z,CONTRIBUTOR,"Thanks @jhamman. You are correct that this could get challenging without proper notions of units. Do we have a utility to transfer attributes from one Dataset to another? If not, perhaps that is the simplest, short term resolution to this issue that is even more general than addition of a `keep_attrs` flag. I don't think it would be to hard to write although it may be out of xarray scope.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,148765426
https://github.com/pydata/xarray/issues/825#issuecomment-210647514,https://api.github.com/repos/pydata/xarray/issues/825,210647514,MDEyOklzc3VlQ29tbWVudDIxMDY0NzUxNA==,4295853,2016-04-15T21:10:36Z,2016-04-15T21:10:36Z,CONTRIBUTOR,"@mcgibbon, I would agree that in general attributes should be preserved to maintain provenance of DataArrays or Datasets unless there is a really good reason to drop them.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,148765426