issue_comments

Table actions

3 rows where issue = 1561508426 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

user 2

author_association 2

issue 1

  • Opening datasets with large object dtype arrays is very slow · 3 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1409323892 https://github.com/pydata/xarray/issues/7484#issuecomment-1409323892 https://api.github.com/repos/pydata/xarray/issues/7484 IC_kwDOAMm_X85UAJN0 Illviljan 14371165 2023-01-30T20:59:36Z 2023-01-30T21:00:40Z MEMBER

You can do var._data instead of var.data. There's been a few cases recently where self.data doesn't play so nice when reading from a file.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Opening datasets with large object dtype arrays is very slow 1561508426
1409299311 https://github.com/pydata/xarray/issues/7484#issuecomment-1409299311 https://api.github.com/repos/pydata/xarray/issues/7484 IC_kwDOAMm_X85UADNv agoodm 5179430 2023-01-30T20:36:46Z 2023-01-30T20:36:46Z CONTRIBUTOR

Great, thanks! It's actually the var.data attribute access itself that's triggering the loading so that's why I needed to put the change there, but I see your point that I should probably update contains_cftime_datetimes as well since selecting the first element again is stylistically redundant. In any case, I'll go ahead and quickly get to work at preparing a PR for this.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Opening datasets with large object dtype arrays is very slow 1561508426
1409243024 https://github.com/pydata/xarray/issues/7484#issuecomment-1409243024 https://api.github.com/repos/pydata/xarray/issues/7484 IC_kwDOAMm_X85T_1eQ Illviljan 14371165 2023-01-30T19:53:03Z 2023-01-30T19:53:03Z MEMBER

Feel free to start working on that PR. 👍 It looks like _contains_cftime_datetimes tries to do a similar thing as your solution so I think the change should be there.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Opening datasets with large object dtype arrays is very slow 1561508426

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