issue_comments

Table actions

3 rows where author_association = "MEMBER", issue = 473142248 and user = 3924836 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

  • scottyhq · 3 ✖

issue 1

  • changed url for rasterio network test · 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
516633175 https://github.com/pydata/xarray/pull/3162#issuecomment-516633175 https://api.github.com/repos/pydata/xarray/issues/3162 MDEyOklzc3VlQ29tbWVudDUxNjYzMzE3NQ== scottyhq 3924836 2019-07-30T23:26:07Z 2019-07-30T23:26:23Z MEMBER

ok. as noted https://github.com/mapbox/rasterio/pull/1709 the aws_unsigned=True should bypass importing boto3 in upcoming rasterio 1.0.25. But I've gone ahead and added boto3 to ci requirements.yml files in case people want to add other tests reading from s3, gcs, etc. in the future

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   changed url for rasterio network test 473142248
515588027 https://github.com/pydata/xarray/pull/3162#issuecomment-515588027 https://api.github.com/repos/pydata/xarray/issues/3162 MDEyOklzc3VlQ29tbWVudDUxNTU4ODAyNw== scottyhq 3924836 2019-07-26T20:24:32Z 2019-07-26T20:24:32Z MEMBER

I'm in no rush to merge this (just wanted to help get rid of the red on Azure :). I think it's important to have network tests since it's increasingly common to be streaming data from s3, gcs, etc rather than reading locally. But I'm not sure of best practices for setting such tests up, so any suggestions for modifications are welcome.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   changed url for rasterio network test 473142248
515452425 https://github.com/pydata/xarray/pull/3162#issuecomment-515452425 https://api.github.com/repos/pydata/xarray/issues/3162 MDEyOklzc3VlQ29tbWVudDUxNTQ1MjQyNQ== scottyhq 3924836 2019-07-26T13:30:04Z 2019-07-26T14:00:48Z MEMBER

but seems like the tests are now run with each build on Azure? https://dev.azure.com/xarray/xarray/_build/results?buildId=389&view=ms.vss-test-web.build-test-results-tab

the new test currently still fails - I think due to https://github.com/mapbox/rasterio/pull/1709

cc @shoyer who created the issue linked in the first post. 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   changed url for rasterio network test 473142248

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