issue_comments

Table actions

4 rows where author_association = "MEMBER" and issue = 1243123406 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

  • CFTime support for polyval · 4 ✖

author_association 1

  • MEMBER · 4 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1142421179 https://github.com/pydata/xarray/pull/6624#issuecomment-1142421179 https://api.github.com/repos/pydata/xarray/issues/6624 IC_kwDOAMm_X85EF_a7 spencerkclark 6628425 2022-05-31T17:34:58Z 2022-05-31T17:34:58Z MEMBER

Many thanks for restoring this functionality @headtr1ck!

 
{
    "total_count": 1,
    "+1": 0,
    "-1": 0,
    "laugh": 1,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   CFTime support for polyval 1243123406
1140140458 https://github.com/pydata/xarray/pull/6624#issuecomment-1140140458 https://api.github.com/repos/pydata/xarray/issues/6624 IC_kwDOAMm_X85D9Smq dcherian 2448579 2022-05-28T02:14:54Z 2022-05-28T02:14:54Z MEMBER

🤦🏾 the failure is because pytest is running the xr.date_range(..., use_cftime=True) always, ignoring the skipif mark.

@headtr1ck Can you add the cftime tests as a separate test instead? This will avoid the need for a max_computes arg too. For the separate cftime test, apply the requires_cftime decorator from tests/__init__.py to skip when cftime is not present.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   CFTime support for polyval 1243123406
1140135221 https://github.com/pydata/xarray/pull/6624#issuecomment-1140135221 https://api.github.com/repos/pydata/xarray/issues/6624 IC_kwDOAMm_X85D9RU1 dcherian 2448579 2022-05-28T01:55:52Z 2022-05-28T01:55:52Z MEMBER

Is it because before only indexes were supported as coord and they are never lazy?

Yes.

The test failure is only because we are trying to test cftime in an environment without cftime.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   CFTime support for polyval 1243123406
1133072138 https://github.com/pydata/xarray/pull/6624#issuecomment-1133072138 https://api.github.com/repos/pydata/xarray/issues/6624 IC_kwDOAMm_X85DiU8K dcherian 2448579 2022-05-20T16:03:26Z 2022-05-20T16:03:26Z MEMBER

I squash merged #6556, but you'll still need to update the max_computes in raise_if_dask_computes for cftime tests. It should only have to compute once.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   CFTime support for polyval 1243123406

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