home / github

Menu
  • Search all tables
  • GraphQL API

issues

Table actions
  • GraphQL API for issues

58 rows where repo = 13221727, type = "pull" and user = 13301940 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

Suggested facets: comments, draft, created_at (date), updated_at (date), closed_at (date)

state 2

  • closed 55
  • open 3

type 1

  • pull · 58 ✖

repo 1

  • xarray · 58 ✖
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) andersy005 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.

  • [ ] 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
}
    xarray 13221727 pull
2203689075 PR_kwDOAMm_X85qjXJq 8870 Enable explicit use of key tuples (instead of *Indexer objects) in indexing adapters and explicitly indexed arrays andersy005 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
}
    xarray 13221727 pull
2212435865 PR_kwDOAMm_X85rAwYu 8885 add `.oindex` and `.vindex` to `BackendArray` andersy005 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

  1. 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
}
    xarray 13221727 pull
2231711080 PR_kwDOAMm_X85sCbN- 8921 Revert `.oindex` and `.vindex` additions in `_ElementwiseFunctionArray`, `NativeEndiannessArray`, and `BoolTypeArray` classes andersy005 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, 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
}
    xarray 13221727 pull
2203546866 PR_kwDOAMm_X85qi4jp 8869 Handle .oindex and .vindex for the PandasMultiIndexingAdapter and PandasIndexingAdapter andersy005 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
}
    xarray 13221727 pull
2196627659 PR_kwDOAMm_X85qLGOs 8857 increase typing annotations coverage in `xarray/core/indexing.py` andersy005 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
}
    xarray 13221727 pull
2189661150 PR_kwDOAMm_X85pzxuX 8845 Implement setitem syntax for `.oindex` and `.vindex` properties andersy005 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

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
}
    xarray 13221727 pull
1958139851 PR_kwDOAMm_X85dlQp4 8364 Migrate from *Indexer objects to `.oindex` and `.vindex` on explicityIndexed array classes andersy005 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
}
    xarray 13221727 pull
2157865512 PR_kwDOAMm_X85oHSTV 8790 Expand use of `.oindex` and `.vindex` andersy005 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. 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
}
    xarray 13221727 pull
2187291438 PR_kwDOAMm_X85prlwg 8835 [skip-ci] Add dask-expr dependency to doc.yml andersy005 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
}
    xarray 13221727 pull
2151987948 PR_kwDOAMm_X85nzdLf 8780 introduce `.vindex` property for Explicitly Indexed Arrays andersy005 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
}
    xarray 13221727 pull
2135390562 PR_kwDOAMm_X85m6nof 8750 refactor `indexing.py`: introduce `.oindex` for Explicitly Indexed Arrays andersy005 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
}
    xarray 13221727 pull
2137484840 PR_kwDOAMm_X85nBx-a 8758 refactor DaskIndexingAdapter `__getitem__` method andersy005 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" 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
}
    xarray 13221727 pull
1852260723 PR_kwDOAMm_X85YA1UQ 8073 add design document for "named-array" andersy005 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
}
    xarray 13221727 pull
1918009494 PR_kwDOAMm_X85beIFo 8250 copy the `dtypes` module to the `namedarray` package. andersy005 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
}
    xarray 13221727 pull
2021745010 PR_kwDOAMm_X85g8efA 8505 roll out the new/refreshed Xarray logo andersy005 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
}
    xarray 13221727 pull
2106574555 PR_kwDOAMm_X85lYkg9 8682 Fix CI: temporary pin pytest version to 7.4.* andersy005 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
}
    xarray 13221727 pull
1964505426 PR_kwDOAMm_X85d61au 8380 [namedarray] split `.set_dims()` into `.expand_dims()` and `broadcast_to()` andersy005 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. - [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
}
    xarray 13221727 pull
2024719599 PR_kwDOAMm_X85hGcWv 8519 [skip-ci] fix RTD docs build andersy005 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
}
    xarray 13221727 pull
768981497 MDExOlB1bGxSZXF1ZXN0NTQxMjMxNjc0 4700 Raise an informative error message when object array has mixed types andersy005 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
}
    xarray 13221727 pull
