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/882#issuecomment-226502318,https://api.github.com/repos/pydata/xarray/issues/882,226502318,MDEyOklzc3VlQ29tbWVudDIyNjUwMjMxOA==,10194086,2016-06-16T14:29:36Z,2016-06-16T14:29:36Z,MEMBER,"What _may_ be possible: extend `@_plot2d` with a `projection=None` and a `transform=None` keyword (and the handling thereof)*. In `geoplot` we import `@_plot2d` set the defaults to `projection=ccrs.PlateCaree()` and `transform=ccrs.PlateCaree()`, possibly intercept `coastlines` from `kwargs`. I don't know if it is possible to update the defaults without explicitly re-writing the call signature of the function (which may change (but this could possibly be tested)).
\* This may conflict with the current `ax = plt.gca()`.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,160009256
https://github.com/pydata/xarray/pull/882#issuecomment-226480439,https://api.github.com/repos/pydata/xarray/issues/882,226480439,MDEyOklzc3VlQ29tbWVudDIyNjQ4MDQzOQ==,10050469,2016-06-16T13:11:29Z,2016-06-16T13:11:29Z,MEMBER,"> We would need to think hard about how to extend the current implementation without needing to duplicate it.
Do you think it would be enough to expose the `@_plot2d` decorator?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,160009256
https://github.com/pydata/xarray/pull/882#issuecomment-226411408,https://api.github.com/repos/pydata/xarray/issues/882,226411408,MDEyOklzc3VlQ29tbWVudDIyNjQxMTQwOA==,10194086,2016-06-16T07:42:13Z,2016-06-16T07:42:13Z,MEMBER,"Yes is is possible to say `subplot_kws=dict(projection=ccrs.PlateCaree()`. For a single plot it is not possible to specify the `projection` (as `ax=plt.gca()`) in the current implementation. Thus one has to create and pass a geoaxes explicitly.
I like the idea of doing `data.geoplot.pcolormesh()`
- We could keep the standard name of the plotting function without the need to make up our own.
- It is possibly easier to extend it to contour and contourf.
- We can possibly offer more control over specific cartopy features (although adding coasts is possibly the one thing that we will be able to agree on).
Some questions:
- Would it also offer `data.geoplot.line` ect.? There is no point but this would be consistent might be asked for.
- We would need to think hard about how to extend the current implementation without needing to duplicate it.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,160009256
https://github.com/pydata/xarray/pull/882#issuecomment-226252313,https://api.github.com/repos/pydata/xarray/issues/882,226252313,MDEyOklzc3VlQ29tbWVudDIyNjI1MjMxMw==,1217238,2016-06-15T17:03:07Z,2016-06-15T17:03:07Z,MEMBER,"I'm pretty sure it's _possible_ to add cartopy axes to a FacetGrid using the `subplot_kwds` argument already. Unfortunately, I don't have any examples handy since I wrote that code at my last job.
What this code does is add the `projection=ccrs.PlateCarree()` automatically, along with a few other geo specific niceties. I made that suggestion for cartopy itself (https://github.com/SciTools/cartopy/issues/645) but it hasn't been acted on yet.
I agree that this is a good fit for xarray because the plotting API is difficult to extend externally. One nice way to present this to users could be to make a `geoplot` accessor that works like the current `plot` accessor but adds these geo specific options. Then you could write, e.g., `data.geoplot.pcolormesh()`.
","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,160009256
https://github.com/pydata/xarray/pull/882#issuecomment-226132209,https://api.github.com/repos/pydata/xarray/issues/882,226132209,MDEyOklzc3VlQ29tbWVudDIyNjEzMjIwOQ==,10050469,2016-06-15T09:13:50Z,2016-06-15T09:13:50Z,MEMBER,"I also agree that this is a not to intrusive and useful ""geo-specific"" addition. Without looking at the implementation, I also wonder if it would be possible to add the same functionality to the existing framework without adding a specific function? The background is that there are other ways than pcolormesh to plot on a map (see also https://github.com/pydata/xarray/issues/657)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,160009256
https://github.com/pydata/xarray/pull/882#issuecomment-225908579,https://api.github.com/repos/pydata/xarray/issues/882,225908579,MDEyOklzc3VlQ29tbWVudDIyNTkwODU3OQ==,6200806,2016-06-14T14:56:30Z,2016-06-14T14:56:30Z,CONTRIBUTOR,"@shoyer maybe just a name change then? cartopy obviously is inherently geo-specific. Is it possible that other fields would find the utilities in geocolormesh useful? Or could the geocolormesh functionality be refactored into the existing plotting framework? (Just thinking out loud...not familiar with the plotting codebase at all)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,160009256
https://github.com/pydata/xarray/pull/882#issuecomment-225772187,https://api.github.com/repos/pydata/xarray/issues/882,225772187,MDEyOklzc3VlQ29tbWVudDIyNTc3MjE4Nw==,1217238,2016-06-14T03:42:13Z,2016-06-14T03:42:13Z,MEMBER,"I was trying to avoid getting too much geo-specific functionality in xarray, but this does nicely extend the existing plotting methods, so I think this is a reasonable addition. Anyone else have opinions on this one?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,160009256
https://github.com/pydata/xarray/pull/882#issuecomment-225663415,https://api.github.com/repos/pydata/xarray/issues/882,225663415,MDEyOklzc3VlQ29tbWVudDIyNTY2MzQxNQ==,10194086,2016-06-13T18:14:04Z,2016-06-13T18:14:04Z,MEMBER,"These are the plots from above.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,160009256