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
2196272235,PR_kwDOAMm_X85qKODl,8856,Migrate indexing and broadcasting logic to `xarray.namedarray` (Part 1),13301940,open,0,,,0,2024-03-19T23:51:46Z,2024-05-03T17:08:11Z,,MEMBER,,1,pydata/xarray/pulls/8856,"
This pull request is the first part of migrating the indexing and broadcasting logic from `xarray.core.variable` to `xarray.namedarray`. I intend to open follow-up pull requests to address additional changes related to this refactoring, as outlined in the [proposal for decoupling lazy indexing functionality from NamedArray](https://github.com/pydata/xarray/blob/main/design_notes/named_array_design_doc.md#plan-for-decoupling-lazy-indexing-functionality-from-namedarray).
- [ ] Closes #xxxx
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8856/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2203689075,PR_kwDOAMm_X85qjXJq,8870,Enable explicit use of key tuples (instead of *Indexer objects) in indexing adapters and explicitly indexed arrays,13301940,closed,0,,,1,2024-03-23T04:34:18Z,2024-05-03T15:27:38Z,2024-05-03T15:27:22Z,MEMBER,,0,pydata/xarray/pulls/8870,"
- [ ] Towards #8856
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8870/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2212435865,PR_kwDOAMm_X85rAwYu,8885,add `.oindex` and `.vindex` to `BackendArray`,13301940,closed,0,,,8,2024-03-28T06:14:43Z,2024-04-30T12:12:50Z,2024-04-17T01:53:23Z,MEMBER,,0,pydata/xarray/pulls/8885,"
this PR builds towards
- https://github.com/pydata/xarray/pull/8870
- https://github.com/pydata/xarray/pull/8856
the primary objective is to partially address
> 2. Implement fall back `.oindex`, `.vindex` properties on `BackendArray` base class. These will simply rewrap the `key` tuple with the appropriate `*Indexer` object, and pass it on to `__getitem__` or `__setitem__`. These methods will also raise DeprecationWarning so that external backends will know to migrate to `.oindex`, and `.vindex` over the next year.
----
- [ ] Closes #xxxx
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8885/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2231711080,PR_kwDOAMm_X85sCbN-,8921,"Revert `.oindex` and `.vindex` additions in `_ElementwiseFunctionArray`, `NativeEndiannessArray`, and `BoolTypeArray` classes",13301940,open,0,,,9,2024-04-08T17:11:08Z,2024-04-30T06:49:46Z,,MEMBER,,0,pydata/xarray/pulls/8921,"
As noted in https://github.com/pydata/xarray/issues/8909, the use of `.oindex` and `.vindex` properties in coding/* appears to have broken some backends (e.g. scipy). This PR reverts those changes. We plan to bundle these changes into a separate backends feature branch (see [this comment](https://github.com/pydata/xarray/pull/8885#issuecomment-2036001828), which will be merged once we are confident about its impact on downstream dependencies.
- [ ] Closes #8909
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8921/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2203546866,PR_kwDOAMm_X85qi4jp,8869,Handle .oindex and .vindex for the PandasMultiIndexingAdapter and PandasIndexingAdapter,13301940,closed,0,,,0,2024-03-23T00:10:50Z,2024-03-23T02:30:46Z,2024-03-23T02:30:44Z,MEMBER,,0,pydata/xarray/pulls/8869,"
- [ ] Closes #xxxx
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8869/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2196627659,PR_kwDOAMm_X85qLGOs,8857,increase typing annotations coverage in `xarray/core/indexing.py`,13301940,closed,0,,,0,2024-03-20T04:08:12Z,2024-03-21T01:50:03Z,2024-03-20T20:20:52Z,MEMBER,,0,pydata/xarray/pulls/8857,"
- [ ] Towards #8856
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8857/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2189661150,PR_kwDOAMm_X85pzxuX,8845,Implement setitem syntax for `.oindex` and `.vindex` properties,13301940,closed,0,,,3,2024-03-16T00:40:46Z,2024-03-19T21:01:45Z,2024-03-19T03:29:58Z,MEMBER,,0,pydata/xarray/pulls/8845,"this is another follow-up to previous Prs
- #8780
- #8750
- #8790
, continuing the proposal outlined in [plan for decoupling lazy indexing functionality from NamedArray](https://github.com/pydata/xarray/blob/main/design_notes/named_array_design_doc.md#plan-for-decoupling-lazy-indexing-functionality-from-namedarray)
Cc @maxrjones for visibility","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8845/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1958139851,PR_kwDOAMm_X85dlQp4,8364,Migrate from *Indexer objects to `.oindex` and `.vindex` on explicityIndexed array classes,13301940,closed,0,,,1,2023-10-23T22:41:35Z,2024-03-15T23:28:48Z,2024-02-14T23:55:06Z,MEMBER,,1,pydata/xarray/pulls/8364,"
- [ ] Closes #xxxx
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8364/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2157865512,PR_kwDOAMm_X85oHSTV,8790,Expand use of `.oindex` and `.vindex`,13301940,closed,0,,,4,2024-02-28T00:47:00Z,2024-03-15T23:28:23Z,2024-03-15T05:00:04Z,MEMBER,,0,pydata/xarray/pulls/8790,"this is a follow-up to previous PRs (#8780 and #8750), continuing the efforts outlined in the [plan for decoupling lazy indexing functionality from NamedArray](https://github.com/pydata/xarray/blob/main/design_notes/named_array_design_doc.md#plan-for-decoupling-lazy-indexing-functionality-from-namedarray). the primary focus of this PR is the removal of vectorized and orthogonal indexing logic from the `__getitem__` method. Now, `__getitem__` exclusively handles basic indexers, aligning with points 3 and 4 of the plan:
- as per point 3, lazy indexing arrays will now implement `__getitem__` solely for basic indexing. For orthogonal indexing, .oindex will be used, and for vectorized indexing, .vindex will be utilized.
- following point 4, IndexingAdapter classes have been updated to consistently implement `__getitem__`, `.oindex`, and `.vindex`
- [ ] Closes #xxxx
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8790/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2187291438,PR_kwDOAMm_X85prlwg,8835,[skip-ci] Add dask-expr dependency to doc.yml,13301940,closed,0,,,0,2024-03-14T21:23:24Z,2024-03-14T21:44:18Z,2024-03-14T21:44:14Z,MEMBER,,0,pydata/xarray/pulls/8835,"- #8827 follow up to fix https://readthedocs.org/projects/xray/builds/23743522/
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8835/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2151987948,PR_kwDOAMm_X85nzdLf,8780,introduce `.vindex` property for Explicitly Indexed Arrays,13301940,closed,0,,,0,2024-02-24T00:27:11Z,2024-02-26T21:13:10Z,2024-02-26T21:11:32Z,MEMBER,,0,pydata/xarray/pulls/8780,"
- [ ] #8750 follow up
- [ ] Tests added
- [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8780/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2135390562,PR_kwDOAMm_X85m6nof,8750,refactor `indexing.py`: introduce `.oindex` for Explicitly Indexed Arrays,13301940,closed,0,,,2,2024-02-14T23:53:53Z,2024-02-23T23:34:44Z,2024-02-23T23:34:43Z,MEMBER,,0,pydata/xarray/pulls/8750,"
this pull request represents an initial effort to refactor the indexing logic within `indexing.py` by separating it from `variable.py`. to enhance the flexibility of explicitly indexed array classes, this PR introduces orthogonal ~~and vectorized~~ indexing methods—`.oindex[key]` ~~and `.vindex[key[`~~. these changes are a part of a broader initiative aimed at enabling indexing support for namedarray
- [x] Towards https://github.com/pydata/xarray/issues/8238
- [ ] Tests added
- [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
edit 1:
- to minimize the blast radius of this PR, i decided to just focus on `oindex[key]` method in this PR. i intend to introduce `vindex[key]` in a separate PR
## TODO:
- [x] https://github.com/pydata/xarray/pull/8750#discussion_r1491169460
- [ ] Enable passing the underlying tuple key","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8750/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2137484840,PR_kwDOAMm_X85nBx-a,8758,refactor DaskIndexingAdapter `__getitem__` method,13301940,closed,0,,,0,2024-02-15T21:31:33Z,2024-02-15T22:50:51Z,2024-02-15T22:44:17Z,MEMBER,,0,pydata/xarray/pulls/8758,"
this pull request removes the [""unnecessary optimization""](https://github.com/pydata/xarray/issues/8377#issuecomment-1781406355) introduced in https://github.com/pydata/xarray/pull/3588 that attempts to rewrite the indexer key for `BasicIndexer` and `OuterIndexer`
- [x] Closes #8377
- [ ] Tests added
- [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8758/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1852260723,PR_kwDOAMm_X85YA1UQ,8073,"add design document for ""named-array""",13301940,closed,0,,,2,2023-08-15T23:02:36Z,2024-02-12T22:22:33Z,2023-08-17T16:22:48Z,MEMBER,,0,pydata/xarray/pulls/8073,"This PR contains the design document for ""named-array"" a proposed new package aimed at providing a lightweight, efficient array structure with named dimensions. the main objective of named-array is to provide a more accessible version of Xarray's Variable class without the heavy dependencies like Pandas.
~~## proposed name~~
~~The ""Xarray-lite"" name is a placeholder for the time being. we have considered two main candidates for the package's name: ""narray"" and ""yarray."" feedback on the proposed names or other suggestions is welcome.~~
Related issues:
- https://github.com/pydata/xarray/issues/5081
- https://github.com/pydata/xarray/issues/3981","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8073/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1918009494,PR_kwDOAMm_X85beIFo,8250,copy the `dtypes` module to the `namedarray` package.,13301940,closed,0,,,3,2023-09-28T17:52:50Z,2024-02-12T22:22:24Z,2023-10-04T18:14:24Z,MEMBER,,0,pydata/xarray/pulls/8250,"
this PR copies the `dtypes` module from the `xarray.core` package to the `xarray.namedarray` package. It also recreates the `ReprObject` class from the `utils` module within the `xarray.namedarray` package
These changes are part of the ongoing work in:
- https://github.com/pydata/xarray/pull/8244
---
- [ ] towards #8238
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8250/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2021745010,PR_kwDOAMm_X85g8efA,8505,roll out the new/refreshed Xarray logo,13301940,closed,0,,,0,2023-12-01T23:53:18Z,2024-02-12T22:22:22Z,2023-12-02T22:42:17Z,MEMBER,,0,pydata/xarray/pulls/8505,"
- ref: https://github.com/xarray-contrib/xarray.dev/issues/66#issuecomment-1664561089
Cc @jhamman
really like how good these look on both light/dark backgrounds:


","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8505/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2106574555,PR_kwDOAMm_X85lYkg9,8682,Fix CI: temporary pin pytest version to 7.4.*,13301940,closed,0,,,1,2024-01-29T22:48:56Z,2024-01-29T23:19:53Z,2024-01-29T23:09:44Z,MEMBER,,0,pydata/xarray/pulls/8682,"
- [ ] towards fixing #8681
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8682/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1964505426,PR_kwDOAMm_X85d61au,8380,[namedarray] split `.set_dims()` into `.expand_dims()` and `broadcast_to()`,13301940,closed,0,,,17,2023-10-26T23:13:29Z,2024-01-29T19:25:58Z,2024-01-29T19:13:28Z,MEMBER,,0,pydata/xarray/pulls/8380,"
this is the initial phase of our plan to split the `.set_dims()` method into two distinct methods `.expand_dims()` and `.broadcast_to()` as outlined in the [design document](https://github.com/pydata/xarray/blob/main/design_notes/named_array_design_doc.md).
- [x] add `expand_dims()`
- [x] add `broadcast_to()`
- [x] add `.transpose()` and `.T` to `NamedArray`
- [x] update `whats-new.rst`","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8380/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
2024719599,PR_kwDOAMm_X85hGcWv,8519,[skip-ci] fix RTD docs build,13301940,closed,0,,,1,2023-12-04T20:56:34Z,2023-12-04T21:08:49Z,2023-12-04T21:04:47Z,MEMBER,,0,pydata/xarray/pulls/8519,"
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8519/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
768981497,MDExOlB1bGxSZXF1ZXN0NTQxMjMxNjc0,4700,Raise an informative error message when object array has mixed types,13301940,closed,0,,,10,2020-12-16T15:07:21Z,2023-11-28T22:19:02Z,2023-11-28T22:19:00Z,MEMBER,,0,pydata/xarray/pulls/4700,"
- [x] Closes #2620
- [x] Tests added
- [x] Passes `isort . && black . && mypy . && flake8`
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/4700/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1959903578,PR_kwDOAMm_X85drMmv,8369,fix `NamedArray.imag` and `NamedArray.real` typing info,13301940,closed,0,,,4,2023-10-24T19:06:48Z,2023-10-25T19:17:55Z,2023-10-25T19:17:53Z,MEMBER,,0,pydata/xarray/pulls/8369,"
- [ ] #8365 follow up
- [ ] Tests added","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8369/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1958381131,PR_kwDOAMm_X85dmDe9,8365,add `.imag` and `.real` properties to `NamedArray`,13301940,closed,0,,,1,2023-10-24T02:55:30Z,2023-10-24T18:19:12Z,2023-10-24T10:53:58Z,MEMBER,,0,pydata/xarray/pulls/8365,"
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8365/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1916603412,PR_kwDOAMm_X85bZS7E,8244,Migrate VariableArithmetic to NamedArrayArithmetic,13301940,open,0,,,6,2023-09-28T02:29:15Z,2023-10-11T17:03:02Z,,MEMBER,,1,pydata/xarray/pulls/8244,"
- [ ] towards #8238
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8244/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1854145928,PR_kwDOAMm_X85YHMlg,8075,initial refactor for NamedArray,13301940,closed,0,,,17,2023-08-17T02:37:37Z,2023-09-27T15:35:13Z,2023-09-27T15:23:21Z,MEMBER,,0,pydata/xarray/pulls/8075,"
this pull request introduces the initial prototype for `NamedArray` as discussed in issue #8073. the intention is to approach the implementation in multiple phases through separate pull requests. the main objectives of this PR are as follows:
1. creation of a new class, `xarray.named_array.NamedArray`, which will encompass most of the necessary functionality for the lightweight version of `xarray.Variable`. As a result, `xarray.Variable` will inherit from this class and implement specific functionality related to xarray.
2. removal of all imports of internal Xarray classes and utilities from the `xarray.named_array` sub-package. the aim is to eliminate dependencies on anything within `xarray.core`. This will facilitate the future goal of moving `named_array` into its own standalone package, with `xarray` then depending on it.
- [ ] Towards #3981
- [x] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/8075/reactions"", ""total_count"": 3, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 3, ""eyes"": 0}",,,13221727,pull
1251404616,PR_kwDOAMm_X844ndTA,6650,import `cleanup` fixture from `distributed`,13301940,closed,0,,,1,2022-05-28T01:39:42Z,2023-06-16T22:52:29Z,2022-05-29T13:10:52Z,MEMBER,,0,pydata/xarray/pulls/6650,"
- [x] Closes #6649
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6650/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
870619014,MDExOlB1bGxSZXF1ZXN0NjI1Nzg2MTY2,5234,Code cleanup,13301940,closed,0,,,14,2021-04-29T05:16:02Z,2023-06-16T22:52:27Z,2021-05-13T16:37:15Z,MEMBER,,0,pydata/xarray/pulls/5234,"
This PR introduces small changes to enhance code maintainability...
- [x] Passes `pre-commit run --all-files`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5234/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
856020853,MDExOlB1bGxSZXF1ZXN0NjEzNjY3MDQx,5142,Trigger CI on push or pull_request but not both,13301940,closed,0,,,2,2021-04-12T14:12:29Z,2023-06-16T22:52:19Z,2022-01-20T01:51:10Z,MEMBER,,0,pydata/xarray/pulls/5142,"
- [ ] Addresses https://github.com/pydata/xarray/pull/5139#issuecomment-817362317
- [x] Passes `pre-commit run --all-files`
Here's a snippet of how this looks like when a pull request is created on the same main repository:
Cc @keewis ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5142/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
539394615,MDExOlB1bGxSZXF1ZXN0MzU0Mzc5Nzk2,3640,Add entrypoint for plotting backends,13301940,closed,0,,,14,2019-12-18T01:11:08Z,2023-06-16T22:52:17Z,2021-09-26T21:53:42Z,MEMBER,,1,pydata/xarray/pulls/3640,"
Libraries, including `xarray`, register backends via `entrypoints`. Xarray adds an EntryPoint ""group"" in its `setup.py`. Third party libraries add items to that group in their `setup.pys`.
Xarray and the third-party library don't need to talk to each other directly to register a backend. They talk through `pkg_resources`.
For instance, `hvplot/holoviews` would add an EntryPoint for `xarray_plotting_backends` in its setup.py:
```python
entry_points={""xarray_plotting_backends"": ['holoviews = hvplot:plotting']}
```
and xarray users should be able to achieve what @jsignell proposed in #3553:
> ```python
> import xarray as xr
> air = xr.tutorial.open_dataset('air_temperature').load().air
> xr.set_options(plotting_backend='holoviews')
>
> air.isel(time=500).plot()
> ```
>
>
- [ ] Closes #3553
- [x] Tests added
- [x] Passes `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/3640/reactions"", ""total_count"": 2, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 2, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1755311785,PR_kwDOAMm_X85S6P0H,7917,Ensure dtype of reindex result matches dtype of the original DataArray,13301940,closed,0,,,0,2023-06-13T16:55:38Z,2023-06-16T22:51:46Z,2023-06-16T03:10:23Z,MEMBER,,0,pydata/xarray/pulls/7917,"
- [x] Closes #7299
- [x] Tests added
- [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/7917/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1380118553,PR_kwDOAMm_X84_T6Nz,7061,Remove team page from docs and link to https://xarray.dev/team,13301940,closed,0,,,1,2022-09-20T23:51:19Z,2022-10-06T18:08:30Z,2022-10-06T18:08:28Z,MEMBER,,0,pydata/xarray/pulls/7061,"
This PR removes the existing team page from the docs and links to the newly created https://xarray.dev/team on the landing page
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/7061/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1330592136,PR_kwDOAMm_X848v_oP,6885,Fix documentation builds,13301940,closed,0,,,0,2022-08-06T00:39:38Z,2022-08-06T00:56:01Z,2022-08-06T00:55:52Z,MEMBER,,0,pydata/xarray/pulls/6885,"
- [ ] Addresses https://github.com/pydata/xarray/pull/6884#issuecomment-1207094758
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6885/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1254335146,PR_kwDOAMm_X844w-ab,6655,Fix notebooks' HTML links,13301940,closed,0,,,0,2022-05-31T19:33:45Z,2022-05-31T19:52:45Z,2022-05-31T19:52:43Z,MEMBER,,0,pydata/xarray/pulls/6655,"
- [ ] #6530 follow-up
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6655/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
978356586,MDExOlB1bGxSZXF1ZXN0NzE4OTU4Mzc1,5734,Enable `flox` in `GroupBy` and `resample`,13301940,closed,0,,,25,2021-08-24T18:09:28Z,2022-05-15T03:33:49Z,2022-05-15T02:38:29Z,MEMBER,,0,pydata/xarray/pulls/5734,"
This PR supersedes https://github.com/pydata/xarray/pull/4540. This is still a very rough draft :)
- Closes
- [x] Closes #4473
- [x] Closes #4498
- [x] Closes #659
- [x] Closes #2237
- [x] xref https://github.com/pangeo-data/pangeo/issues/271
- [x] Tests added
- [x] Passes `pre-commit run --all-files`
- [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
TODO:
- [x] flox vs legacy tests with nans
- [x] test passing flox-specific kwargs like engine and method.","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5734/reactions"", ""total_count"": 8, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 8, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1155507950,PR_kwDOAMm_X84zvNBA,6314,Add General issue template,13301940,closed,0,,,2,2022-03-01T15:54:16Z,2022-03-02T17:14:40Z,2022-03-02T13:56:25Z,MEMBER,,0,pydata/xarray/pulls/6314,"
- [x] Addresses https://github.com/pydata/xarray/issues/6027#issuecomment-1048737617
Cc @benbovy & @max-sixty ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6314/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1155512531,PR_kwDOAMm_X84zvN8f,6315,Disable CI runs on forks,13301940,closed,0,,,0,2022-03-01T15:58:29Z,2022-03-01T16:58:28Z,2022-03-01T16:57:43Z,MEMBER,,0,pydata/xarray/pulls/6315,"
This PR ensures all CI workflows are only enabled on the main repository. A few workflows were still being triggered on forks.
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6315/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1120237582,PR_kwDOAMm_X84x4Smo,6224,Verify built dist/wheel,13301940,closed,0,,,0,2022-02-01T05:02:42Z,2022-02-03T15:44:41Z,2022-02-03T15:41:17Z,MEMBER,,0,pydata/xarray/pulls/6224,"
- [x] Closes #6222
- [ ] Tests added
- [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst`
- [ ] New functions/methods are listed in `api.rst`
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6224/reactions"", ""total_count"": 3, ""+1"": 3, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1090118252,PR_kwDOAMm_X84wWQZg,6119,Replace markdown issue templates with issue forms,13301940,closed,0,,,2,2021-12-28T20:41:40Z,2021-12-29T16:48:33Z,2021-12-29T16:47:48Z,MEMBER,,0,pydata/xarray/pulls/6119,"
- [ ] Closes #6027
**Preview**:
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6119/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
1090121529,PR_kwDOAMm_X84wWREQ,6120,Remove pre-commit GHA workflow,13301940,closed,0,,,1,2021-12-28T20:50:09Z,2021-12-29T16:45:03Z,2021-12-29T16:28:45Z,MEMBER,,0,pydata/xarray/pulls/6120,"
We've been using [pre-commit](https://pre-commit.ci/) for the last month or so, and it seems to be working fine. This PR removes the redundant pre-commit hooks GHA workflow.
- [ ] Towards https://github.com/pydata/xarray/issues/5940
","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/6120/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
877830157,MDExOlB1bGxSZXF1ZXN0NjMxNzIyNjgy,5273,Release-workflow: Bug fix,13301940,closed,0,,,3,2021-05-06T18:21:24Z,2021-05-06T18:41:19Z,2021-05-06T18:41:17Z,MEMBER,,0,pydata/xarray/pulls/5273,This fixes https://github.com/pydata/xarray/issues/5232#issuecomment-833729171,"{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5273/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
877586869,MDExOlB1bGxSZXF1ZXN0NjMxNTEyOTA1,5267,Disable workflows on forks,13301940,closed,0,,,2,2021-05-06T14:55:05Z,2021-05-06T16:17:26Z,2021-05-06T16:17:25Z,MEMBER,,0,pydata/xarray/pulls/5267,"
Currently, the CI runs both on a user's fork and `pydata/xarray`. This leads to duplicate runs and the user is flooded with GitHub notifications in case of failures... This PR addresses this issue by making sure the workflows are triggered on `pydata/xarray` only.

","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5267/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull
877605250,MDExOlB1bGxSZXF1ZXN0NjMxNTI3OTEw,5268,Remove push event trigger for the release workflow,13301940,closed,0,,,1,2021-05-06T15:13:08Z,2021-05-06T15:14:20Z,2021-05-06T15:14:13Z,MEMBER,,0,pydata/xarray/pulls/5268,"
It turns out that pushes on main branch result in [an invalid package version](https://github.com/pydata/xarray/runs/2519311659). To address this issue, I am disabling the `push` event trigger on the release workflow
Cc @keewis
```html
Uploading xarray-0.17.1.dev108+gf5e4fd5f-py3-none-any.whl
0%| | 0.00/796k [00:00, ?B/s]
73%|███████▎ | 584k/796k [00:00<00:00, 3.39MB/s]
100%|██████████| 796k/796k [00:00<00:00, 1.52MB/s]
Content received from server: