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/2277#issuecomment-405081849,https://api.github.com/repos/pydata/xarray/issues/2277,405081849,MDEyOklzc3VlQ29tbWVudDQwNTA4MTg0OQ==,1197350,2018-07-15T10:31:31Z,2018-07-15T10:31:31Z,MEMBER,"This seems like a very cool and useful feature! A few comments:

- This should probably live under the `.plot` namespace, i.e. `da.plot.scatter` rather than `da.scatter`.
- You should try to emulate the pandas [scatter api](https://pandas.pydata.org/pandas-docs/version/0.23/generated/pandas.DataFrame.plot.scatter.html) as much as possible (which itself emulates the matplotlib api). That means using the `c` keyword instead of `hue` and also implementing `s` for size

Happy to provide a more detailed review once the tests are implemented. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,340069538