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/1926#issuecomment-370586654,https://api.github.com/repos/pydata/xarray/issues/1926,370586654,MDEyOklzc3VlQ29tbWVudDM3MDU4NjY1NA==,10050469,2018-03-05T22:15:09Z,2018-03-05T22:15:09Z,MEMBER,Thanks @dcherian ! This is a nice feature,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-370174039,https://api.github.com/repos/pydata/xarray/issues/1926,370174039,MDEyOklzc3VlQ29tbWVudDM3MDE3NDAzOQ==,1217238,2018-03-03T19:39:30Z,2018-03-03T19:39:30Z,MEMBER,@fmaussion any further concerns here? This looks good to me.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-369467411,https://api.github.com/repos/pydata/xarray/issues/1926,369467411,MDEyOklzc3VlQ29tbWVudDM2OTQ2NzQxMQ==,2448579,2018-03-01T04:01:59Z,2018-03-01T04:01:59Z,MEMBER,Done and rebased on master.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-369032553,https://api.github.com/repos/pydata/xarray/issues/1926,369032553,MDEyOklzc3VlQ29tbWVudDM2OTAzMjU1Mw==,10050469,2018-02-27T21:29:20Z,2018-02-27T21:29:20Z,MEMBER,Thanks! In view of @shoyer ' [comment](https://github.com/pydata/xarray/pull/1926#issuecomment-367383795) we could revisit the API for datasets at a later stage.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-368760427,https://api.github.com/repos/pydata/xarray/issues/1926,368760427,MDEyOklzc3VlQ29tbWVudDM2ODc2MDQyNw==,2448579,2018-02-27T06:20:50Z,2018-02-27T06:20:50Z,MEMBER,"Updated and rebased. Now these are the only options.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-367414417,https://api.github.com/repos/pydata/xarray/issues/1926,367414417,MDEyOklzc3VlQ29tbWVudDM2NzQxNDQxNw==,1217238,2018-02-21T17:59:17Z,2018-02-21T17:59:17Z,MEMBER,"Yes, this is not valid syntax on Python 2.
On Wed, Feb 21, 2018 at 9:53 AM Deepak Cherian
wrote:
> *@dcherian* commented on this pull request.
> ------------------------------
>
> In xarray/plot/plot.py
> :
>
> > add_legend = kwargs.pop('add_legend', True)
>
> ax = get_axis(figsize, size, aspect, ax)
>
> + error_msg = 'must be either None or one of %r' % list(darray.dims)
> +
> + if x not in [None, *darray.dims]:
>
> I don't know what's wrong here. It runs locally; the tests pass and flake8
> goes through plot.py without errors. Is it a python2 vs python3 thing?
>
> —
> 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}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-367397397,https://api.github.com/repos/pydata/xarray/issues/1926,367397397,MDEyOklzc3VlQ29tbWVudDM2NzM5NzM5Nw==,2448579,2018-02-21T17:05:37Z,2018-02-21T17:06:13Z,MEMBER,"> either by (i) specifying the name of the variable to the axis you want to plot it onto, or (ii) by specifying the name of the coordinate you want to use for the axis. Since (ii) is the default and only option for 1D and 2D plots until now, I just wondered if (i) is very necessary.
> It does feel a little strange that if you had a DataArray with non-dimension coordinates you could make a plot without including any of the DataArray values
Good points. I'll change it so that only one of `x` and `y` can be specified.
> My main thought is that this API would feel much more natural on a Dataset object, alongside a .plot.scatter() method.
It's very useful in oceanography (and meteorology) where we want to make plots of physical quantities like temperature against depth; it is much more natural to see depth on the y-axis than the x-axis. This is the original usecase @rabernat mention in #575.
The Dataset API is a more involved undertaking that I'm not confident of executing at this point.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-367383795,https://api.github.com/repos/pydata/xarray/issues/1926,367383795,MDEyOklzc3VlQ29tbWVudDM2NzM4Mzc5NQ==,1217238,2018-02-21T16:26:26Z,2018-02-21T16:26:26Z,MEMBER,"My main thought is that this API would feel much more natural on a Dataset object, alongside a `.plot.scatter()` method.
That said, I suppose this could still be useful and I don't think it's harmful to expand the API here. It does feel a little strange that if you had a DataArray with non-dimension coordinates you could make a plot without including any of the DataArray values, e.g., `xr.DataArray(..., dims=['x'], coords={'x': ..., 'y': ('x', ...)}, name='f').plot.line(x='x', y='y')`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-367357926,https://api.github.com/repos/pydata/xarray/issues/1926,367357926,MDEyOklzc3VlQ29tbWVudDM2NzM1NzkyNg==,10050469,2018-02-21T15:12:19Z,2018-02-21T15:12:19Z,MEMBER,"> Do you mean da.plot(y='time', x=None, hue='lat')
Yes, I meant that there should be one and only way to specify which axis should do what: my understanding of your current implementation is that there are two ways to reach the same goal: either by (i) specifying the name of the variable to the axis you want to plot it onto, or (ii) by specifying the name of the coordinate you want to use for the axis. Since (ii) is the default and only option for 1D and 2D plots until now, I just wondered if (i) is very necessary. (or maybe I missed something)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284
https://github.com/pydata/xarray/pull/1926#issuecomment-367152500,https://api.github.com/repos/pydata/xarray/issues/1926,367152500,MDEyOklzc3VlQ29tbWVudDM2NzE1MjUwMA==,2448579,2018-02-20T23:04:59Z,2018-02-20T23:07:03Z,MEMBER,"@fmaussion when I added the `hue` argument, @shoyer suggested that we have a ""fully explicit way to make these plots"".
In this case this would mean
```da.plot(x='temp', y='time', hue='lat')```
which seems a lot more readable than
```da.plot(y='coordinate', hue='lat')```
It would also mean that `x,y` are different from `hue` (for `hue` we do specify dimension name).
@shoyer Thoughts?
EDIT:
@fmaussion Do you mean `da.plot(y='time', x=None, hue='lat')`?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,298481284