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/1712#issuecomment-360789654,https://api.github.com/repos/pydata/xarray/issues/1712,360789654,MDEyOklzc3VlQ29tbWVudDM2MDc4OTY1NA==,10050469,2018-01-26T13:51:15Z,2018-01-26T13:51:15Z,MEMBER,All green! Will try to get to the tests in the upcoming days,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,273268690
https://github.com/pydata/xarray/pull/1712#issuecomment-357457494,https://api.github.com/repos/pydata/xarray/issues/1712,357457494,MDEyOklzc3VlQ29tbWVudDM1NzQ1NzQ5NA==,10050469,2018-01-13T18:51:23Z,2018-01-13T18:51:23Z,MEMBER,Thanks for the review @shoyer ! I will wait for https://github.com/pydata/xarray/pull/1817 to be merged before going on here.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,273268690
https://github.com/pydata/xarray/pull/1712#issuecomment-356737816,https://api.github.com/repos/pydata/xarray/issues/1712,356737816,MDEyOklzc3VlQ29tbWVudDM1NjczNzgxNg==,10050469,2018-01-10T21:10:55Z,2018-01-10T21:10:55Z,MEMBER,I wonder if someone from @pydata/xarray wants to have a look -- if possible I'd like to get this in before the next point release. Thanks!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,273268690
https://github.com/pydata/xarray/pull/1712#issuecomment-355060935,https://api.github.com/repos/pydata/xarray/issues/1712,355060935,MDEyOklzc3VlQ29tbWVudDM1NTA2MDkzNQ==,10050469,2018-01-03T16:44:29Z,2018-01-03T16:44:29Z,MEMBER,"Thanks @maaleske for having a look.
> having the coordinate generation separately exposed would be nice
I agree, but this could be done in a separate PR as this needs another kind of discussion: do we want these kind of domain-specific tools in the codebase (and where?), and is `Affine` sufficiently used to justify this? Here we use `Affine` because it is a `rasterio` dependency, but I don't know if it is used outside of it.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,273268690
https://github.com/pydata/xarray/pull/1712#issuecomment-351759168,https://api.github.com/repos/pydata/xarray/issues/1712,351759168,MDEyOklzc3VlQ29tbWVudDM1MTc1OTE2OA==,10050469,2017-12-14T16:19:31Z,2017-12-14T16:19:31Z,MEMBER,"@snowman2 I added an example in the code on how to do this - can be useful for downstream libraries
@maaleske @jhamman this is ready for review / opinions. This PR still closes https://github.com/pydata/xarray/issues/1686 in a backwards compatible manner and adds a special case for irregular coordinates. I don't think that 2D coordinates happen very often (never encountered one in my work), but xarray should handle them now. The PR also makes some minor other modifications.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,273268690
https://github.com/pydata/xarray/pull/1712#issuecomment-348722604,https://api.github.com/repos/pydata/xarray/issues/1712,348722604,MDEyOklzc3VlQ29tbWVudDM0ODcyMjYwNA==,10050469,2017-12-02T21:49:01Z,2017-12-02T21:49:01Z,MEMBER,"I added a `parse_coordinates=True` kwarg and added a check for 2D coordinates.
I proposed an implementation for the 2D case, but it feels a bit clunky. A second option would be to *not* parse 2D coords and raise a Warning in case of ``parse_coordinates=True`` and 2D coordinates.
(I don't think that 2D coords happen very often).
Thoughts?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,273268690
https://github.com/pydata/xarray/pull/1712#issuecomment-343777829,https://api.github.com/repos/pydata/xarray/issues/1712,343777829,MDEyOklzc3VlQ29tbWVudDM0Mzc3NzgyOQ==,10050469,2017-11-12T23:29:56Z,2017-11-12T23:29:56Z,MEMBER,"OK, I think this is ready for review (@shoyer or @jhamman) ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,273268690