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/issues/5589#issuecomment-877287358,https://api.github.com/repos/pydata/xarray/issues/5589,877287358,MDEyOklzc3VlQ29tbWVudDg3NzI4NzM1OA==,2448579,2021-07-09T15:57:27Z,2021-07-09T15:57:27Z,MEMBER,"We should create a DataArray with just the data that is needed based on `x`, `y` etc. and then call compute on the whole thing, rather than computing `x` separately and `y` separately.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,940702754 https://github.com/pydata/xarray/issues/5589#issuecomment-877278379,https://api.github.com/repos/pydata/xarray/issues/5589,877278379,MDEyOklzc3VlQ29tbWVudDg3NzI3ODM3OQ==,35968931,2021-07-09T15:42:50Z,2021-07-09T15:42:50Z,MEMBER,"But let me know if you spot anywhere that #5561 isn't going to cover with `to_numpy()` calls! > I guess the 1d plot functions would all need to be modified individually. @Illviljan was talking about adding a `@_plot1d` decorator like the 2d one, which might be a good place to enforce this.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,940702754 https://github.com/pydata/xarray/issues/5589#issuecomment-877268860,https://api.github.com/repos/pydata/xarray/issues/5589,877268860,MDEyOklzc3VlQ29tbWVudDg3NzI2ODg2MA==,35968931,2021-07-09T15:27:24Z,2021-07-09T15:27:24Z,MEMBER,"We should be calling `.compute()` before passing anything to matplotlib - if we're not then that's a bug. I've also noticed that not doing so can cause huge slowdown. I think that #5561 might (or really should) fix this as a side effect though.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,940702754