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/814#issuecomment-348357025,https://api.github.com/repos/pydata/xarray/issues/814,348357025,MDEyOklzc3VlQ29tbWVudDM0ODM1NzAyNQ==,1217238,2017-11-30T23:41:55Z,2017-11-30T23:41:55Z,MEMBER,"I think either way works. Probably easiest to fork this branch and then
merge in the latest from master.
On Thu, Nov 30, 2017 at 3:38 PM Neil Parley
wrote:
> I think also I could make you a contributor or transfer the branch to
> someone. So that might be an option too.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> , or mute
> the thread
>
> .
>
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-309810114,https://api.github.com/repos/pydata/xarray/issues/814,309810114,MDEyOklzc3VlQ29tbWVudDMwOTgxMDExNA==,1217238,2017-06-20T16:14:36Z,2017-06-20T16:14:36Z,MEMBER,"No worries! The great thing about GitHub is that it's pretty easy for
someone else to pick up your work :)
On Tue, Jun 20, 2017 at 12:25 AM, Neil Parley
wrote:
> Sorry guys I don't have time to do anything on this (or any open source
> stuff) at the moment.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> , or mute
> the thread
>
> .
>
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-309485895,https://api.github.com/repos/pydata/xarray/issues/814,309485895,MDEyOklzc3VlQ29tbWVudDMwOTQ4NTg5NQ==,1217238,2017-06-19T16:02:00Z,2017-06-19T16:02:00Z,MEMBER,"Yes, we would still love to get this in, though it might need a few other
changes to work now. Could you open a new PR with the tests added back in?
(Assuming Neil is not interested in picking this back up)
On Mon, Jun 19, 2017 at 8:58 AM, DWesl wrote:
> If you're still looking for the old tests, it looks like they disappeared
> in the last merge commit, f48de5
>
> .
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> , or mute
> the thread
>
> .
>
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-262372139,https://api.github.com/repos/pydata/xarray/issues/814,262372139,MDEyOklzc3VlQ29tbWVudDI2MjM3MjEzOQ==,1217238,2016-11-22T21:32:57Z,2016-11-22T21:32:57Z,MEMBER,"> This is looking pretty comprehensive to me. Could maybe do with a couple of tests perhaps, but otherwise :+1: .
Indeed, merging this is really just blocked on tests right now. We need at least *something* so we can verify that this basically works.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-228608416,https://api.github.com/repos/pydata/xarray/issues/814,228608416,MDEyOklzc3VlQ29tbWVudDIyODYwODQxNg==,1217238,2016-06-26T16:04:11Z,2016-06-26T16:04:11Z,MEMBER,"No worries, no rush here from my end :).
On Sun, Jun 26, 2016 at 1:35 PM Neil Parley notifications@github.com
wrote:
> @shoyer https://github.com/shoyer I have not forgotten about this PR, I
> just been working a lot on a pandas PR updating the ci and it's been taking
> up a lot of my spare time.
>
> —
> You are receiving this because you were mentioned.
>
> Reply to this email directly, view it on GitHub
> https://github.com/pydata/xarray/pull/814#issuecomment-228596914, or mute
> the thread
> https://github.com/notifications/unsubscribe/ABKS1tPi3K7IyPa2MwwQWDfnrwMemmetks5qPmQJgaJpZM4H9jGH
> .
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-228414378,https://api.github.com/repos/pydata/xarray/issues/814,228414378,MDEyOklzc3VlQ29tbWVudDIyODQxNDM3OA==,1217238,2016-06-24T17:51:42Z,2016-06-24T17:51:42Z,MEMBER,"@pelson @rhattersley any tips on handling the edge cases of cubes with some, but not all missing dim_coords?
@nparley I would hate for this to get lost. I'm OK merging this before it covers every edge case -- let's just document the limitations and leave that for later cleanup.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-221758131,https://api.github.com/repos/pydata/xarray/issues/814,221758131,MDEyOklzc3VlQ29tbWVudDIyMTc1ODEzMQ==,1217238,2016-05-26T01:58:38Z,2016-05-26T01:58:38Z,MEMBER,"OK, I was not paying attention here with #860 -- I assumed that the build problem was with the build including Iris, but it was actually the other build that started spontaneously failing.
I'm afraid I already merged that PR, so could you please try rebasing again? Sorry for the hassle! On the plus side, I think using conda-forge will ultimately work better than using all the separate channels.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-221629657,https://api.github.com/repos/pydata/xarray/issues/814,221629657,MDEyOklzc3VlQ29tbWVudDIyMTYyOTY1Nw==,1217238,2016-05-25T16:24:14Z,2016-05-25T16:24:14Z,MEMBER,"@nparley Let's see if https://github.com/pydata/xarray/pull/860 works. I think that will probably solve the CI issue (at least on Travis-CI).
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-219908364,https://api.github.com/repos/pydata/xarray/issues/814,219908364,MDEyOklzc3VlQ29tbWVudDIxOTkwODM2NA==,1217238,2016-05-18T02:23:45Z,2016-05-18T02:23:45Z,MEMBER,"@rhattersley I think this function is cleaner without the `cf_var_name` argument, so I would probably remove it. That said, I usually try to avoid issuing warnings _at all_.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-218525434,https://api.github.com/repos/pydata/xarray/issues/814,218525434,MDEyOklzc3VlQ29tbWVudDIxODUyNTQzNA==,1217238,2016-05-11T17:09:39Z,2016-05-11T17:09:39Z,MEMBER,"@jhamman and I have a few minor suggestions, but otherwise this looks good!
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-205368702,https://api.github.com/repos/pydata/xarray/issues/814,205368702,MDEyOklzc3VlQ29tbWVudDIwNTM2ODcwMg==,1217238,2016-04-04T16:04:26Z,2016-04-04T16:04:26Z,MEMBER,"One other thing that I noticed from the latest example -- it would be nice to covert cell methods from Cubes into string attributes on DataArrays.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-205368137,https://api.github.com/repos/pydata/xarray/issues/814,205368137,MDEyOklzc3VlQ29tbWVudDIwNTM2ODEzNw==,1217238,2016-04-04T16:02:38Z,2016-04-04T16:02:38Z,MEMBER,"> Is there any way to know for sure whether a coordinate has been automatically created? Or is the name the only clue?
I'm afraid the name is the only clue. My inclination would be not to do anything special here to ""un-name"" default coordinates, assuming Iris has a straightforward way to get rid of them. Dimensions like `dim0` come up quickly in demo code but rarely in practice -- it's mostly a signal to the user that the should pick a meaningful name.
> If this is the way to do things then does this comment need deleting?
Yes, that comment is out of date. I'm currently refactoring that code though, so let me take care care of it.
There's a separate argument about whether or not xarray should automatically convert cubes when passed into the `xarray.DataArray` constructor. We do this for pandas objects, but not any other labelled array types, currently.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-204802488,https://api.github.com/repos/pydata/xarray/issues/814,204802488,MDEyOklzc3VlQ29tbWVudDIwNDgwMjQ4OA==,1217238,2016-04-02T20:41:21Z,2016-04-02T20:41:21Z,MEMBER,"There's no good solution for representing bounds in xarray -- we can put these as coordinates on Dataset objects, but they aren't supported on DataArray objects. I had some plans to add an `IntervalIndex` type to pandas (https://github.com/pydata/pandas/pull/8707) which would let us support bounds, but honestly I'm not sure if/when I'll finish that.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657
https://github.com/pydata/xarray/pull/814#issuecomment-204617927,https://api.github.com/repos/pydata/xarray/issues/814,204617927,MDEyOklzc3VlQ29tbWVudDIwNDYxNzkyNw==,1217238,2016-04-02T00:37:47Z,2016-04-02T00:38:31Z,MEMBER,"This is awesome!
Let's try to add some documentation -- at least a mention in What's New.
CC @pelson
","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,145140657