home / github

Menu
  • GraphQL API
  • Search all tables

issues

Table actions
  • GraphQL API for issues

14 rows where comments = 5, type = "pull" and user = 1217238 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)

type 1

  • pull · 14 ✖

state 1

  • closed 14

repo 1

  • xarray 14
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
645062817 MDExOlB1bGxSZXF1ZXN0NDM5NTg4OTU1 4178 Fix min_deps_check; revert to support numpy=1.14 and pandas=0.24 shoyer 1217238 closed 0     5 2020-06-25T00:37:19Z 2021-02-27T21:46:43Z 2021-02-27T21:46:42Z MEMBER   1 pydata/xarray/pulls/4178

Fixes the issue noticed in: https://github.com/pydata/xarray/pull/4175#issuecomment-649135372

Let's see if this passes CI...

  • [x] Passes isort -rc . && black . && mypy . && flake8
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/4178/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
613546626 MDExOlB1bGxSZXF1ZXN0NDE0MjgwMDEz 4039 Revise pull request template shoyer 1217238 closed 0     5 2020-05-06T19:08:19Z 2020-06-18T05:45:11Z 2020-06-18T05:45:10Z MEMBER   0 pydata/xarray/pulls/4039

See below for the new language, to clarify that documentation is only necessary for "user visible changes."

I added "including notable bug fixes" to indicate that minor bug fixes may not be worth noting (I was thinking of test-suite only fixes in this category) but perhaps that is too confusing.

cc @pydata/xarray for opinions!

  • [ ] Closes #xxxx
  • [ ] Tests added
  • [ ] Passes isort -rc . && black . && mypy . && flake8
  • [ ] Fully documented, including whats-new.rst for user visible changes (including notable bug fixes) and api.rst for new API
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/4039/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
612214951 MDExOlB1bGxSZXF1ZXN0NDEzMjIyOTEx 4028 Remove broken test for Panel with to_pandas() shoyer 1217238 closed 0     5 2020-05-04T22:41:42Z 2020-05-06T01:50:21Z 2020-05-06T01:50:21Z MEMBER   0 pydata/xarray/pulls/4028

We don't support creating a Panel with to_pandas() with any version of pandas at present, so this test was previous broken if pandas < 0.25 was installed.

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/4028/reactions",
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
479914290 MDExOlB1bGxSZXF1ZXN0MzA2NzExNDYx 3210 sparse=True option for from_dataframe and from_series shoyer 1217238 closed 0     5 2019-08-13T01:09:19Z 2019-08-27T16:04:13Z 2019-08-27T08:54:26Z MEMBER   0 pydata/xarray/pulls/3210

Fixes https://github.com/pydata/xarray/issues/3206

Example usage:

In [3]: import pandas as pd
   ...: import numpy as np
   ...: import xarray
   ...: df = pd.DataFrame({
   ...:     'w': range(10),
   ...:     'x': list('abcdefghij'),
   ...:     'y': np.arange(0, 100, 10),
   ...:     'z': np.ones(10),
   ...: }).set_index(['w', 'x', 'y'])
   ...:

In [4]: ds = xarray.Dataset.from_dataframe(df, sparse=True)

