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: ![Screenshot 2023-12-01 at 16 05 19](https://github.com/pydata/xarray/assets/13301940/11484d91-3eae-466b-a4d6-6794006b6e6c) ![Screenshot 2023-12-01 at 16 05 13](https://github.com/pydata/xarray/assets/13301940/77047c74-f038-4230-8118-539b22ede33b) ","{""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. ![Screen Shot 2021-05-06 at 8 55 24 AM](https://user-images.githubusercontent.com/13301940/117319777-daffaf00-ae48-11eb-8523-5309542c524e.png) ","{""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 400 '0.17.1.dev108+gf5e4fd5f' is an invalid value for Version. Error: Can't use PEP 440 local versions. See packaging.python.org/specifications/core-metadata for more information.

400 '0.17.1.dev108+gf5e4fd5f' is an invalid value for Version. Error: Can't use PEP 440 local versions. See packaging.python.org/specifications/core-metadata for more information.

The server could not comply with the request since it is either malformed or otherwise incorrect.

'0.17.1.dev108+gf5e4fd5f' is an invalid value for Version. Error: Can't use PEP 440 local versions. See packaging.python.org/specifications/core-metadata for more information. HTTPError: 400 Bad Request from test.pypi.org/legacy '0.17.1.dev108+gf5e4fd5f' is an invalid value for Version. Error: Can't use PEP 440 local versions. See packaging.python.org/specifications/core-metadata for more information. ```","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5268/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 873842812,MDExOlB1bGxSZXF1ZXN0NjI4NTQ0NTY3,5244,Add GitHub action for publishing artifacts to PyPI,13301940,closed,0,,,15,2021-05-02T05:18:42Z,2021-05-06T15:00:32Z,2021-05-06T14:46:10Z,MEMBER,,0,pydata/xarray/pulls/5244," This addresses https://github.com/pydata/xarray/issues/5232#issuecomment-830242266, and will (hopefully) reduce the time it takes to cut a new release. - [x] Passes `pre-commit run --all-files` ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5244/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 873831430,MDExOlB1bGxSZXF1ZXN0NjI4NTM2NzE0,5243,Parametrize test for __setitem__ (for dask array),13301940,closed,0,,,4,2021-05-02T04:01:17Z,2021-05-06T12:49:49Z,2021-05-06T07:25:21Z,MEMBER,,0,pydata/xarray/pulls/5243," Follow up PR for #5174 - [x] Tests added - [x] Passes `pre-commit run --all-files` ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5243/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 874231291,MDExOlB1bGxSZXF1ZXN0NjI4ODMyOTM3,5250,Fix bulleted list indentation in docstrings,13301940,closed,0,,,10,2021-05-03T05:10:53Z,2021-05-04T18:09:00Z,2021-05-03T23:23:49Z,MEMBER,,0,pydata/xarray/pulls/5250," - [x] Closes #5248 - [x] Passes `pre-commit run --all-files` As I understand it, the issue described in #5248 is caused by bad indentation... With this PR, we get the following: ![Screen Shot 2021-05-02 at 11 06 41 PM](https://user-images.githubusercontent.com/13301940/116842888-60f6ce00-ab9b-11eb-826e-7eeb26827d34.png) @max-sixty, let me know if there are other instances that need to be fixed as part of this PR.... ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5250/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 856035141,MDExOlB1bGxSZXF1ZXN0NjEzNjc5MDU5,5143,Add dependabot config (for updating github actions),13301940,closed,0,,,0,2021-04-12T14:27:04Z,2021-04-14T14:39:43Z,2021-04-14T14:39:42Z,MEMBER,,0,pydata/xarray/pulls/5143," - [x] Passes `pre-commit run --all-files` ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5143/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 832768296,MDExOlB1bGxSZXF1ZXN0NTkzODc1MDU4,5039,Fix sticky sidebar responsiveness on small screens,13301940,closed,0,,,2,2021-03-16T12:53:32Z,2021-03-17T23:00:47Z,2021-03-17T23:00:46Z,MEMBER,,0,pydata/xarray/pulls/5039," - This PR reduces the max-width for `.container` divs so as to address issues with RTD's multiple version badges. - Latest: https://xarray.pydata.org/en/latest/index.html: the rtd badge box doesn't work - This PR: https://xray--5039.org.readthedocs.build/en/5039/index.html: appears to be working fine - Fixes CSS properties to enable appropriate toggle/responsiveness when using small screens. Unfortunately, this doesn't seem to have fixed the problem on small screens: ![Screen Shot 2021-03-16 at 10 43 05 AM](https://user-images.githubusercontent.com/13301940/111347113-814ae780-8644-11eb-815d-0056c6404313.png) - [x] Passes `pre-commit run --all-files`","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5039/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 790677360,MDExOlB1bGxSZXF1ZXN0NTU4ODUwMjUy,4835,📚 New theme & rearrangement of the docs,13301940,closed,0,,,25,2021-01-21T04:17:33Z,2021-03-15T23:20:53Z,2021-03-15T23:20:51Z,MEMBER,,0,pydata/xarray/pulls/4835," - [ ] Closes #xxxx - [x] Passes `pre-commit run --all-files` - [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/4835/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 830167079,MDExOlB1bGxSZXF1ZXN0NTkxNzIxMTIx,5025,GHA for automatically canceling previous CI runs,13301940,closed,0,,,0,2021-03-12T14:24:27Z,2021-03-15T15:06:29Z,2021-03-15T15:06:27Z,MEMBER,,0,pydata/xarray/pulls/5025," The `styfle/cancel-workflow-action` requires a token with write permissions in order to cancel duplicate runs. It turns out that the default GitHub token access made available to pull requests from forks has read permissions only. To address this issue, this PR introduces a new workflow that uses a special `workflow_run` trigger which allows the workflow to run to run with a token with write permissions. - [x] Passes `pre-commit run --all-files` Cc @keewis","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5025/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 773945564,MDExOlB1bGxSZXF1ZXN0NTQ0OTQ5MTU3,4730,Migrate CI from azure pipelines to GitHub Actions,13301940,closed,0,,,22,2020-12-23T17:22:49Z,2021-02-25T22:03:22Z,2021-01-11T12:11:04Z,MEMBER,,0,pydata/xarray/pulls/4730," As discussed in today's dev meeting, it'd be convenient to migrate all our CIs from azure pipelines to GitHub Actions. - [x] Consolidate all style/type checks (isort, black, mypy, flake8) into a single workflow and use the `pre-commit` GitHub action. - [x] Create a main CI workflow and parametrize it for different settings - [x] Add minimum version policy workflow - [x] Add Doctests workflow - [x] Remove azure pipelines config - [x] Update docs: remove references to azure pipelines","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/4730/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 773934304,MDExOlB1bGxSZXF1ZXN0NTQ0OTM5ODY5,4729,Trigger upstream CI on cron schedule (by default),13301940,closed,0,,,7,2020-12-23T17:00:24Z,2021-01-06T22:17:13Z,2021-01-06T22:17:11Z,MEMBER,,0,pydata/xarray/pulls/4729," As discussed in today's dev meeting, this PR modifies the `push` and `pull_request` event triggers by making sure the upstream dev CI is triggered by cron schedule by default. - [ ] 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/4729/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 767049213,MDExOlB1bGxSZXF1ZXN0NTM5ODkzMTI5,4695,Remove dictionary unpacking when using `.loc` to avoid collision with `.sel` parameters,13301940,closed,0,,,2,2020-12-15T00:30:04Z,2020-12-16T14:50:34Z,2020-12-16T14:50:32Z,MEMBER,,0,pydata/xarray/pulls/4695," - [x] Closes #2840 - [x] Tests added - [x] Passes `isort . && black . && mypy . && flake8` - [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/4695/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 749116752,MDExOlB1bGxSZXF1ZXN0NTI1OTg3NDE1,4604,Amend upstream-dev GitHub action + Bug fixes,13301940,closed,0,,,5,2020-11-23T20:51:45Z,2020-12-05T00:52:48Z,2020-12-05T00:30:11Z,MEMBER,,0,pydata/xarray/pulls/4604," - [x] Closes #4603 - [x] Closes #4643 - [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst` ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/4604/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 741791697,MDExOlB1bGxSZXF1ZXN0NTIwMDQ0MDYx,4578,Direct users to GitHub discussions page for usage questions,13301940,closed,0,,,3,2020-11-12T17:34:05Z,2020-11-25T17:43:47Z,2020-11-25T16:09:08Z,MEMBER,,0,pydata/xarray/pulls/4578," Towards https://github.com/pydata/xarray/issues/4549 - [ ] Closes #xxxx - [ ] 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/4578/reactions"", ""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 742845971,MDExOlB1bGxSZXF1ZXN0NTIwODk3NDY1,4583,Add GH action for running tests against upstream dev,13301940,closed,0,,,7,2020-11-13T23:19:36Z,2020-11-22T04:37:52Z,2020-11-22T02:37:14Z,MEMBER,,0,pydata/xarray/pulls/4583," This GitHub action will run tests against the upstream dependencies. When the tests fail, the GH action will create an issue (with ""CI"" label). The body of the created issue includes: - test summary report - a link to the failed workflow run Here's a couple screenshots from a toy repository: - [x] Closes #4574 - [x] Tests added - [x] Passes `isort . && black . && mypy . && flake8` - [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/4583/reactions"", ""total_count"": 5, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 5, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 536694911,MDExOlB1bGxSZXF1ZXN0MzUyMTgyMTQ1,3612,Extend DatetimeAccessor properties and support `.dt` accessor for Timedelta,13301940,closed,0,,,2,2019-12-12T00:54:18Z,2019-12-18T18:03:13Z,2019-12-18T15:27:31Z,MEMBER,,0,pydata/xarray/pulls/3612," - [x] Closes #3609 - [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/3612/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 520306672,MDExOlB1bGxSZXF1ZXN0MzM4OTE5MTg1,3502,Harmonize `FillValue` and `missing_value` during encoding and decoding steps,13301940,closed,0,,,7,2019-11-09T01:07:25Z,2019-12-11T20:09:15Z,2019-11-14T01:22:51Z,MEMBER,,0,pydata/xarray/pulls/3502," As pointed out in https://github.com/jbusecke/cmip6_preprocessing/issues/5, xarray appears to be very strict during the *encoding* and *decoding* steps even when there are (harmless) discrepancies between `missing_value` and `_FillValue`. For instance, when dtypes of `missing_value` and `_FillValue` are different, xarray gives up: ```python In [74]: from xarray.coding import variables In [75]: import numpy as np In [76]: import xarray as xr In [77]: original = xr.Variable( ...: (""x"",), ...: [0.0, -1.0, 1.0], ...: encoding={""_FillValue"": np.float32(1e20), ""missing_value"": np.float64(1e20)}, ...: ) In [78]: coder = variables.CFMaskCoder() In [79]: encoded = coder.encode(original) --------------------------------------------------------------------------- ValueError Traceback (most recent call last) in ----> 1 encoded = coder.encode(original) /glade/work/abanihi/devel/pangeo/xarray/xarray/coding/variables.py in encode(self, variable, name) 156 raise ValueError( 157 ""Variable {!r} has multiple fill values {}. "" --> 158 ""Cannot encode data. "".format(name, [fv, mv]) 159 ) 160 ValueError: Variable None has multiple fill values [1e+20, 1e+20]. Cannot encode data. ``` - [ ] Closes #xxxx - [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/3502/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 496533454,MDExOlB1bGxSZXF1ZXN0MzE5ODY5NTAw,3328,Documentation improvements,13301940,closed,0,,,9,2019-09-20T20:28:15Z,2019-10-28T18:00:26Z,2019-09-29T13:50:22Z,MEMBER,,0,pydata/xarray/pulls/3328," Cc @dcherian ","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/3328/reactions"", ""total_count"": 1, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 1, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull 510631260,MDExOlB1bGxSZXF1ZXN0MzMwOTcxNzk1,3430,Sync with latest version of cftime (v1.0.4),13301940,closed,0,,,5,2019-10-22T12:45:26Z,2019-10-22T19:25:18Z,2019-10-22T18:31:35Z,MEMBER,,0,pydata/xarray/pulls/3430," - Don't need `dayofwk=-1` for cftime>=1.0.4 ---- - [x] Closes #3426 and Fixes #3427 - [ ] 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/3430/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull