issue_comments

Table actions

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

issue 1

  • add MultiPandasIndex helper class · 2 ✖

author_association 1

id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
1283038653 https://github.com/pydata/xarray/pull/7182#issuecomment-1283038653 https://api.github.com/repos/pydata/xarray/issues/7182 IC_kwDOAMm_X85MeZ29 benbovy 4160723 2022-10-18T21:40:49Z 2022-10-18T21:40:49Z MEMBER

I wonder if it is possible to create a generic MultiIndex?

Hmm that could be possible but it think there are just too many possible edge cases for something generic like that.

In your specific example

python ds.set_xindex( ["a", "b"], MultiIndex([("a", PandasIndex), ("b", PandasIndex), (["a", "b"], BallTreeIndex)), )

we could probably use the BallTreeIndex for point-wise indexing (i.e., with ds.sel(a=xr.DataArray(...), b=xr.DataArray(...))) and use the two PandasIndex instances for other kinds of selection (e.g., with slices, scalars, etc.) so there's no conflict, but I doubt this would be what we want in other cases.

I guess your suggestion is a way around the constraint in the Xarray data model that a coordinate cannot have multiple indexes? I'm afraid there's no easy solution that is generic enough. Maybe some cache to avoid rebuilding the indexes? I.e., .set_xindex() doesn't drop the pre-existing index(es) but rather disable them so that it is possible to re-enable them later with another .set_xindex() call (.xindexes only returns the "active" indexes but there may be other "inactive" indexes attached to a dataset).

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   add MultiPandasIndex helper class 1412901282
1282857157 https://github.com/pydata/xarray/pull/7182#issuecomment-1282857157 https://api.github.com/repos/pydata/xarray/issues/7182 IC_kwDOAMm_X85MdtjF keewis 14808389 2022-10-18T18:46:05Z 2022-10-18T21:15:42Z MEMBER

I wonder if it is possible to create a generic MultiIndex? Something like python ds.set_xindex( ["a", "b"], MultiIndex([("a", PandasIndex), ("b", PandasIndex), (["a", "b"], BallTreeIndex)), ) (but I'm sure we can find a better syntax... maybe create a hashable sequence that is not a tuple, since that is already taken? and change the list of tuples to a dict).

For that concept to work, the return value of MultiIndex would have to be a factory function / class, which would instantiate the actual index.

What do you think?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   add MultiPandasIndex helper class 1412901282

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