In [5]: ds.z.data
Out[5]: <COO: shape=(10, 10, 10), dtype=float64, nnz=10, fill_value=nan>
  • [x] Closes #3206, Closes #2139
  • [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/3210/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
427451138 MDExOlB1bGxSZXF1ZXN0MjY2MDQ4MzEw 2858 Various fixes for explicit Dataset.indexes shoyer 1217238 closed 0     5 2019-03-31T21:48:47Z 2019-04-04T22:59:48Z 2019-04-04T21:58:24Z MEMBER   0 pydata/xarray/pulls/2858

I've added internal consistency checks to the uses of assert_equal in our test suite, so this shouldn't happen again.

  • [x] Closes #2856, closes #2854
  • [x] Tests added
  • [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/2858/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
365961291 MDExOlB1bGxSZXF1ZXN0MjE5NzUyOTE3 2458 WIP: sketch of resample support for CFTimeIndex shoyer 1217238 closed 0     5 2018-10-02T15:44:36Z 2019-02-03T03:21:52Z 2019-02-03T03:21:52Z MEMBER   0 pydata/xarray/pulls/2458

Example usage:

```

import xarray times = xarray.cftime_range('2000', periods=30, freq='MS') da = xarray.DataArray(range(30), [('time', times)]) da.resample(time='1AS').mean() <xarray.DataArray (time: 3)> array([ 5.5, 17.5, 26.5]) Coordinates: * time (time) object 2001-01-01 00:00:00 ... 2003-01-01 00:00:00 ```

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2458/reactions",
    "total_count": 1,
    "+1": 0,
    "-1": 0,
    "laugh": 1,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
388977754 MDExOlB1bGxSZXF1ZXN0MjM3MTAyNjYz 2595 Close files when CachingFileManager is garbage collected shoyer 1217238 closed 0     5 2018-12-09T01:53:50Z 2018-12-23T20:11:35Z 2018-12-23T20:11:32Z MEMBER   0 pydata/xarray/pulls/2595

This frees users from needing to worry about this.

Using __del__ turned up to be easier than using weak references.

  • [x] Closes #2560
  • [x] Closes #2614
  • [x] Tests added
  • [x] Fully documented, including whats-new.rst for all changes and api.rst for new AP
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/2595/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
299601789 MDExOlB1bGxSZXF1ZXN0MTcwOTM0ODg1 1936 Tweak stickler config: ignore Python files in the docs & disable fixer shoyer 1217238 closed 0     5 2018-02-23T05:18:29Z 2018-02-25T20:51:42Z 2018-02-25T20:49:15Z MEMBER   0 pydata/xarray/pulls/1936

It doesn't always make sense to lint these files fully.

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1936/reactions",
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
272460887 MDExOlB1bGxSZXF1ZXN0MTUxNTc2MzA1 1705 Make Indexer classes not inherit from tuple. shoyer 1217238 closed 0     5 2017-11-09T07:08:27Z 2017-11-17T16:33:40Z 2017-11-14T03:32:34Z MEMBER   0 pydata/xarray/pulls/1705

I'm not entirely sure this is a good idea. The advantage is that it ensures that all our indexing code is entirely explicit: everything that reaches a backend must be an ExplicitIndexer. The downside is that it removes a bit of internal flexibility: we can't just use tuples in place of basic indexers anymore. On the whole, I think this is probably worth it but I would appreciate feedback.

@fujiisoup can you take a look?

  • [x] Tests added / passed
  • [x] Passes git diff upstream/master **/*py | flake8 --diff
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/1705/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
112928260 MDExOlB1bGxSZXF1ZXN0NDg1MzUxMTA= 637 size and aspect arguments for plotting methods even without faceting shoyer 1217238 closed 0     5 2015-10-23T02:10:06Z 2016-12-20T10:08:35Z 2016-12-20T10:08:35Z MEMBER   0 pydata/xarray/pulls/637

I was finding myself writting plt.figure(figsize=(x, y)) way too often. This will be a convenient shortcut.

Still needs tests.

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/637/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
173908126 MDExOlB1bGxSZXF1ZXN0ODMxOTM2NTI= 993 Coordinate -> IndexVariable and other deprecations shoyer 1217238 closed 0     5 2016-08-30T01:12:19Z 2016-09-01T21:56:07Z 2016-09-01T21:56:02Z MEMBER   0 pydata/xarray/pulls/993
  • Renamed the Coordinate class from xarray's low level API to IndexVariable. xref https://github.com/pydata/xarray/pull/947#issuecomment-238549129
  • Deprecated supplying coords as a dictionary to the DataArray constructor without also supplying an explicit dims argument. The old behavior encouraged relying on the iteration order of dictionaries, which is a bad practice (fixes #727).
  • Removed a number of methods deprecated since v0.7.0 or earlier: load_data, vars, drop_vars, dump, dumps and the variables keyword argument alias to Dataset.
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/993/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
98498103 MDExOlB1bGxSZXF1ZXN0NDEzOTQyNzY= 507 Add sel_points for point-wise indexing by label shoyer 1217238 closed 0     5 2015-08-01T01:52:52Z 2015-08-05T03:51:46Z 2015-08-05T03:51:44Z MEMBER   0 pydata/xarray/pulls/507

xref #475

Example usage:

``` In [1]: da = xray.DataArray(np.arange(56).reshape((7, 8)), ...: coords={'x': list('abcdefg'), ...: 'y': 10 * np.arange(8)}, ...: dims=['x', 'y']) ...:

In [2]: da Out[2]: <xray.DataArray (x: 7, y: 8)> array([[ 0, 1, 2, 3, 4, 5, 6, 7], [ 8, 9, 10, 11, 12, 13, 14, 15], [16, 17, 18, 19, 20, 21, 22, 23], [24, 25, 26, 27, 28, 29, 30, 31], [32, 33, 34, 35, 36, 37, 38, 39], [40, 41, 42, 43, 44, 45, 46, 47], [48, 49, 50, 51, 52, 53, 54, 55]]) Coordinates: * y (y) int64 0 10 20 30 40 50 60 70 * x (x) |S1 'a' 'b' 'c' 'd' 'e' 'f' 'g'

we can index by position along each dimension

In [3]: da.isel_points(x=[0, 1, 6], y=[0, 1, 0], dim='points') Out[3]: <xray.DataArray (points: 3)> array([ 0, 9, 48]) Coordinates: y (points) int64 0 10 0 x (points) |S1 'a' 'b' 'g' * points (points) int64 0 1 2

or equivalently by label

In [4]: da.sel_points(x=['a', 'b', 'g'], y=[0, 10, 0], dim='points') Out[4]: <xray.DataArray (points: 3)> array([ 0, 9, 48]) Coordinates: y (points) int64 0 10 0 x (points) |S1 'a' 'b' 'g' * points (points) int64 0 1 2 Bug fixes ```

cc @jhamman

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/507/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
43442970 MDExOlB1bGxSZXF1ZXN0MjE1NjU3Mjg= 236 WIP: convert to/from cdms2 variables shoyer 1217238 closed 0   0.3.2 836999 5 2014-09-22T08:48:52Z 2014-12-19T09:11:42Z 2014-12-19T09:11:39Z MEMBER   0 pydata/xarray/pulls/236

Fixes #133

@DamienIrving am I missing anything obvious here?

{
    "url": "https://api.github.com/repos/pydata/xarray/issues/236/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
    xarray 13221727 pull
27625970 MDExOlB1bGxSZXF1ZXN0MTI1NzI5OTE= 12 Stephan's sprintbattical shoyer 1217238 closed 0     5 2014-02-14T21:23:09Z 2014-08-04T00:03:21Z 2014-02-21T00:36:53Z MEMBER   0 pydata/xarray/pulls/12
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/12/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 96.845ms · About: xarray-datasette