issue_comments

Table actions

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

  • Use getitem_with_mask in reindex_variables · 5 ✖

author_association 1

  • MEMBER · 5 ✖
id html_url issue_url node_id user created_at updated_at ▲ author_association body reactions performed_via_github_app issue
391497264 https://github.com/pydata/xarray/pull/1847#issuecomment-391497264 https://api.github.com/repos/pydata/xarray/issues/1847 MDEyOklzc3VlQ29tbWVudDM5MTQ5NzI2NA== shoyer 1217238 2018-05-23T21:09:43Z 2018-05-23T21:09:43Z MEMBER

@smithsp OK, I'd be happy to take a look if you're able to track it down into a self-contained example.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Use getitem_with_mask in reindex_variables 290320242
391196199 https://github.com/pydata/xarray/pull/1847#issuecomment-391196199 https://api.github.com/repos/pydata/xarray/issues/1847 MDEyOklzc3VlQ29tbWVudDM5MTE5NjE5OQ== shoyer 1217238 2018-05-23T02:00:56Z 2018-05-23T02:00:56Z MEMBER

@smithsp that link doesn't work. Did you mean to refer to something else?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Use getitem_with_mask in reindex_variables 290320242
364793621 https://github.com/pydata/xarray/pull/1847#issuecomment-364793621 https://api.github.com/repos/pydata/xarray/issues/1847 MDEyOklzc3VlQ29tbWVudDM2NDc5MzYyMQ== shoyer 1217238 2018-02-11T22:04:29Z 2018-02-11T22:04:29Z MEMBER

I added fixes (bool, float) -> object and (str, bytes) -> object, and also applied these to concatenate/stack as well as where in duck_array_ops.

From a user-facing perspective, this now fixes dtypes xarray.concat() for mixed dtypes as well.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Use getitem_with_mask in reindex_variables 290320242
364353009 https://github.com/pydata/xarray/pull/1847#issuecomment-364353009 https://api.github.com/repos/pydata/xarray/issues/1847 MDEyOklzc3VlQ29tbWVudDM2NDM1MzAwOQ== shoyer 1217238 2018-02-09T07:05:01Z 2018-02-09T07:05:01Z MEMBER

@fujiisoup Indeed, that seems to be the case. I made a fixed version of duck_array_ops.where(), relying on the new dtypes.result_type() function that's basically a fixed version of np.result_type().

I believe this is ready for review now.

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Use getitem_with_mask in reindex_variables 290320242
364293763 https://github.com/pydata/xarray/pull/1847#issuecomment-364293763 https://api.github.com/repos/pydata/xarray/issues/1847 MDEyOklzc3VlQ29tbWVudDM2NDI5Mzc2Mw== fujiisoup 6815844 2018-02-09T00:18:52Z 2018-02-09T00:19:33Z MEMBER

It looks that our where does not do type-casting (inheriting from numpy's where), ```python In [1]: import numpy as np ...: import xarray as xr ...: a = xr.Variable('x', np.array(['a', 'b', 'c']))

In [2]: a._getitem_with_mask([1, 2, -1]) Out[2]: <xarray.Variable (x: 3)> array(['b', 'c', 'nan'], dtype='<U32') ``` nan is converted to string.

I guess we need to cast arrays manually in _getitem_with_mask before where is applied. python In [9]: dtype, fill_value = xr.core.dtypes.maybe_promote(a.dtype) ...: a.astype(dtype)._getitem_with_mask([1, 2, -1]) Out[9]: <xarray.Variable (x: 3)> array(['b', 'c', nan], dtype=object)

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   Use getitem_with_mask in reindex_variables 290320242

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