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
1306543211,PR_kwDOAMm_X847f3yo,6792,Raise an error if you pass an invalid key in `chunks`,4806877,closed,0,,,3,2022-07-15T21:23:20Z,2022-08-02T15:57:50Z,2022-07-22T16:52:32Z,CONTRIBUTOR,,0,pydata/xarray/pulls/6792,"- [x] Tests added
This was a minor issue that came up at the Dask BOF. Currently if the key of chunks dict isn't included in the dims then it gets silently ignored. This PR makes xarray raise an error instead.
I'm not sure if this is the right place to put this change. So just let me know if it should go somewhere else.
I changed an existing test. I am _pretty_ sure it was not intentionally using a key that isn't in the dims. ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6792/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1301237961,PR_kwDOAMm_X847OKqc,6774,Make the `sel` error more descriptive when `method` is unset,4806877,closed,0,,,1,2022-07-11T21:17:07Z,2022-07-13T14:49:24Z,2022-07-12T20:33:00Z,CONTRIBUTOR,,0,pydata/xarray/pulls/6774,"- [x] Tests added
This came up in the tutorial and I was wondering if the error could be made a little clearer. Not sure if the error message should hint that a user might want to use ``method``.
```python
import numpy as np
import pandas as pd
import xarray as xr
arr = xr.DataArray(
data=np.arange(48).reshape(4, 2, 6),
dims=(""u"", ""v"", ""time""),
coords={
""u"": [-3.2, 2.1, 5.3, 6.5],
""v"": [-1, 2.6],
""time"": pd.date_range(""2009-01-05"", periods=6, freq=""M""),
},
)
arr.sel(u=5, time=""2009-04-28"") # I removed `method=""nearest""`
```
Before this PR:
```python-traceback
...
KeyError: 5.0
```
After this PR:
```python-traceback
...
KeyError: ""not all values found in index 'u'. Did you mean to set `method=`?""
```
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6774/reactions"", ""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
392361367,MDExOlB1bGxSZXF1ZXN0MjM5NjUxOTU3,2618,Adding mask to open_rasterio,4806877,closed,0,,,17,2018-12-18T22:24:04Z,2021-06-24T13:44:33Z,2021-06-23T16:14:28Z,CONTRIBUTOR,,0,pydata/xarray/pulls/2618,"
- [x] Discussed in #1736
- [x] Tests added
- [ ] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API
Not sure if this is the right approach @snowman2","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/2618/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
573007307,MDExOlB1bGxSZXF1ZXN0MzgxNTk3Mzkw,3812,Turn on html repr by default,4806877,closed,0,,,6,2020-02-28T21:12:43Z,2020-03-26T02:19:22Z,2020-03-02T23:01:44Z,CONTRIBUTOR,,0,pydata/xarray/pulls/3812,"
- [x] Closes #3806
- [x] Passes `isort -rc . && 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/3812/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
512071129,MDExOlB1bGxSZXF1ZXN0MzMyMTUyMDg4,3443,jupyterlab dark theme,4806877,closed,0,,,11,2019-10-24T17:08:27Z,2019-10-29T03:47:28Z,2019-10-29T03:47:28Z,CONTRIBUTOR,,0,pydata/xarray/pulls/3443,"
- [ ] Closes #xxxx
- [ ] Tests added
- [x] Passes `black . && mypy . && flake8`
- [ ] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API
Follow on to #3425 to include support for jupyterlab dark theme. Note that this includes slight color changes. The most striking of which is that in jupyterlab light and regular notebook the even rows are white like the background.
## Jlab dark
## Jlab light
## notebook
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/3443/reactions"", ""total_count"": 4, ""+1"": 4, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
510294810,MDExOlB1bGxSZXF1ZXN0MzMwNjk0MDc5,3425,Html repr,4806877,closed,0,,,54,2019-10-21T21:08:54Z,2019-10-25T07:00:26Z,2019-10-24T16:48:47Z,CONTRIBUTOR,,0,pydata/xarray/pulls/3425,"
This PR supersedes #1820 - see that PR for original discussion. See [this gist](https://nbviewer.jupyter.org/gist/jsignell/2b7843a6e4852198bf963fbd299e8d46) to try out the new MultiIndex and options functionality.
- [x] Closes #1627, closes #1820
- [x] Tests added
- [x] Passes `black . && mypy . && flake8`
- [x] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API
TODO:
- [x] Add support for Multi-indexes
- [x] Probably good to have some opt-in or fail back system in case where we (or users) know that the rendering will not work
- [x] Add some tests
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/3425/reactions"", ""total_count"": 2, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 2, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
512162755,MDExOlB1bGxSZXF1ZXN0MzMyMjI2NzU0,3444,Escaping dtypes,4806877,closed,0,,,2,2019-10-24T20:24:33Z,2019-10-24T21:51:18Z,2019-10-24T21:50:20Z,CONTRIBUTOR,,0,pydata/xarray/pulls/3444,"
- [ ] Closes #xxxx
- [x] Tests added
- [x] Passes `black . && mypy . && flake8`
- [ ] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API
Follow-on to https://github.com/pydata/xarray/pull/3425 to make html_repr work with dtypes like '