issue_comments

Table actions

7 rows where author_association = "MEMBER" and issue = 707745196 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

issue 1

  • Raise error when datetime64 or timedelta64 values that are outside the valid range for ns precision are converted to ns precision · 7 ✖

author_association 1

  • MEMBER · 7 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
701106214 https://github.com/pydata/xarray/pull/4454#issuecomment-701106214 https://api.github.com/repos/pydata/xarray/issues/4454 MDEyOklzc3VlQ29tbWVudDcwMTEwNjIxNA== max-sixty 5635139 2020-09-30T01:31:34Z 2020-09-30T01:31:34Z MEMBER

Thanks a lot @andrewpauling ! 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Raise error when datetime64 or timedelta64 values that are outside the valid range for ns precision are converted to ns precision 707745196
701095631 https://github.com/pydata/xarray/pull/4454#issuecomment-701095631 https://api.github.com/repos/pydata/xarray/issues/4454 MDEyOklzc3VlQ29tbWVudDcwMTA5NTYzMQ== spencerkclark 6628425 2020-09-30T00:50:55Z 2020-09-30T00:50:55Z MEMBER

Thanks again @andrewpauling!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Raise error when datetime64 or timedelta64 values that are outside the valid range for ns precision are converted to ns precision 707745196
700914758 https://github.com/pydata/xarray/pull/4454#issuecomment-700914758 https://api.github.com/repos/pydata/xarray/issues/4454 MDEyOklzc3VlQ29tbWVudDcwMDkxNDc1OA== max-sixty 5635139 2020-09-29T18:54:54Z 2020-09-29T18:54:54Z MEMBER

Great, let's merge on green!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Raise error when datetime64 or timedelta64 values that are outside the valid range for ns precision are converted to ns precision 707745196
698670864 https://github.com/pydata/xarray/pull/4454#issuecomment-698670864 https://api.github.com/repos/pydata/xarray/issues/4454 MDEyOklzc3VlQ29tbWVudDY5ODY3MDg2NA== spencerkclark 6628425 2020-09-25T01:30:05Z 2020-09-25T23:56:23Z MEMBER

You're absolutely right @andrewpauling -- pandas currently doesn't seem to have overflow protection in the case of casting timedelta64 types. I wrongly assumed that it would! I went ahead and opened an issue there: https://github.com/pandas-dev/pandas/issues/36615. We'll see what they say.

For now maybe you can add the test and attach an expected failure decorator to it?

In researching this, I came across https://github.com/numpy/numpy/issues/16352; if that eventually gets addressed this might all be moot.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Raise error when datetime64 or timedelta64 values that are outside the valid range for ns precision are converted to ns precision 707745196
699204478 https://github.com/pydata/xarray/pull/4454#issuecomment-699204478 https://api.github.com/repos/pydata/xarray/issues/4454 MDEyOklzc3VlQ29tbWVudDY5OTIwNDQ3OA== max-sixty 5635139 2020-09-25T23:05:07Z 2020-09-25T23:05:07Z MEMBER

Excellent, thanks @andrewpauling ! LGTM

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Raise error when datetime64 or timedelta64 values that are outside the valid range for ns precision are converted to ns precision 707745196
698069611 https://github.com/pydata/xarray/pull/4454#issuecomment-698069611 https://api.github.com/repos/pydata/xarray/issues/4454 MDEyOklzc3VlQ29tbWVudDY5ODA2OTYxMQ== max-sixty 5635139 2020-09-24T02:13:29Z 2020-09-24T02:13:29Z MEMBER

It looks great! I made one suggestion.

Any thoughts from others?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Raise error when datetime64 or timedelta64 values that are outside the valid range for ns precision are converted to ns precision 707745196
698032306 https://github.com/pydata/xarray/pull/4454#issuecomment-698032306 https://api.github.com/repos/pydata/xarray/issues/4454 MDEyOklzc3VlQ29tbWVudDY5ODAzMjMwNg== max-sixty 5635139 2020-09-23T23:56:20Z 2020-09-23T23:56:20Z MEMBER

Thanks @andrewpauling , this looks good.

Could you add a test?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Raise error when datetime64 or timedelta64 values that are outside the valid range for ns precision are converted to ns precision 707745196

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