issue_comments

Table actions

7 rows where author_association = "MEMBER" and issue = 820480606 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

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

issue 1

  • A read-only TileDB backend · 7 ✖

author_association 1

  • MEMBER · 7 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
796925819 https://github.com/pydata/xarray/issues/4987#issuecomment-796925819 https://api.github.com/repos/pydata/xarray/issues/4987 MDEyOklzc3VlQ29tbWVudDc5NjkyNTgxOQ== max-sixty 5635139 2021-03-11T17:55:28Z 2021-03-11T17:55:28Z MEMBER

Not at this point. We just released 0.17 so I would think we're at least a week or two away from 0.18.

Just to set expectations — I hadn't thought we were releasing so soon — though I'd be happy to, and it's getting easier & easier to do releases.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   A read-only TileDB backend 820480606
796919656 https://github.com/pydata/xarray/issues/4987#issuecomment-796919656 https://api.github.com/repos/pydata/xarray/issues/4987 MDEyOklzc3VlQ29tbWVudDc5NjkxOTY1Ng== jhamman 2443309 2021-03-11T17:46:18Z 2021-03-11T17:46:18Z MEMBER

Not at this point. We just released 0.17 so I would think we're at least a week or two away from 0.18. 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   A read-only TileDB backend 820480606
791681130 https://github.com/pydata/xarray/issues/4987#issuecomment-791681130 https://api.github.com/repos/pydata/xarray/issues/4987 MDEyOklzc3VlQ29tbWVudDc5MTY4MTEzMA== jhamman 2443309 2021-03-05T20:52:06Z 2021-03-05T20:52:06Z MEMBER

4989 includes the full refactor. The plan is to merge this to xarray/master on Monday.

 
{
    "total_count": 1,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 1,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   A read-only TileDB backend 820480606
790782639 https://github.com/pydata/xarray/issues/4987#issuecomment-790782639 https://api.github.com/repos/pydata/xarray/issues/4987 MDEyOklzc3VlQ29tbWVudDc5MDc4MjYzOQ== alexamici 226037 2021-03-04T17:16:48Z 2021-03-04T17:22:31Z MEMBER

@jp-dark it is in fact possible to write an xarray backend without explicitly depending on xarray in your setup.py if you put all your backend glue code in a separate module not imported by the main __init__.py.

We use the setuptools entrypoints infrastructure that triggers a module load only from within xarray itself.

This is still work in progress, but we are implementing this strategy in cfgrib with success. You can get inspiration from the following PR by @aurghs:

https://github.com/ecmwf/cfgrib/pull/203

 
{
    "total_count": 2,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 2,
    "rocket": 0,
    "eyes": 0
}
   A read-only TileDB backend 820480606
790140398 https://github.com/pydata/xarray/issues/4987#issuecomment-790140398 https://api.github.com/repos/pydata/xarray/issues/4987 MDEyOklzc3VlQ29tbWVudDc5MDE0MDM5OA== shoyer 1217238 2021-03-03T23:12:37Z 2021-03-03T23:12:37Z MEMBER

@jp-dark How would you feel about writing another small library, e.g., "xarray-tiledb" that can explicitly depend on both xarray and tiledb?

We can potentially do some of xarray's backend stuff with protocols, but there are some aspects (especially for more advanced features like lazy loading) that will likely need the hard xarray dependency.

 
{
    "total_count": 2,
    "+1": 2,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   A read-only TileDB backend 820480606
790024577 https://github.com/pydata/xarray/issues/4987#issuecomment-790024577 https://api.github.com/repos/pydata/xarray/issues/4987 MDEyOklzc3VlQ29tbWVudDc5MDAyNDU3Nw== jhamman 2443309 2021-03-03T20:13:44Z 2021-03-03T20:13:44Z MEMBER

This is a great spot @jp-dark! Looking forward to seeing your progress.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   A read-only TileDB backend 820480606
789933725 https://github.com/pydata/xarray/issues/4987#issuecomment-789933725 https://api.github.com/repos/pydata/xarray/issues/4987 MDEyOklzc3VlQ29tbWVudDc4OTkzMzcyNQ== jhamman 2443309 2021-03-03T17:52:36Z 2021-03-03T17:54:38Z MEMBER

Hi @jp-dark! Thanks for opening this issue and the draft pull request in #4988. As you probably know, we're in the process of completing a major refactor of our storage backends system (see #4989 and #4810 for the current state of that work). One of the main feature additions in this work is the new entrypoints functionality which will allow backends (like TileDB) to declare backend support without including the code in the Xarray itself.

In light of this new functionality, we'd like to see if we can put the TileDB backend in TileDB itself (or in an another adjacent package). The end user functionality would be the same as the entrypoint would be registered at install time. We'd be happy to document the TileDB in the Xarray documentation as well.

This is the development pattern we are headed to with most of our backends, including some of the current backends. We'd be happy to help work with you to sort out the details as I'm sure there will be one or two early adopter bumps to work through.

cc @alexamici @aurghs @shoyer 

{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   A read-only TileDB backend 820480606

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