issue_comments

Table actions

3 rows where author_association = "MEMBER" and issue = 1377097243 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

Suggested facets: reactions, created_at (date), updated_at (date)

user 2

issue 1

  • Add parse_dims func · 3 ✖

author_association 1

  • MEMBER · 3 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1255518131 https://github.com/pydata/xarray/pull/7051#issuecomment-1255518131 https://api.github.com/repos/pydata/xarray/issues/7051 IC_kwDOAMm_X85K1a-z mathause 10194086 2022-09-22T20:31:13Z 2022-09-22T20:31:13Z MEMBER

For xr.dot it's also different:

python xr.dot(a, b, dims=None) # reduce over common dimensions xr.dot(a, b, dims=...) # reduce over all dimensions

 
{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add parse_dims func 1377097243
1252964437 https://github.com/pydata/xarray/pull/7051#issuecomment-1252964437 https://api.github.com/repos/pydata/xarray/issues/7051 IC_kwDOAMm_X85KrrhV max-sixty 5635139 2022-09-20T22:06:53Z 2022-09-20T22:06:53Z MEMBER

Nothing wrong with None, it is just pythons default.

The intention of replace_none=False was to leave None as None, which is important for some low level functions as they might be optimized (like numpy sum, which sums over all axes for None).

Not relevant to this PR but for background — it used to be that da.groupby() defaulted to None, and reduced all dimensions — now for a couple of years we require ....

But still da.sum() is equivalent to da.sum(...), which is arguably a bit incongruent with da.sum('a') reducing fewer dimensions than da.sum(['a','b']). But would be quite hard to change now.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add parse_dims func 1377097243
1252829551 https://github.com/pydata/xarray/pull/7051#issuecomment-1252829551 https://api.github.com/repos/pydata/xarray/issues/7051 IC_kwDOAMm_X85KrKlv max-sixty 5635139 2022-09-20T19:47:20Z 2022-09-20T19:47:20Z MEMBER

I always thought that these two methods are incompatible. But I guess ... Is synonym for None (= all dims) and if ... Appears in an iterable, it just means replace it with all leftover dims.

Yes! ... is the better synonym — None is somewhat an artifact of history. So +1 for replace_none, which maybe we can gradually turn to False by default over time.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Add parse_dims func 1377097243

Advanced export

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

CSV options: 

CREATE TABLE [issue_comments] (
   [html_url] TEXT,
   [issue_url] TEXT,
   [id] INTEGER PRIMARY KEY,
   [node_id] TEXT,
   [user] INTEGER REFERENCES [users]([id]),
   [created_at] TEXT,
   [updated_at] TEXT,
   [author_association] TEXT,
   [body] TEXT,
   [reactions] TEXT,
   [performed_via_github_app] TEXT,
   [issue] INTEGER REFERENCES [issues]([id])
);
CREATE INDEX [idx_issue_comments_issue]
    ON [issue_comments] ([issue]);
CREATE INDEX [idx_issue_comments_user]
    ON [issue_comments] ([user]);