issue_comments

Table actions

2 rows where issue = 373742432 and user = 5635139 sorted by updated_at descending

✎ View and edit SQL

This data as json, CSV (advanced)

user 1

  • max-sixty · 2 ✖

issue 1

  • Remove .T as shortcut for transpose() · 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
433233234 https://github.com/pydata/xarray/pull/2509#issuecomment-433233234 https://api.github.com/repos/pydata/xarray/issues/2509 MDEyOklzc3VlQ29tbWVudDQzMzIzMzIzNA== max-sixty 5635139 2018-10-25T23:00:15Z 2018-10-25T23:00:15Z MEMBER

I just wanted to illustrate that these are the same problems and that only removing the property from the Dataset may lead to continued confusion.

I think that's a good point. There are a few methods on DataArrays but not Datasets, but not many:

python In [4]: {x for x in set(dir(xr.DataArray)) - set(dir(xr.Dataset)) if not x.startswith("_")} Out[4]: {'data', 'dot', 'dt', 'dtype', 'from_cdms2', 'from_iris', 'from_series', 'get_axis_num', 'item', 'name', 'ndim', 'plot', 'searchsorted', 'shape', 'size', 'to_cdms2', 'to_dataset', 'to_index', 'to_iris', 'to_masked_array', 'to_pandas', 'to_series', 'variable'}

Regardless, sounds like @shoyer thinks we should merge (and I'm probably still +0.1)

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Remove .T as shortcut for transpose() 373742432
433097642 https://github.com/pydata/xarray/pull/2509#issuecomment-433097642 https://api.github.com/repos/pydata/xarray/issues/2509 MDEyOklzc3VlQ29tbWVudDQzMzA5NzY0Mg== max-sixty 5635139 2018-10-25T15:30:55Z 2018-10-25T15:30:55Z MEMBER

I'm +0.2 for merging this, but at least issuing a DeprecationWarning before removing from DataArray.T (with no strong view on whether we do remove it)

 
{
    "total_count": 3,
    "+1": 3,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Remove .T as shortcut for transpose() 373742432

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