issue_comments

Looks good to me!

Lmk if that was an "OK to merge" @shoyer ! Thanks

Ready to merge!

This is ready for review, I'll add a whatsnew depending on whether we decide to do the same for GroupBy

I think docs stopped building and just froze (so timed out rather than erroring), and so the error wasn't reported up to Azure Pipelines. I got this from the log:

```

---

Warning in /home/vsts/work/1/s/doc/computation.rst at block ending on line 472 Specify :okwarning: as an option in the ipython:: block to suppress this message

/usr/share/miniconda/envs/xarray-tests/bin/sphinx-build:1: DeprecationWarning: Dataset.apply is deprecated in favor of Dataset.map and will be changed or removed in a future version of xarray #!/usr/share/miniconda/envs/xarray-tests/bin/python3.7 <<<------------------------------------------------------------------------- /home/vsts/work/1/s/xarray/core/dataarray.py:docstring of xarray.DataArray.integrate:12: WARNING: Unexpected indentation. /home/vsts/work/1/s/xarray/core/dataarray.py:docstring of xarray.DataArray.interp:20: WARNING: Inline strong start-string without end-string. /home/vsts/work/1/s/xarray/core/dataarray.py:docstring of xarray.DataArray.interpolate_na:8: WARNING: Definition list ends without a blank line; unexpected unindent.

Sphinx parallel build error: RuntimeError: Non Expected warning in /home/vsts/work/1/s/doc/computation.rst line 472 ```

I see this in the GH Checks page (but, again, no test runs failed, apparently)

```python self = <xarray.tests.test_backends.TestPydapOnline object at 0x7fd9047f8518>

def test_cmp_local_file(self):

  with self.create_datasets() as (actual, expected):

xarray/tests/test_backends.py:3137:

/usr/share/miniconda/envs/xarray-tests/lib/python3.6/contextlib.py:81: in enter return next(self.gen) xarray/tests/test_backends.py:3191: in create_datasets actual = open_dataset(url, engine="pydap", kwargs) xarray/backends/api.py:504: in open_dataset store = backends.PydapDataStore.open(filename_or_obj, backend_kwargs) xarray/backends/pydap_.py:78: in open ds = pydap.client.open_url(url, session=session) /usr/share/miniconda/envs/xarray-tests/lib/python3.6/site-packages/pydap/client.py:67: in open_url timeout).dataset /usr/share/miniconda/envs/xarray-tests/lib/python3.6/site-packages/pydap/handlers/dap.py:54: in init raise_for_status(r)

response = <Response at 0x7fd9045ac588 500 Internal Server Error>

def raise_for_status(response):
    # Raise error if status is above 300:
    if response.status_code >= 300:
        raise HTTPError(
            detail=response.status+'\n'+response.text,
            headers=response.headers,

          comment=response.body
        )

E webob.exc.HTTPError: 500 Internal Server Error E Error { E code = 500; E message = "Problem encountered with BES connection. Message: 'Failed to write to socket: Broken pipe (Write failed)' OPeNDAPClient executed 131 prior commands."; E }

/usr/share/miniconda/envs/xarray-tests/lib/python3.6/site-packages/pydap/net.py:39: HTTPError ```

I'm getting bemuisng test failures here! All the tests apart from docs say they pass, Docs says Cancelled after 60m. There are no explicit changes to docs here, and I can't imagine any implicit changes. If anyone has any bright ideas, help is welcome!