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/3199#issuecomment-538431183,https://api.github.com/repos/pydata/xarray/issues/3199,538431183,MDEyOklzc3VlQ29tbWVudDUzODQzMTE4Mw==,6213168,2019-10-04T14:53:50Z,2019-10-04T14:53:50Z,MEMBER,"Hi, check out the work I did on the RTD requirements file here, it may help: https://github.com/pydata/xarray/pull/3358","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-538424892,https://api.github.com/repos/pydata/xarray/issues/3199,538424892,MDEyOklzc3VlQ29tbWVudDUzODQyNDg5Mg==,14808389,2019-10-04T14:38:49Z,2019-10-04T14:38:49Z,MEMBER,"looking at this once again, there are a lot of projects that don't install (eg [aiohttp](https://github.com/aio-libs/aiohttp) ([build](https://readthedocs.org/projects/aiohttp/builds/9751985/)), but I'm not sure how this is set up or which parts of their configuration comes from somewhere other than their `.readthedocs.yml`). Also, it seems we are using the [old version](https://docs.readthedocs.io/en/stable/config-file/v1.html) of [`readthedocs.yml`](https://docs.readthedocs.io/en/stable/config-file/v2.html). And finally, it seems strange to me that the `scipy19-docs` fails but `master` doesn't, which means that `master` does something different? As I don't have access to readthedocs I can't really experiment on these suggestions (making this informed guessing at best), so I would like to leave the tracking down and fixing to someone else.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-535224907,https://api.github.com/repos/pydata/xarray/issues/3199,535224907,MDEyOklzc3VlQ29tbWVudDUzNTIyNDkwNw==,14808389,2019-09-25T21:16:11Z,2019-09-25T22:50:41Z,MEMBER,"I'm currently not planning to because I will probably be offline for the next few days Edit: the only workaround I can imagine is building the docs without installing (which is mentioned in the docs), but I neither know why the current configuration is the way it is or how building the docs without installing works with RTD","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-535218487,https://api.github.com/repos/pydata/xarray/issues/3199,535218487,MDEyOklzc3VlQ29tbWVudDUzNTIxODQ4Nw==,1197350,2019-09-25T20:58:43Z,2019-09-25T20:58:43Z,MEMBER,"Is someone planning to open this RTD issue? Or is there another workaround? I would love to get this scipy19-docs branch merged to master, but this seems to be standing in the way.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534755294,https://api.github.com/repos/pydata/xarray/issues/3199,534755294,MDEyOklzc3VlQ29tbWVudDUzNDc1NTI5NA==,1217238,2019-09-24T21:26:04Z,2019-09-24T21:26:04Z,MEMBER,"Yikes, OK. Yes, it would be good to open an issue on the RTD GitHub page to discuss our options. --upgrade-strategy eager seems like a bad idea for us -- it really undermines the value of dependency pinning. On Tue, Sep 24, 2019 at 1:31 PM keewis wrote: > it seems all of these problems boil down to that single command. What gets > executed is (essentially) > > $ python -m pip install --upgrade --upgrade-strategy eager . > > which upgrades pandas to 0.25.1, but leaves dask as is because it is an > optional dependency. This, however, defeats the purpose of version pinning > in the conda file (or a requirements file). > > I can't find an option to tell readthedocs to install without the eager > upgrade (normal upgrade is fine). Maybe ask support? The addition of the > upgrade strategy happened in readthedocs/readthedocs.org#5635 > > > — > 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}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534735363,https://api.github.com/repos/pydata/xarray/issues/3199,534735363,MDEyOklzc3VlQ29tbWVudDUzNDczNTM2Mw==,14808389,2019-09-24T20:31:22Z,2019-09-24T20:31:22Z,MEMBER,"it seems all of these problems boil down to that single command. What gets executed is (essentially) ```bash $ python -m pip install --upgrade --upgrade-strategy eager . ``` which upgrades pandas to 0.25.1, but leaves dask as is because it is an optional dependency. This, however, defeats the purpose of version pinning in the conda file (or a requirements file). I can't find an option to tell readthedocs to install without the eager upgrade (normal upgrade is fine). Maybe ask support? The addition of the upgrade strategy happened in readthedocs/readthedocs.org#5635 ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534702846,https://api.github.com/repos/pydata/xarray/issues/3199,534702846,MDEyOklzc3VlQ29tbWVudDUzNDcwMjg0Ng==,2448579,2019-09-24T19:03:06Z,2019-09-24T19:03:06Z,MEMBER,"OK that almost worked. Now it looks like we need to bump pandas? or dask? ``` AttributeError Traceback (most recent call last) in ----> 1 df = ds.to_dask_dataframe() ~/checkouts/readthedocs.org/user_builds/xray/conda/scipy19-docs/lib/python3.7/site-packages/xarray/core/dataset.py in to_dask_dataframe(self, dim_order, set_index) 4278 series_list.append(series) 4279 -> 4280 df = dd.concat(series_list, axis=1) 4281 4282 if set_index: ~/checkouts/readthedocs.org/user_builds/xray/conda/scipy19-docs/lib/python3.7/site-packages/dask/dataframe/multi.py in concat(dfs, axis, join, interleave_partitions) 598 axis = DataFrame._validate_axis(axis) 599 dasks = [df for df in dfs if isinstance(df, _Frame)] --> 600 dfs = _maybe_from_pandas(dfs) 601 602 if axis == 1: ~/checkouts/readthedocs.org/user_builds/xray/conda/scipy19-docs/lib/python3.7/site-packages/dask/dataframe/core.py in _maybe_from_pandas(dfs) 3461 dfs = [from_pandas(df, 1) 3462 if (is_series_like(df) or is_dataframe_like(df)) and not is_dask_collection(df) -> 3463 else df for df in dfs] 3464 return dfs 3465 ~/checkouts/readthedocs.org/user_builds/xray/conda/scipy19-docs/lib/python3.7/site-packages/dask/dataframe/core.py in (.0) 3461 dfs = [from_pandas(df, 1) 3462 if (is_series_like(df) or is_dataframe_like(df)) and not is_dask_collection(df) -> 3463 else df for df in dfs] 3464 return dfs 3465 ~/checkouts/readthedocs.org/user_builds/xray/conda/scipy19-docs/lib/python3.7/site-packages/dask/dataframe/utils.py in is_series_like(s) 503 def is_series_like(s): 504 """""" Looks like a Pandas Series """""" --> 505 return set(dir(s)) > {'name', 'dtype', 'groupby', 'head'} 506 507 ~/checkouts/readthedocs.org/user_builds/xray/conda/scipy19-docs/lib/python3.7/site-packages/dask/dataframe/core.py in __dir__(self) 2554 o = set(dir(type(self))) 2555 o.update(self.__dict__) -> 2556 o.update(c for c in self.columns if 2557 (isinstance(c, pd.compat.string_types) and 2558 pd.compat.isidentifier(c))) ~/checkouts/readthedocs.org/user_builds/xray/conda/scipy19-docs/lib/python3.7/site-packages/dask/dataframe/core.py in (.0) 2555 o.update(self.__dict__) 2556 o.update(c for c in self.columns if -> 2557 (isinstance(c, pd.compat.string_types) and 2558 pd.compat.isidentifier(c))) 2559 return list(o) AttributeError: module 'pandas.compat' has no attribute 'string_types' <<<------------------------------------------------------------------------- ```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534678734,https://api.github.com/repos/pydata/xarray/issues/3199,534678734,MDEyOklzc3VlQ29tbWVudDUzNDY3ODczNA==,2448579,2019-09-24T18:05:11Z,2019-09-24T18:05:11Z,MEMBER,":+1: Let's see if it works: https://readthedocs.org/projects/xray/builds/9705929/","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534642189,https://api.github.com/repos/pydata/xarray/issues/3199,534642189,MDEyOklzc3VlQ29tbWVudDUzNDY0MjE4OQ==,14808389,2019-09-24T16:38:29Z,2019-09-24T16:38:59Z,MEMBER,"After some experimentation, it seems this is only the case if `setup.py install` is used, not with `pip install`: for me, at least, in the commands from https://github.com/pydata/xarray/pull/3199#issuecomment-534342306, replacing ```bash $ python setup.py install --force ``` with ```bash $ python -m pip install . ``` makes this work, even without the merge commit. The docs on the [settings](https://read-the-docs.readthedocs.io/en/latest/yaml-config.html#python-pip-install) mention `pip_install`, maybe use that?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534607562,https://api.github.com/repos/pydata/xarray/issues/3199,534607562,MDEyOklzc3VlQ29tbWVudDUzNDYwNzU2Mg==,14808389,2019-09-24T15:17:39Z,2019-09-24T16:08:24Z,MEMBER,"The problem is still the same: while the output of the environment printed by sphinx lists the new version as `xarray 0.13.0+22.gaeb15b56 pypi_0 pypi` which in theory should be used, but building the docs prints this: `xarray: 0.13.0, /home/docs/checkouts/readthedocs.org/user_builds/xray/conda/scipy19-docs/lib/python3.7/site-packages/xarray/__init__.py` which does not have the version of `open_dataset` that allows non-`nc` files. Honestly, I'm no expert on imports and version management, so I don't know what the best fix for this is. Something that comes to mind is waiting for a new release with the new version of `open_dataset`, but that is not an immediate fix (and only a workaround at best).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534607485,https://api.github.com/repos/pydata/xarray/issues/3199,534607485,MDEyOklzc3VlQ29tbWVudDUzNDYwNzQ4NQ==,2448579,2019-09-24T15:17:27Z,2019-09-24T15:17:27Z,MEMBER,still fails with 404: https://readthedocs.org/projects/xray/builds/9704981/,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534595817,https://api.github.com/repos/pydata/xarray/issues/3199,534595817,MDEyOklzc3VlQ29tbWVudDUzNDU5NTgxNw==,2448579,2019-09-24T14:52:01Z,2019-09-24T14:52:01Z,MEMBER,Thanks @keewis . I just merged master and pushed. hopefully this works,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534352985,https://api.github.com/repos/pydata/xarray/issues/3199,534352985,MDEyOklzc3VlQ29tbWVudDUzNDM1Mjk4NQ==,14808389,2019-09-24T01:50:42Z,2019-09-24T02:01:51Z,MEMBER,"err, well, this is somewhat confusing: I'm not sure what pulls it, but it seems something depends on xarray (installed in the creation of the environment). The version that is conda-installed is 0.13.0, so as master does not have the new version of `open_dataset` that does not enforce "".nc"" files, the 404 happens. In short, the version from `setup.py install` gets shadowed, I guess because the version is 0.12.3? In that case, merging master into scipy19-docs might fix this?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534343242,https://api.github.com/repos/pydata/xarray/issues/3199,534343242,MDEyOklzc3VlQ29tbWVudDUzNDM0MzI0Mg==,1217238,2019-09-24T01:03:38Z,2019-09-24T01:03:38Z,MEMBER,"Unfortunately, I don't think we can remove the second conda install. RTD insists on it to ensure that it has the build environment it needs. We could certainly ask on their GitHub support page, though. On Mon, Sep 23, 2019 at 5:59 PM keewis wrote: > (roughly) following the instructions in the build log, I can reproduce it > raising a 404: > > $ conda env create --quiet --name scipy19-docs --file doc/environment.yml > $ conda install --yes --quiet --name scipy19-docs mock pillow sphinx sphinx_rtd_theme > (scipy19-docs)$ python -m pip install -U recommonmark readthedocs-sphinx-ext > (scipy19-docs)$ python setup.py install --force > (scipy19-docs)$ cd doc; make html > > I didn't check yet, but maybe the second conda install should not happen, > as all those get installed on env creation? > > — > 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}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534342306,https://api.github.com/repos/pydata/xarray/issues/3199,534342306,MDEyOklzc3VlQ29tbWVudDUzNDM0MjMwNg==,14808389,2019-09-24T00:58:49Z,2019-09-24T00:58:49Z,MEMBER,"(roughly) following the instructions in the build log, I can reproduce it raising a 404: ```bash $ conda env create --quiet --name scipy19-docs --file doc/environment.yml $ conda install --yes --quiet --name scipy19-docs mock pillow sphinx sphinx_rtd_theme (scipy19-docs)$ python -m pip install -U recommonmark readthedocs-sphinx-ext (scipy19-docs)$ python setup.py install --force (scipy19-docs)$ cd doc; make html ``` I didn't check yet, but maybe the second `conda install` should not happen, as all those get installed on env creation?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534338729,https://api.github.com/repos/pydata/xarray/issues/3199,534338729,MDEyOklzc3VlQ29tbWVudDUzNDMzODcyOQ==,2448579,2019-09-24T00:39:45Z,2019-09-24T00:39:45Z,MEMBER,"![image](https://user-images.githubusercontent.com/2448579/65472596-bfa68280-de63-11e9-810d-e9db3e968896.png) ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534331505,https://api.github.com/repos/pydata/xarray/issues/3199,534331505,MDEyOklzc3VlQ29tbWVudDUzNDMzMTUwNQ==,1197350,2019-09-24T00:04:07Z,2019-09-24T00:04:07Z,MEMBER,"Unfortunately the RTD build failed! https://readthedocs.org/projects/xray/builds/9700942/ 😭😭😭 This is really trying my patience!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534310110,https://api.github.com/repos/pydata/xarray/issues/3199,534310110,MDEyOklzc3VlQ29tbWVudDUzNDMxMDExMA==,1197350,2019-09-23T22:30:36Z,2019-09-23T22:30:36Z,MEMBER,We got it! Thanks everyone!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534302715,https://api.github.com/repos/pydata/xarray/issues/3199,534302715,MDEyOklzc3VlQ29tbWVudDUzNDMwMjcxNQ==,2448579,2019-09-23T22:02:01Z,2019-09-23T22:02:01Z,MEMBER,Thanks @keewis ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534302656,https://api.github.com/repos/pydata/xarray/issues/3199,534302656,MDEyOklzc3VlQ29tbWVudDUzNDMwMjY1Ng==,2448579,2019-09-23T22:01:48Z,2019-09-23T22:01:48Z,MEMBER,:man_facepalming: finally!,"{""total_count"": 3, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 3, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534301971,https://api.github.com/repos/pydata/xarray/issues/3199,534301971,MDEyOklzc3VlQ29tbWVudDUzNDMwMTk3MQ==,14808389,2019-09-23T21:59:35Z,2019-09-23T21:59:35Z,MEMBER,"that seems to have worked, the failures should be unrelated.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534299772,https://api.github.com/repos/pydata/xarray/issues/3199,534299772,MDEyOklzc3VlQ29tbWVudDUzNDI5OTc3Mg==,2448579,2019-09-23T21:52:05Z,2019-09-23T21:52:05Z,MEMBER,I removed the newline upstream.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534297108,https://api.github.com/repos/pydata/xarray/issues/3199,534297108,MDEyOklzc3VlQ29tbWVudDUzNDI5NzEwOA==,14808389,2019-09-23T21:42:56Z,2019-09-23T21:42:56Z,MEMBER,"unless something else is wrong, either modifying the md5 file (remove the newline) or adding `strip()` to `f.read()` in `tutorial.open_dataset` should fix the md5 check.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534266780,https://api.github.com/repos/pydata/xarray/issues/3199,534266780,MDEyOklzc3VlQ29tbWVudDUzNDI2Njc4MA==,1197350,2019-09-23T20:18:25Z,2019-09-23T20:18:25Z,MEMBER,"Still failing. Instead of ``` ds = xr.tutorial.load_dataset('data/era5-2mt-2019-03-uk.grib', engine='cfgrib') ``` it should be ``` ds = xr.tutorial.load_dataset('era5-2mt-2019-03-uk.grib', engine='cfgrib') ```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-519979952,https://api.github.com/repos/pydata/xarray/issues/3199,519979952,MDEyOklzc3VlQ29tbWVudDUxOTk3OTk1Mg==,22429695,2019-08-09T16:21:37Z,2019-09-23T19:14:16Z,NONE,"# [Codecov](https://codecov.io/gh/pydata/xarray/pull/3199?src=pr&el=h1) Report > Merging [#3199](https://codecov.io/gh/pydata/xarray/pull/3199?src=pr&el=desc) into [scipy19-docs](https://codecov.io/gh/pydata/xarray/commit/f7c12a9a2c6bd3ec286574558add718798fecc0d?src=pr&el=desc) will **decrease** coverage by `0.35%`. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## scipy19-docs #3199 +/- ## ================================================ - Coverage 95.76% 95.41% -0.36% ================================================ Files 64 64 Lines 13188 13188 ================================================ - Hits 12630 12583 -47 - Misses 558 605 +47 ``` ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534240540,https://api.github.com/repos/pydata/xarray/issues/3199,534240540,MDEyOklzc3VlQ29tbWVudDUzNDI0MDU0MA==,29253833,2019-09-23T19:07:40Z,2019-09-23T19:07:40Z,NONE,"Ah, oops. Updated to use that method now.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534238685,https://api.github.com/repos/pydata/xarray/issues/3199,534238685,MDEyOklzc3VlQ29tbWVudDUzNDIzODY4NQ==,2448579,2019-09-23T19:02:58Z,2019-09-23T19:02:58Z,MEMBER,Oops I think you lost ryan's commit with `xr.tutorial.load_dataset`,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534237310,https://api.github.com/repos/pydata/xarray/issues/3199,534237310,MDEyOklzc3VlQ29tbWVudDUzNDIzNzMxMA==,29253833,2019-09-23T18:59:41Z,2019-09-23T18:59:41Z,NONE,Rebased on scipy19-docs. Hope that helps!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-534234559,https://api.github.com/repos/pydata/xarray/issues/3199,534234559,MDEyOklzc3VlQ29tbWVudDUzNDIzNDU1OQ==,2448579,2019-09-23T18:52:45Z,2019-09-23T18:52:45Z,MEMBER,@zbruick Can you merge in scipy19-docs? That might fix it.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-532302896,https://api.github.com/repos/pydata/xarray/issues/3199,532302896,MDEyOklzc3VlQ29tbWVudDUzMjMwMjg5Ng==,14808389,2019-09-17T16:43:20Z,2019-09-17T17:45:52Z,MEMBER,"I'm not sure if this helps you, but I noticed two issues in `xr.tutorial.open_dataset`: first, it requires the file extension to be "".nc"" (resulting in a 404 when fetching the file. The version from the scipy19-docs branch does not have that issue), and second, there is a newline in the md5 file. Modifying `remotemd5 = f.read()` to `remotemd5 = f.read().strip()` should do the trick.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-532280765,https://api.github.com/repos/pydata/xarray/issues/3199,532280765,MDEyOklzc3VlQ29tbWVudDUzMjI4MDc2NQ==,1197350,2019-09-17T15:46:47Z,2019-09-17T15:46:47Z,MEMBER,"I am really stumped here. When I run ```python xr.tutorial.load_dataset('era5-2mt-2019-03-uk.grib', engine='cfgrib') ``` on the scipy19-docs branch, it works fine.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-532263787,https://api.github.com/repos/pydata/xarray/issues/3199,532263787,MDEyOklzc3VlQ29tbWVudDUzMjI2Mzc4Nw==,1197350,2019-09-17T15:07:11Z,2019-09-17T15:07:19Z,MEMBER,Can someone with the proper credentials (e.g. @shoyer) restart the azure build? I can't reproduce the md5 error locally.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-530407152,https://api.github.com/repos/pydata/xarray/issues/3199,530407152,MDEyOklzc3VlQ29tbWVudDUzMDQwNzE1Mg==,29253833,2019-09-11T14:30:02Z,2019-09-11T14:30:02Z,NONE,"Thanks for pushing that commit and updating the data import call. I forgot I had to switch that from my local file. Let me know if this issue is solvable on my side of things, but that seems like that's an issue with the data file? ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-530189455,https://api.github.com/repos/pydata/xarray/issues/3199,530189455,MDEyOklzc3VlQ29tbWVudDUzMDE4OTQ1NQ==,1197350,2019-09-11T02:10:43Z,2019-09-11T02:11:00Z,MEMBER,"Now we are getting `MD5 checksum does not match, try downloading dataset again.` Doesn't make sense to me. https://dev.azure.com/xarray/xarray/_build/results?buildId=780","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-530183810,https://api.github.com/repos/pydata/xarray/issues/3199,530183810,MDEyOklzc3VlQ29tbWVudDUzMDE4MzgxMA==,1197350,2019-09-11T01:42:32Z,2019-09-11T01:42:32Z,MEMBER,I just fixed a file naming issue in https://github.com/pydata/xarray-data/pull/18 and can confirm that my code above works for me.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-530182624,https://api.github.com/repos/pydata/xarray/issues/3199,530182624,MDEyOklzc3VlQ29tbWVudDUzMDE4MjYyNA==,1197350,2019-09-11T01:36:23Z,2019-09-11T01:36:23Z,MEMBER,"It looks like the [doc build](https://dev.azure.com/xarray/xarray/_build/results?buildId=779) is failing with this error ``` Sphinx parallel build error: nbsphinx.NotebookError: CellExecutionError in examples/ERA5-GRIB-example.ipynb: ------------------ ds = xr.load_dataset('data/era5-2mt-2019-03-uk.grib', engine='cfgrib') ------------------ ---------------------------------------------------------------------------‌ FileNotFoundError‌ Traceback (most recent call last)‌ <ipython-input-2-90ecf53ec89c>‌ in ‌<module>‌ ----> 1‌ ‌ds‌ ‌=‌ ‌xr‌.‌load_dataset‌(‌'data/era5-2mt-2019-03-uk.grib'‌,‌ ‌engine‌=‌'cfgrib'‌)‌ ~/work/1/s/xarray/backends/api.py‌ in ‌load_dataset‌(filename_or_obj, **kwargs)‌ 256‌ ‌raise‌ ‌TypeError‌(‌"cache has no effect in this context"‌)‌ 257‌ ‌ --> 258‌ ‌with‌ ‌open_dataset‌(‌filename_or_obj‌,‌ ‌**‌kwargs‌)‌ ‌as‌ ‌ds‌:‌ 259‌ ‌return‌ ‌ds‌.‌load‌(‌)‌ 260‌ ‌ ~/work/1/s/xarray/backends/api.py‌ in ‌open_dataset‌(filename_or_obj, group, decode_cf, mask_and_scale, decode_times, autoclose, concat_characters, decode_coords, engine, chunks, lock, cache, drop_variables, backend_kwargs, use_cftime)‌ 516‌ ‌elif‌ ‌engine‌ ‌==‌ ‌"cfgrib"‌:‌ 517‌ store = backends.CfGribDataStore(‌ --> 518‌ ‌filename_or_obj‌,‌ ‌lock‌=‌lock‌,‌ ‌**‌backend_kwargs‌ 519‌ )‌ 520‌ ‌ ~/work/1/s/xarray/backends/cfgrib_.py‌ in ‌__init__‌(self, filename, lock, **backend_kwargs)‌ 41‌ ‌lock‌ ‌=‌ ‌ECCODES_LOCK‌ 42‌ ‌self‌.‌lock‌ ‌=‌ ‌ensure_lock‌(‌lock‌)‌ ---> 43‌ ‌self‌.‌ds‌ ‌=‌ ‌cfgrib‌.‌open_file‌(‌filename‌,‌ ‌**‌backend_kwargs‌)‌ 44‌ ‌ 45‌ ‌def‌ ‌open_store_variable‌(‌self‌,‌ ‌name‌,‌ ‌var‌)‌:‌ /usr/share/miniconda/envs/xarray-tests/lib/python3.7/site-packages/cfgrib/dataset.py‌ in ‌open_file‌(path, grib_errors, indexpath, filter_by_keys, **kwargs)‌ 601‌ ):‌ 602‌ ‌"""Open a GRIB file as a ``cfgrib.Dataset``."""‌ --> 603‌ ‌index‌ ‌=‌ ‌open_fileindex‌(‌path‌,‌ ‌grib_errors‌,‌ ‌indexpath‌,‌ ‌filter_by_keys‌)‌ 604‌ ‌return‌ ‌Dataset‌(‌*‌build_dataset_components‌(‌index‌,‌ ‌**‌kwargs‌)‌)‌ /usr/share/miniconda/envs/xarray-tests/lib/python3.7/site-packages/cfgrib/dataset.py‌ in ‌open_fileindex‌(path, grib_errors, indexpath, filter_by_keys)‌ 594‌ ‌filter_by_keys‌ ‌=‌ ‌dict‌(‌filter_by_keys‌)‌ 595‌ ‌stream‌ ‌=‌ ‌messages‌.‌FileStream‌(‌path‌,‌ ‌message_class‌=‌cfmessage‌.‌CfMessage‌,‌ ‌errors‌=‌grib_errors‌)‌ --> 596‌ ‌return‌ ‌stream‌.‌index‌(‌ALL_KEYS‌,‌ ‌indexpath‌=‌indexpath‌)‌.‌subindex‌(‌filter_by_keys‌)‌ 597‌ ‌ 598‌ ‌ /usr/share/miniconda/envs/xarray-tests/lib/python3.7/site-packages/cfgrib/messages.py‌ in ‌index‌(self, index_keys, indexpath)‌ 235‌ ‌def‌ ‌index‌(‌self‌,‌ ‌index_keys‌,‌ ‌indexpath‌=‌'{path}.{short_hash}.idx'‌)‌:‌ 236‌ ‌# type: (T.List[str], str) -> FileIndex‌ --> 237‌ ‌return‌ ‌FileIndex‌.‌from_indexpath_or_filestream‌(‌self‌,‌ ‌index_keys‌,‌ ‌indexpath‌)‌ 238‌ ‌ 239‌ ‌ /usr/share/miniconda/envs/xarray-tests/lib/python3.7/site-packages/cfgrib/messages.py‌ in ‌from_indexpath_or_filestream‌(cls, filestream, index_keys, indexpath, log)‌ 331‌ ‌log‌.‌exception‌(‌"Can't read index file %r"‌,‌ ‌indexpath‌)‌ 332‌ ‌ --> 333‌ ‌return‌ ‌cls‌.‌from_filestream‌(‌filestream‌,‌ ‌index_keys‌)‌ 334‌ ‌ 335‌ ‌def‌ ‌__iter__‌(‌self‌)‌:‌ /usr/share/miniconda/envs/xarray-tests/lib/python3.7/site-packages/cfgrib/messages.py‌ in ‌from_filestream‌(cls, filestream, index_keys)‌ 262‌ ‌offsets‌ ‌=‌ ‌collections‌.‌OrderedDict‌(‌)‌ 263‌ ‌count_offsets‌ ‌=‌ ‌{‌}‌ ‌# type: T.Dict[int, int]‌ --> 264‌ ‌for‌ ‌message‌ ‌in‌ ‌filestream‌:‌ 265‌ ‌header_values‌ ‌=‌ ‌[‌]‌ 266‌ ‌for‌ ‌key‌ ‌in‌ ‌index_keys‌:‌ /usr/share/miniconda/envs/xarray-tests/lib/python3.7/site-packages/cfgrib/messages.py‌ in ‌__iter__‌(self)‌ 208‌ ‌def‌ ‌__iter__‌(‌self‌)‌:‌ 209‌ ‌# type: () -> T.Generator[Message, None, None]‌ --> 210‌ ‌with‌ ‌open‌(‌self‌.‌path‌,‌ ‌'rb'‌)‌ ‌as‌ ‌file‌:‌ 211‌ ‌valid_message_found‌ ‌=‌ ‌False‌ 212‌ ‌while‌ ‌True‌:‌ FileNotFoundError‌: [Errno 2] No such file or directory: '/home/vsts/work/1/s/doc/examples/data/era5-2mt-2019-03-uk.grib'‌ FileNotFoundError: [Errno 2] No such file or directory: '/home/vsts/work/1/s/doc/examples/data/era5-2mt-2019-03-uk.grib' ``` It appears that you are loading the data from a local file, rather than from the tutorial datasets api. This won't work, because the local file is not available in the build environment. That's why we added it to the tutorial repo in https://github.com/pydata/xarray-data/pull/14. Can you try something like ```python ds = xr.tutorial.load_dataset('era5-2mt-2019-03-uk.grib', engine='cfgrib') ``` ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-530125352,https://api.github.com/repos/pydata/xarray/issues/3199,530125352,MDEyOklzc3VlQ29tbWVudDUzMDEyNTM1Mg==,29253833,2019-09-10T21:20:55Z,2019-09-10T21:20:55Z,NONE,Happy to add some markdown to this - that was an oversight. I'll work on that and push it up shortly.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-530112691,https://api.github.com/repos/pydata/xarray/issues/3199,530112691,MDEyOklzc3VlQ29tbWVudDUzMDExMjY5MQ==,1197350,2019-09-10T20:47:44Z,2019-09-10T20:47:44Z,MEMBER,"In its current form, I'd say that your example is missing some text (markdown cells) which provide a narrative. See this example - http://xarray.pydata.org/en/scipy19-docs/examples/ROMS_ocean_model.html - for what I am talking about.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574 https://github.com/pydata/xarray/pull/3199#issuecomment-530112177,https://api.github.com/repos/pydata/xarray/issues/3199,530112177,MDEyOklzc3VlQ29tbWVudDUzMDExMjE3Nw==,1197350,2019-09-10T20:46:21Z,2019-09-10T20:46:21Z,MEMBER,"Hi @zbruick - thanks for this and sorry for not following up sooner. Do you feel this example is ready for review?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,479063574