issue_comments

Table actions

6 rows where issue = 988426640 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 2

issue 1

  • Open mfdataset with long time axis · 6 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
914754778 https://github.com/pydata/xarray/pull/5767#issuecomment-914754778 https://api.github.com/repos/pydata/xarray/issues/5767 IC_kwDOAMm_X842hgza znicholls 23625371 2021-09-08T01:06:28Z 2021-09-08T01:06:28Z CONTRIBUTOR

Thanks @TomNicholas and @mathause. I've opened a new PR (#5776), turns out all the tests are well setup already, I just needed to be told where to look. Cheers!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Open mfdataset with long time axis 988426640
914653049 https://github.com/pydata/xarray/pull/5767#issuecomment-914653049 https://api.github.com/repos/pydata/xarray/issues/5767 IC_kwDOAMm_X842hH95 mathause 10194086 2021-09-07T21:53:44Z 2021-09-07T21:53:44Z MEMBER

Maybe something like this:

python attrs = dict(units="days since 1850-01-01", calendar="proleptic_gregorian") ds1 = xr.Dataset(coords=dict(time=("time", [164678], attrs))) xr.conventions.decode_cf(ds1)

(or you can directly create two time arrays as suggested by @TomNicholas )

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Open mfdataset with long time axis 988426640
913090831 https://github.com/pydata/xarray/pull/5767#issuecomment-913090831 https://api.github.com/repos/pydata/xarray/issues/5767 IC_kwDOAMm_X842bKkP github-actions[bot] 41898282 2021-09-05T05:43:30Z 2021-09-05T07:52:46Z CONTRIBUTOR

Unit Test Results

6 files           6 suites   54m 53s :stopwatch: 16 229 tests 14 494 :heavy_check_mark: 1 735 :zzz: 0 :x: 90 570 runs  82 393 :heavy_check_mark: 8 177 :zzz: 0 :x:

Results for commit 548644aa.

:recycle: This comment has been updated with latest results.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Open mfdataset with long time axis 988426640
913098489 https://github.com/pydata/xarray/pull/5767#issuecomment-913098489 https://api.github.com/repos/pydata/xarray/issues/5767 IC_kwDOAMm_X842bMb5 znicholls 23625371 2021-09-05T07:07:16Z 2021-09-05T07:07:16Z CONTRIBUTOR

use_cftime=True

I knew there must be a more elegant option, thanks!

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Open mfdataset with long time axis 988426640
913094980 https://github.com/pydata/xarray/pull/5767#issuecomment-913094980 https://api.github.com/repos/pydata/xarray/issues/5767 IC_kwDOAMm_X842bLlE mathause 10194086 2021-09-05T06:31:41Z 2021-09-05T06:31:41Z MEMBER

I like the idea. I would still raise an error, though.Maybe: "Found a mix of pandas and cftime datetime data types. Re-opening the data with the option use_cftime=True may fix this issue." That would be less magic and should tell the users what's going wrong. In addition we would not need an additional keyword argument. I expect a merge_by_coords operation to be typically fairly early in the analysis pipeline.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Open mfdataset with long time axis 988426640
913089052 https://github.com/pydata/xarray/pull/5767#issuecomment-913089052 https://api.github.com/repos/pydata/xarray/issues/5767 IC_kwDOAMm_X842bKIc znicholls 23625371 2021-09-05T05:19:45Z 2021-09-05T05:19:45Z CONTRIBUTOR

@mathause what do you think?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Open mfdataset with long time axis 988426640

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