issue_comments

Table actions

7 rows where issue = 124573556 sorted by updated_at descending

✖
✖

✎ View and edit SQL

This data as json, CSV (advanced)

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

user 3

author_association 2

issue 1

  • TST: add test for converting 0-len Series · 7 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
458492945 https://github.com/pydata/xarray/issues/697#issuecomment-458492945 https://api.github.com/repos/pydata/xarray/issues/697 MDEyOklzc3VlQ29tbWVudDQ1ODQ5Mjk0NQ== stale[bot] 26384082 2019-01-29T10:42:18Z 2019-01-29T10:42:18Z NONE

In order to maintain a list of currently relevant issues, we mark issues as stale after a period of inactivity If this issue remains relevant, please comment here; otherwise it will be marked as closed automatically

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   TST: add test for converting 0-len Series 124573556
178665973 https://github.com/pydata/xarray/issues/697#issuecomment-178665973 https://api.github.com/repos/pydata/xarray/issues/697 MDEyOklzc3VlQ29tbWVudDE3ODY2NTk3Mw== shoyer 1217238 2016-02-02T16:19:28Z 2016-02-02T16:19:28Z MEMBER

@MaximilianR Just updated the title of this issue to match

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   TST: add test for converting 0-len Series 124573556
178420783 https://github.com/pydata/xarray/issues/697#issuecomment-178420783 https://api.github.com/repos/pydata/xarray/issues/697 MDEyOklzc3VlQ29tbWVudDE3ODQyMDc4Mw== max-sixty 5635139 2016-02-02T07:23:05Z 2016-02-02T07:23:05Z MEMBER

TBC, this does work, even if it's not tested:

``` python

In [11]: xr.DataArray.from_series(pd.Series()) Out[11]: <xarray.DataArray (index: 0)> array([], dtype=float64) Coordinates: * index (index) object ```

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   TST: add test for converting 0-len Series 124573556
168527681 https://github.com/pydata/xarray/issues/697#issuecomment-168527681 https://api.github.com/repos/pydata/xarray/issues/697 MDEyOklzc3VlQ29tbWVudDE2ODUyNzY4MQ== max-sixty 5635139 2016-01-03T18:38:29Z 2016-01-03T18:38:29Z MEMBER

Ah yes, that goes through the same path as the empty DataFrame. Alas I didn't add a test, but it works

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   TST: add test for converting 0-len Series 124573556
168526843 https://github.com/pydata/xarray/issues/697#issuecomment-168526843 https://api.github.com/repos/pydata/xarray/issues/697 MDEyOklzc3VlQ29tbWVudDE2ODUyNjg0Mw== shoyer 1217238 2016-01-03T18:22:33Z 2016-01-03T18:22:33Z MEMBER

@jreback's first case uses the from_series constructor.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   TST: add test for converting 0-len Series 124573556
168524404 https://github.com/pydata/xarray/issues/697#issuecomment-168524404 https://api.github.com/repos/pydata/xarray/issues/697 MDEyOklzc3VlQ29tbWVudDE2ODUyNDQwNA== max-sixty 5635139 2016-01-03T18:03:33Z 2016-01-03T18:03:33Z MEMBER

@shoyer Was that ever a problem?

``` python In [36]: xray.DataArray(pd.Series()) Out[36]: <xray.DataArray (dim_0: 0)> array([], dtype=float64) Coordinates: * dim_0 (dim_0) object

In [37]: xray.DataArray(pd.Series(range(20))) Out[37]: <xray.DataArray (dim_0: 20)> array([ 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19]) Coordinates: * dim_0 (dim_0) int64 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

```

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   TST: add test for converting 0-len Series 124573556
168506834 https://github.com/pydata/xarray/issues/697#issuecomment-168506834 https://api.github.com/repos/pydata/xarray/issues/697 MDEyOklzc3VlQ29tbWVudDE2ODUwNjgzNA== shoyer 1217238 2016-01-03T15:44:29Z 2016-01-03T15:44:29Z MEMBER

These definitely should work.

@MaximilianR did your fix handle Series -> DataArray? Let's leave this open if not...

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   TST: add test for converting 0-len Series 124573556

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