1959903578 PR_kwDOAMm_X85drMmv 8369 fix `NamedArray.imag` and `NamedArray.real` typing info andersy005 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
}
    xarray 13221727 pull
1958381131 PR_kwDOAMm_X85dmDe9 8365 add `.imag` and `.real` properties to `NamedArray` andersy005 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
}
    xarray 13221727 pull
1916603412 PR_kwDOAMm_X85bZS7E 8244 Migrate VariableArithmetic to NamedArrayArithmetic andersy005 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
}
    xarray 13221727 pull
1854145928 PR_kwDOAMm_X85YHMlg 8075 initial refactor for NamedArray andersy005 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.

  3. [ ] Towards #3981

  4. [x] Tests added
  5. [ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
  6. [ ] 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
}
    xarray 13221727 pull
1251404616 PR_kwDOAMm_X844ndTA 6650 import `cleanup` fixture from `distributed` andersy005 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
}
    xarray 13221727 pull
870619014 MDExOlB1bGxSZXF1ZXN0NjI1Nzg2MTY2 5234 Code cleanup andersy005 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
}
    xarray 13221727 pull
856020853 MDExOlB1bGxSZXF1ZXN0NjEzNjY3MDQx 5142 Trigger CI on push or pull_request but not both andersy005 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
}
    xarray 13221727 pull
539394615 MDExOlB1bGxSZXF1ZXN0MzU0Mzc5Nzk2 3640 Add entrypoint for plotting backends andersy005 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
}
    xarray 13221727 pull
1755311785 PR_kwDOAMm_X85S6P0H 7917 Ensure dtype of reindex result matches dtype of the original DataArray andersy005 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
}
    xarray 13221727 pull
1380118553 PR_kwDOAMm_X84_T6Nz 7061 Remove team page from docs and link to https://xarray.dev/team andersy005 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
}
    xarray 13221727 pull
1330592136 PR_kwDOAMm_X848v_oP 6885 Fix documentation builds andersy005 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
}
    xarray 13221727 pull
1254335146 PR_kwDOAMm_X844w-ab 6655 Fix notebooks' HTML links andersy005 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
}
    xarray 13221727 pull
978356586 MDExOlB1bGxSZXF1ZXN0NzE4OTU4Mzc1 5734 Enable `flox` in `GroupBy` and `resample` andersy005 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
}
    xarray 13221727 pull
1155507950 PR_kwDOAMm_X84zvNBA 6314 Add General issue template andersy005 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
}
    xarray 13221727 pull
1155512531 PR_kwDOAMm_X84zvN8f 6315 Disable CI runs on forks andersy005 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
}
    xarray 13221727 pull
1120237582 PR_kwDOAMm_X84x4Smo 6224 Verify built dist/wheel andersy005 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
}
    xarray 13221727 pull
1090118252 PR_kwDOAMm_X84wWQZg 6119 Replace markdown issue templates with issue forms andersy005 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
}
    xarray 13221727 pull
1090121529 PR_kwDOAMm_X84wWREQ 6120 Remove pre-commit GHA workflow andersy005 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 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
}
    xarray 13221727 pull
877830157 MDExOlB1bGxSZXF1ZXN0NjMxNzIyNjgy 5273 Release-workflow: Bug fix andersy005 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
}
    xarray 13221727 pull
877586869 MDExOlB1bGxSZXF1ZXN0NjMxNTEyOTA1 5267 Disable workflows on forks andersy005 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
}
    xarray 13221727 pull
877605250 MDExOlB1bGxSZXF1ZXN0NjMxNTI3OTEw 5268 Remove push event trigger for the release workflow andersy005 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. 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:

<html> <head> <title>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.</title> </head> <body>

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. </body> </html>

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
}
    xarray 13221727 pull
873842812 MDExOlB1bGxSZXF1ZXN0NjI4NTQ0NTY3 5244 Add GitHub action for publishing artifacts to PyPI andersy005 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
}
    xarray 13221727 pull
873831430 MDExOlB1bGxSZXF1ZXN0NjI4NTM2NzE0 5243 Parametrize test for __setitem__ (for dask array) andersy005 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
}
    xarray 13221727 pull
874231291 MDExOlB1bGxSZXF1ZXN0NjI4ODMyOTM3 5250 Fix bulleted list indentation in docstrings andersy005 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:

@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
}
    xarray 13221727 pull
856035141 MDExOlB1bGxSZXF1ZXN0NjEzNjc5MDU5 5143 Add dependabot config (for updating github actions) andersy005 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
}
    xarray 13221727 pull
832768296 MDExOlB1bGxSZXF1ZXN0NTkzODc1MDU4 5039 Fix sticky sidebar responsiveness on small screens andersy005 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:

  • [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
}
    xarray 13221727 pull
790677360 MDExOlB1bGxSZXF1ZXN0NTU4ODUwMjUy 4835 📚 New theme & rearrangement of the docs andersy005 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
}
    xarray 13221727 pull
830167079 MDExOlB1bGxSZXF1ZXN0NTkxNzIxMTIx 5025 GHA for automatically canceling previous CI runs andersy005 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
}
    xarray 13221727 pull
773945564 MDExOlB1bGxSZXF1ZXN0NTQ0OTQ5MTU3 4730 Migrate CI from azure pipelines to GitHub Actions andersy005 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
}
    xarray 13221727 pull
773934304 MDExOlB1bGxSZXF1ZXN0NTQ0OTM5ODY5 4729 Trigger upstream CI on cron schedule (by default) andersy005 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
}
    xarray 13221727 pull
767049213 MDExOlB1bGxSZXF1ZXN0NTM5ODkzMTI5 4695 Remove dictionary unpacking when using `.loc` to avoid collision with `.sel` parameters andersy005 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
}
    xarray 13221727 pull
749116752 MDExOlB1bGxSZXF1ZXN0NTI1OTg3NDE1 4604 Amend upstream-dev GitHub action + Bug fixes andersy005 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
}
    xarray 13221727 pull
741791697 MDExOlB1bGxSZXF1ZXN0NTIwMDQ0MDYx 4578 Direct users to GitHub discussions page for usage questions andersy005 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
}
    xarray 13221727 pull
742845971 MDExOlB1bGxSZXF1ZXN0NTIwODk3NDY1 4583 Add GH action for running tests against upstream dev andersy005 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
}
    xarray 13221727 pull
536694911 MDExOlB1bGxSZXF1ZXN0MzUyMTgyMTQ1 3612 Extend DatetimeAccessor properties and support `.dt` accessor for Timedelta andersy005 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
}
    xarray 13221727 pull
520306672 MDExOlB1bGxSZXF1ZXN0MzM4OTE5MTg1 3502 Harmonize `FillValue` and `missing_value` during encoding and decoding steps andersy005 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) <ipython-input-79-9fbc3632e28b> in <module> ----> 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
}
    xarray 13221727 pull
496533454 MDExOlB1bGxSZXF1ZXN0MzE5ODY5NTAw 3328 Documentation improvements andersy005 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
}
    xarray 13221727 pull
510631260 MDExOlB1bGxSZXF1ZXN0MzMwOTcxNzk1 3430 Sync with latest version of cftime (v1.0.4) andersy005 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
}
    xarray 13221727 pull

Advanced export

JSON shape: default, array, newline-delimited, object

CSV options:

CREATE TABLE [issues] (
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [number] INTEGER,
   [title] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [state] TEXT,
   [locked] INTEGER,
   [assignee] INTEGER REFERENCES [users]([id]),
   [milestone] INTEGER REFERENCES [milestones]([id]),
   [comments] INTEGER,
   [created_at] TEXT,
   [updated_at] TEXT,
   [closed_at] TEXT,
   [author_association] TEXT,
   [active_lock_reason] TEXT,
   [draft] INTEGER,
   [pull_request] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [state_reason] TEXT,
   [repo] INTEGER REFERENCES [repos]([id]),
   [type] TEXT
);
CREATE INDEX [idx_issues_repo]
    ON [issues] ([repo]);
CREATE INDEX [idx_issues_milestone]
    ON [issues] ([milestone]);
CREATE INDEX [idx_issues_assignee]
    ON [issues] ([assignee]);
CREATE INDEX [idx_issues_user]
    ON [issues] ([user]);
Powered by Datasette · Queries took 81.818ms · About: xarray-datasette