issue_comments

Table actions

3 rows where issue = 987551524 sorted by updated_at descending

✖
✖

✎ View and edit SQL

This data as json, CSV (advanced)

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

author_association 3

issue 1

  • Plotting of labelled data fails · 3 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1464348270 https://github.com/pydata/xarray/issues/5762#issuecomment-1464348270 https://api.github.com/repos/pydata/xarray/issues/5762 IC_kwDOAMm_X85XSC5u headtr1ck 43316012 2023-03-10T20:01:18Z 2023-03-10T20:01:18Z COLLABORATOR

Closing this issue in favor of #7593 since the provided MVCE in this issue now works.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Plotting of labelled data fails 987551524
912625730 https://github.com/pydata/xarray/issues/5762#issuecomment-912625730 https://api.github.com/repos/pydata/xarray/issues/5762 IC_kwDOAMm_X842ZZBC Illviljan 14371165 2021-09-03T15:33:45Z 2021-09-03T15:55:05Z MEMBER

I think if you try out 0.19 you'll find that this works, see #5464. I just tried your example with <s>master</s> main and it works.

Though I'm not opposed to removing it, it makes sense to me to just trust the backend will do its job. But there's always some weird edge case that doesn't always make it that easy.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Plotting of labelled data fails 987551524
912636489 https://github.com/pydata/xarray/issues/5762#issuecomment-912636489 https://api.github.com/repos/pydata/xarray/issues/5762 IC_kwDOAMm_X842ZbpJ dschwoerer 5637662 2021-09-03T15:49:40Z 2021-09-03T15:49:40Z CONTRIBUTOR

I tried it with master, and it failed. Trying with main worked :-D

I think there will be cases where the data is not suitable for plotting, where the new error will be less clear than the old one, but I still think that would be overall an improvement.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Plotting of labelled data fails 987551524

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