issue_comments

Table actions

4 rows where author_association = "CONTRIBUTOR" and issue = 514191264 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

user 1

issue 1

  • Poor error message on Dataset.sum(axis=...) · 4 ✖

author_association 1

  • CONTRIBUTOR · 4 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
547619497 https://github.com/pydata/xarray/issues/3461#issuecomment-547619497 https://api.github.com/repos/pydata/xarray/issues/3461 MDEyOklzc3VlQ29tbWVudDU0NzYxOTQ5Nw== rpgoldman 3274 2019-10-29T20:40:35Z 2019-10-29T20:40:35Z CONTRIBUTOR

I guess so. Minor preference for checking for inappropriate numpy errors like this and giving a better error message, but I can see this would be difficult.

This won't be the first bewildering error message problem from **kwargs, and it won't be the last. This is a python problem, not an xarray one, per se.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Poor error message on Dataset.sum(axis=...) 514191264
547614650 https://github.com/pydata/xarray/issues/3461#issuecomment-547614650 https://api.github.com/repos/pydata/xarray/issues/3461 MDEyOklzc3VlQ29tbWVudDU0NzYxNDY1MA== rpgoldman 3274 2019-10-29T20:28:30Z 2019-10-29T20:28:30Z CONTRIBUTOR

@crusaderky Thanks. Yes, that was exactly what I was trying to do.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Poor error message on Dataset.sum(axis=...) 514191264
547614052 https://github.com/pydata/xarray/issues/3461#issuecomment-547614052 https://api.github.com/repos/pydata/xarray/issues/3461 MDEyOklzc3VlQ29tbWVudDU0NzYxNDA1Mg== rpgoldman 3274 2019-10-29T20:27:00Z 2019-10-29T20:27:00Z CONTRIBUTOR

@crusaderky I see what I did wrong. I mistakenly read the documentation page for DataArray.sum() instead of DataSet.sum(). The former does support an axis argument, and I believe it makes sense there.

Blacklisting these arguments sounds like a great idea, thanks.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Poor error message on Dataset.sum(axis=...) 514191264
547612958 https://github.com/pydata/xarray/issues/3461#issuecomment-547612958 https://api.github.com/repos/pydata/xarray/issues/3461 MDEyOklzc3VlQ29tbWVudDU0NzYxMjk1OA== rpgoldman 3274 2019-10-29T20:24:25Z 2019-10-29T20:24:25Z CONTRIBUTOR

@max-sixty I was trying to sum across the data variables, rather than across a dimension.

What I ended up doing was taking the data for each of the data variables and adding the resulting numpy arrays. This gave me the sum across the data variables, along the dim (each data variable is one dimensional and they share a single dimension).

I'm not sure how to suggest fixing the error message, because I'm not sure what xarray doesn't like.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Poor error message on Dataset.sum(axis=...) 514191264

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]);