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/1967#issuecomment-371011595,https://api.github.com/repos/pydata/xarray/issues/1967,371011595,MDEyOklzc3VlQ29tbWVudDM3MTAxMTU5NQ==,12229877,2018-03-07T03:34:37Z,2018-03-07T03:34:37Z,CONTRIBUTOR,"Not any more :wink: - it links to the docs that ""[`xarray.plot.plot()`] calls an xarray plotting function based on the dimensions of darray.squeeze()"".

I'd actually like to keep the magic in `plot()` - the whole point of this is that we can do *something* with the data, not that we do anything in particular.  For interactive use, this is valuable enough - in my view - to justify keeping an inconsistent API; particularly when there is an obvious non-magic version to use in a script.  In any case, that's a topic for another pull.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,302695966
https://github.com/pydata/xarray/pull/1967#issuecomment-370974503,https://api.github.com/repos/pydata/xarray/issues/1967,370974503,MDEyOklzc3VlQ29tbWVudDM3MDk3NDUwMw==,12229877,2018-03-07T00:06:15Z,2018-03-07T03:08:31Z,CONTRIBUTOR,"Ah, I see what you mean but don't think we need any change or additional test.

There's a catch though - `xarray.plot()` is special, because it (and only it) [squeezes the dimensions of the array](http://xarray.pydata.org/en/stable/generated/xarray.plot.plot.html#xarray.plot.plot) before plotting it.  Therefore, the following calls all produce the same plot:

    xr.DataArray(np.arange(9).reshape((3,3))).plot()
    xr.DataArray(np.arange(9).reshape((1,3,3))).plot()
    xr.DataArray(np.arange(9).reshape((1,1,3,1,1,1,1,3,1,1))).plot()

My view is that the test you linked to is sufficient for the test you're asking for - `imshow` is a special case because it *can* accept 3D input for RGB plots.

TLDR - working as intended IMO, it's just that nobody reads the docs.  Changing the API would avoid this but at cost of convenience which is the whole point of `DataArray.plot()`.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,302695966
https://github.com/pydata/xarray/pull/1967#issuecomment-370813600,https://api.github.com/repos/pydata/xarray/issues/1967,370813600,MDEyOklzc3VlQ29tbWVudDM3MDgxMzYwMA==,12229877,2018-03-06T15:13:43Z,2018-03-06T15:13:43Z,CONTRIBUTOR,"No problem - I had an hour free and no open pulls waiting on me, so the timing was good.

As a regression test it's specific to imshow, so I'm not sure what you'd want here (or whether it would work at all on the 2d mixin). More details please? ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,302695966