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/2917#issuecomment-493115946,https://api.github.com/repos/pydata/xarray/issues/2917,493115946,MDEyOklzc3VlQ29tbWVudDQ5MzExNTk0Ng==,1217238,2019-05-16T15:28:41Z,2019-05-16T15:28:41Z,MEMBER,"LGTM, too. Thanks @dnowacki-usgs !","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,436999500
https://github.com/pydata/xarray/pull/2917#issuecomment-488403346,https://api.github.com/repos/pydata/xarray/issues/2917,488403346,MDEyOklzc3VlQ29tbWVudDQ4ODQwMzM0Ng==,1217238,2019-05-01T20:17:29Z,2019-05-01T20:17:37Z,MEMBER,"I think the dask argument is really an argument that `tutorial.open_dataset()` needs to exist. But if we're also including a `xarray.load_dataset()` function, then for consistency it makes sense to have both tutorial options, too.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,436999500
https://github.com/pydata/xarray/pull/2917#issuecomment-487827162,https://api.github.com/repos/pydata/xarray/issues/2917,487827162,MDEyOklzc3VlQ29tbWVudDQ4NzgyNzE2Mg==,1217238,2019-04-30T05:43:49Z,2019-04-30T05:44:21Z,MEMBER,"> Forgive me, I don't understand what you mean by download vs open. I see that `tutorial.load_dataset()` is different from this PR in that it keeps the file handle open instead of closing it. Is that the distinction you are referring to?

The difference I was referring to is that `xarray.tutorial.load_dataset()` needs to call `xarray.tutorial.open_dataset()` internally, not `xarray.open_dataset()`. But otherwise it can look exactly like `xarray.load_dataset`, e.g., it should be:
```python
# in tutorial.py
def load_dataset(*args, **kwargs):
    # this calls tutorial.open_dataset, not xarray.open_dataset
    with open_dataset(*args, **kwargs) as ds:
        return ds.load() 
```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,436999500
https://github.com/pydata/xarray/pull/2917#issuecomment-487722453,https://api.github.com/repos/pydata/xarray/issues/2917,487722453,MDEyOklzc3VlQ29tbWVudDQ4NzcyMjQ1Mw==,1217238,2019-04-29T20:02:10Z,2019-04-29T20:02:10Z,MEMBER,"> What if we just imported the new `load_dataset` as `xarray.tutorial.load_dataset` for the purposes of the tutorial?

`tutorial.load_dataset()` is a little different. It downloads the indicated file from GitHub to open instead of opening it directly.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,436999500
https://github.com/pydata/xarray/pull/2917#issuecomment-486747316,https://api.github.com/repos/pydata/xarray/issues/2917,486747316,MDEyOklzc3VlQ29tbWVudDQ4Njc0NzMxNg==,1217238,2019-04-25T16:36:52Z,2019-04-25T16:36:52Z,MEMBER,What do you think un-deprecating `xarray.tutoral.load_dataset` as part of this change? It might be useful to have that stick around for consistency.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,436999500