issue_comments

Table actions

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

issue 1

  • deprecate open_zarr · 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
1412408324 https://github.com/pydata/xarray/pull/7496#issuecomment-1412408324 https://api.github.com/repos/pydata/xarray/issues/7496 IC_kwDOAMm_X85UL6QE rabernat 1197350 2023-02-01T17:06:47Z 2023-02-01T17:06:47Z MEMBER

It is true that Xarray is now becoming very different from pandas in how it opens data.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   deprecate open_zarr 1564661430
1412396693 https://github.com/pydata/xarray/pull/7496#issuecomment-1412396693 https://api.github.com/repos/pydata/xarray/issues/7496 IC_kwDOAMm_X85UL3aV shoyer 1217238 2023-02-01T17:00:21Z 2023-02-01T17:00:21Z MEMBER

I like open_zarr(...) because it's less typing than open_dataset(..., engine='zarr'). The automatic backend detection logic doesn't currently work for Zarr, and in every case it adds overhead, which could be significant in the case of remote storage backends like Zarr.

So personally I would rather go the other direction and add open_netcdf().

The inconsistency in the chunks argument is non-ideal, but that could be handled by a separate deprecation process.

 
{
    "total_count": 5,
    "+1": 5,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   deprecate open_zarr 1564661430
1411070038 https://github.com/pydata/xarray/pull/7496#issuecomment-1411070038 https://api.github.com/repos/pydata/xarray/issues/7496 IC_kwDOAMm_X85UGzhW jhamman 2443309 2023-01-31T21:05:47Z 2023-01-31T21:05:47Z MEMBER

@paigem recently added the following to the open_dataset docstring (https://github.com/pydata/xarray/pull/7438):

In order to reproduce the default behavior of xr.open_zarr(...) use `xr.open_dataset(..., engine='zarr', chunks={})`.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   deprecate open_zarr 1564661430
1410959131 https://github.com/pydata/xarray/pull/7496#issuecomment-1410959131 https://api.github.com/repos/pydata/xarray/issues/7496 IC_kwDOAMm_X85UGYcb Illviljan 14371165 2023-01-31T19:38:59Z 2023-01-31T19:38:59Z MEMBER

It would be nice to have a good copy/paste example for open_dataset. For example I think chunks have different defaults compared to open_zarr.

 
{
    "total_count": 1,
    "+1": 1,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   deprecate open_zarr 1564661430

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