id,node_id,number,title,user,state,locked,assignee,milestone,comments,created_at,updated_at,closed_at,author_association,active_lock_reason,draft,pull_request,body,reactions,performed_via_github_app,state_reason,repo,type
626083981,MDExOlB1bGxSZXF1ZXN0NDI0MTUwNjA0,4102,cache rasterio example files,14808389,closed,0,,,17,2020-05-27T22:43:04Z,2021-03-24T17:58:48Z,2021-03-24T17:56:49Z,MEMBER,,0,pydata/xarray/pulls/4102,"To make building the documentation a bit faster (by 5-10 minutes for me), this adds a `xr.tutorial.open_rasterio` function with a signature and behavior that is almost identical to `tutorial.open_dataset` (when we replace the `open_*` functions with `open_dataset(..., format=""..."")` we should do that for the tutorial functions, too).

It uses `requests.get` instead of [`urllib.request.urlretrieve`](https://docs.python.org/3/library/urllib.request.html#urllib.request.urlretrieve) so that would be a new dependency. I'm not sure if that's an issue since it's installed in the `bare-minimum` CI's environment.

The `tutorial.open_dataset` code could be rewritten to use the same structure but I wanted to get feedback on `open_rasterio` first.

Edit: I also changed the default cache directory to `~/.cache/xarray_tutorial_data` with a fallback to the old default if `~/.cache` does not exist

 - [x] Closes #3986
 - [x] Tests added (but that's just a comparison between cached and uncached)
 - [x] Passes `isort -rc . && black . && mypy . && flake8`
 - [ ] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/4102/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
443157666,MDExOlB1bGxSZXF1ZXN0Mjc4MDg4MTA1,2956,Picking up #1118: Do not convert subclasses of `ndarray` unless required,14808389,closed,0,,,17,2019-05-12T23:12:43Z,2019-12-25T14:14:00Z,2019-12-25T14:11:45Z,MEMBER,,0,pydata/xarray/pulls/2956,"This picks up the work done in #1118, plus converting the `unittest`
based tests to `pytest` (since that seems to be used in the other
tests).

closes #1118

Some of the suggestions there are not implemented yet:
* the disabling of bottleneck if a `ndarray` subclass is used
* testing for operations like `xarray.concat`. This does not work with any of the unit libraries (at least, `pint`, `astropy.units`, `unyt` and `quantities` don't support it yet).
* testing with other unit libraries that use subclasses (e.g. `astropy.units` or `unyt`)
* renaming the test file to `test_units_subclass.py`
* fix indexing (`NumpyIndexingAdapter`)

The requirements files for CI changed since then, so maybe more than
the two right now need the `quantities` entry.","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/2956/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
522523336,MDExOlB1bGxSZXF1ZXN0MzQwNzA5MjI0,3527,Add DatasetGroupBy.quantile,14808389,closed,0,,,17,2019-11-13T23:03:07Z,2019-11-15T23:55:14Z,2019-11-15T19:58:02Z,MEMBER,,0,pydata/xarray/pulls/3527,"This adds `DatasetGroupBy.quantile` by moving `DataArrayGroupBy.quantile` to `GroupBy` as proposed in https://github.com/pydata/xarray/issues/3018#issuecomment-507749287.

The tests are a modified copy of the ones from #2828. What confuses me is that `expected_yy` in `test_ds_groupby_quantile` needs the `transpose` whereas the equivalent in `test_da_groupby_quantile` doesn't. Does anyone have an idea about why that is?

 - [x] Closes #3525
 - [x] Tests added
 - [x] Passes `black . && mypy . && flake8`
 - [x] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/3527/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull