issues

Table actions

22 rows where comments = 2, type = "pull" and user = 35968931 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

state 2

type 1

  • pull · 22 ✖

repo 1

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
2267780811 PR_kwDOAMm_X85t8kgX 8979 Warn on automatic coercion to coordinate variables in Dataset constructor TomNicholas 35968931 open 0     2 2024-04-28T19:44:20Z 2024-04-29T21:13:00Z   MEMBER   0 pydata/xarray/pulls/8979
  • [x] Starts the deprecation cycle for #8959
  • [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~~
  • [ ] Change existing code + examples so as not to emit this new warning everywhere.
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/8979/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
2254350395 PR_kwDOAMm_X85tPTua 8960 Option to not auto-create index during expand_dims TomNicholas 35968931 closed 0     2 2024-04-20T03:27:23Z 2024-04-27T16:48:30Z 2024-04-27T16:48:24Z MEMBER   0 pydata/xarray/pulls/8960

TODO: - [x] Add new kwarg to DataArray.expand_dims - [ ] Add examples to docstrings? - [x] Check it actually solves the problem in #8872

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/8960/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
2213406564 PR_kwDOAMm_X85rEF-X 8886 Allow multidimensional variable with same name as dim when constructing dataset via coords TomNicholas 35968931 closed 0     2 2024-03-28T14:37:27Z 2024-03-28T17:07:10Z 2024-03-28T16:28:09Z MEMBER   0 pydata/xarray/pulls/8886

Supercedes #8884 as a way to close #8883, in light of me having learnt that this is now allowed! https://github.com/pydata/xarray/issues/8883#issuecomment-2024645815. So this is really a follow-up to #7989.

  • [x] Closes #8883
  • [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/8886/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
2212211084 PR_kwDOAMm_X85rABMo 8884 Forbid invalid Coordinates object TomNicholas 35968931 closed 0     2 2024-03-28T02:14:01Z 2024-03-28T14:38:43Z 2024-03-28T14:38:03Z MEMBER   0 pydata/xarray/pulls/8884
  • [x] Closes #8883
  • [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/8884/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
2092346228 PR_kwDOAMm_X85ko-Y2 8632 Pin sphinx-book-theme to 1.0.1 to try to deal with #8619 TomNicholas 35968931 closed 0     2 2024-01-21T02:18:49Z 2024-01-23T20:16:13Z 2024-01-23T18:28:35Z MEMBER   0 pydata/xarray/pulls/8632
  • [x] Hopefully closes #8619
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/8632/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1820788594 PR_kwDOAMm_X85WW40r 8019 Generalize cumulative reduction (scan) to non-dask types TomNicholas 35968931 closed 0     2 2023-07-25T17:22:07Z 2023-12-18T19:30:18Z 2023-12-18T19:30:18Z MEMBER   0 pydata/xarray/pulls/8019
  • [x] Needed for https://github.com/tomwhite/cubed/issues/277#issuecomment-1648567431 - should have been added in #7019
  • [ ] ~~Tests added~~ (would go in cubed-xarray)
  • [x] User visible changes (including notable bug fixes) are documented in whats-new.rst
  • [x] New functions/methods are listed in api.rst (new ABC method will be documented on chunked array types page automatically)
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/8019/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1036473974 PR_kwDOAMm_X84tsaL3 5900 Add .chunksizes property TomNicholas 35968931 closed 0     2 2021-10-26T15:51:09Z 2023-10-20T16:00:15Z 2021-10-29T18:12:22Z MEMBER   0 pydata/xarray/pulls/5900

Adds a new .chunksizes property to Dataset, DataArray and Variable, which returns a mapping from dimensions names to chunk sizes in all cases.

Supercedes #5846 because this PR is backwards-compatible.

  • [x] Closes #5843
  • [x] Tests added
  • [x] Passes pre-commit run --all-files
  • [x] User visible changes (including notable bug fixes) are documented in whats-new.rst
  • [x] New functions/methods are listed in api.rst
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5900/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1790161818 PR_kwDOAMm_X85UvI4i 7963 Suggest installing dask when not discovered by ChunkManager TomNicholas 35968931 open 0     2 2023-07-05T19:34:06Z 2023-10-16T13:31:44Z   MEMBER   0 pydata/xarray/pulls/7963
  • [x] Closes #7962
  • [ ] 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/7963/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1083507645 PR_kwDOAMm_X84wBDeq 6083 Manifest as variables attribute TomNicholas 35968931 closed 0     2 2021-12-17T18:14:26Z 2023-09-14T15:37:38Z 2023-09-14T15:37:37Z MEMBER   1 pydata/xarray/pulls/6083

Another attempt like #5961

@shoyer

  • [ ] Closes #xxxx
  • [ ] Tests added
  • [ ] Passes pre-commit run --all-files
  • [ ] 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/6083/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1695244129 PR_kwDOAMm_X85PvJSS 7815 Array API fixes for astype TomNicholas 35968931 closed 0     2 2023-05-04T04:33:52Z 2023-05-18T20:10:48Z 2023-05-18T20:10:43Z MEMBER   0 pydata/xarray/pulls/7815

While it's common for duck arrays to have a .astype method, this doesn't exist in the new array API standard. We now have duck_array_ops.astype to deal with this, but for some reason changing it in just a couple more places broke practically every pint test in test_units.py :confused: @keewis

Builds on top of #7019 with just one extra commit to separate out this issue.

  • [ ] 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/7815/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1337587854 PR_kwDOAMm_X849HJCV 6912 Automatic PR labeler TomNicholas 35968931 closed 0     2 2022-08-12T18:40:27Z 2022-08-12T19:52:49Z 2022-08-12T19:47:19Z MEMBER   0 pydata/xarray/pulls/6912

GH action to automatically label new PRs according to which files they touch.

Idea stolen from dask, see https://github.com/dask/dask/pull/7506 . Their PR labelling by file/module is specified here.

(My first use of this bot so might well be a mistake.)

@max-sixty you will probably enjoy this extra automation :robot:

  • [ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/6912/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1157289286 PR_kwDOAMm_X84z1Xnf 6319 v2022.03.0 release notes TomNicholas 35968931 closed 0     2 2022-03-02T14:43:34Z 2022-03-02T19:49:25Z 2022-03-02T15:49:23Z MEMBER   0 pydata/xarray/pulls/6319   
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/6319/reactions",
    "total_count": 4,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 4,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1041675013 PR_kwDOAMm_X84t8yv7 5924 v0.20 Release notes TomNicholas 35968931 closed 0     2 2021-11-01T21:53:29Z 2021-11-02T19:22:46Z 2021-11-02T16:37:45Z MEMBER   0 pydata/xarray/pulls/5924

@pydata/xarray the release notes for your approval

5889

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5924/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
1039714252 PR_kwDOAMm_X84t25p8 5916 Update open_rasterio deprecation version number TomNicholas 35968931 closed 0     2 2021-10-29T15:56:04Z 2021-11-02T18:03:59Z 2021-11-02T18:03:58Z MEMBER   0 pydata/xarray/pulls/5916   
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5916/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
959317311 MDExOlB1bGxSZXF1ZXN0NzAyNDQ5NDg1 5669 Combine='by_coords' and concat dim deprecation in open_mfdataset TomNicholas 35968931 closed 0     2 2021-08-03T17:03:44Z 2021-10-01T18:52:00Z 2021-10-01T18:52:00Z MEMBER   0 pydata/xarray/pulls/5669

Noticed this hadn't been completed in https://github.com/pydata/xarray/discussions/5659

  • [x] Tests added
  • [x] Passes pre-commit run --all-files
  • [x] User visible changes (including notable bug fixes) are documented in whats-new.rst
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5669/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
902830027 MDExOlB1bGxSZXF1ZXN0NjU0MTU2NTA5 5383 Corrected reference to blockwise to refer to apply_gufunc instead TomNicholas 35968931 closed 0     2 2021-05-26T19:23:53Z 2021-05-26T21:34:06Z 2021-05-26T21:34:06Z MEMBER   0 pydata/xarray/pulls/5383

I noticed that the apply_ufunc tutorial notebook says that xarray.apply_ufunc uses dask.array.blockwise, but that's no longer true as of PR #4060 .

  • [x] Passes pre-commit run --all-files
  • [ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5383/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
887597884 MDExOlB1bGxSZXF1ZXN0NjQwODE5NTMz 5289 Explained what a deprecation cycle is TomNicholas 35968931 closed 0     2 2021-05-11T15:15:08Z 2021-05-13T16:38:19Z 2021-05-13T16:38:19Z MEMBER   0 pydata/xarray/pulls/5289

Inspired by a question asked in #4696, but does not close that issue - [x] Passes pre-commit run --all-files - [x] User visible changes (including notable bug fixes) are documented in whats-new.rst

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5289/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
871234354 MDExOlB1bGxSZXF1ZXN0NjI2Mjg2ODQy 5237 Add deprecation warnings for lock kwarg TomNicholas 35968931 closed 0     2 2021-04-29T16:45:45Z 2021-05-04T19:17:31Z 2021-05-04T19:17:31Z MEMBER   0 pydata/xarray/pulls/5237

Does this need a test?

  • [x] Closes #5073
  • [ ] Tests added
  • [x] Passes pre-commit run --all-files
  • [x] User visible changes (including notable bug fixes) are documented in whats-new.rst
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/5237/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
604265838 MDExOlB1bGxSZXF1ZXN0NDA2ODkyMzc3 3993 dim -> coord in DataArray.integrate TomNicholas 35968931 closed 0     2 2020-04-21T20:30:35Z 2021-02-05T21:48:39Z 2021-01-29T22:59:30Z MEMBER   0 pydata/xarray/pulls/3993
  • [x] Closes #3992
  • [x] Tests added
  • [x] Passes isort -rc . && black . && mypy . && flake8
  • [x] Fully documented, including whats-new.rst for all changes and api.rst for new API
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/3993/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
599597677 MDExOlB1bGxSZXF1ZXN0NDAzMjA0MTI5 3970 keep attrs in interpolate_na TomNicholas 35968931 closed 0     2 2020-04-14T14:02:33Z 2020-04-17T20:16:27Z 2020-04-17T20:16:27Z MEMBER   0 pydata/xarray/pulls/3970

dataarray.interpolate_na was dropping attrs because even though the internal apply_ufunc call was being passed keep_attrs=True, the order of arguments index, da to apply_ufunc meant that it was trying to keep only the non-existent attrs from the first argument. I just swapped them round, and added a globally-aware keep_attrs kwarg.

  • [x] Closes #3968
  • [x] Tests added
  • [x] Passes isort -rc . && black . && mypy . && flake8
  • [x] Fully documented, including whats-new.rst for all changes and api.rst for new API
 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/3970/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
595602834 MDExOlB1bGxSZXF1ZXN0NDAwMDQ4MTEz 3943 Updated list of core developers TomNicholas 35968931 closed 0     2 2020-04-07T05:31:55Z 2020-04-07T19:28:37Z 2020-04-07T19:28:25Z MEMBER   0 pydata/xarray/pulls/3943

Names listed in order of date added.

  • [x] Closes #3892
  • [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/3943/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
     xarray 13221727 pull
460674994 MDExOlB1bGxSZXF1ZXN0MjkxNzU1NjA4 3043 Rename combine functions TomNicholas 35968931 closed 0     2 2019-06-25T22:31:40Z 2019-06-26T15:01:01Z 2019-06-26T15:00:38Z MEMBER   0 pydata/xarray/pulls/3043

Finishes #2616 by renaming combine_manual -> combine_nested, combine_auto -> combine_by_coords, and renaming the arguments to open_mfdataset: combine='manual' -> combine='nested' combine='auto' -> combine='by_coords'

[x] code changes [x] docs changes [x] what's new updated

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/3043/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]);