issue_comments
9 rows where author_association = "MEMBER", issue = 443157666 and user = 14808389 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: reactions, created_at (date), updated_at (date)
issue 1
- Picking up #1118: Do not convert subclasses of `ndarray` unless required · 9 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
568903698 | https://github.com/pydata/xarray/pull/2956#issuecomment-568903698 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDU2ODkwMzY5OA== | keewis 14808389 | 2019-12-25T14:11:45Z | 2019-12-25T14:11:45Z | MEMBER | Since we have been making a lot of progress using |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 | |
524881974 | https://github.com/pydata/xarray/pull/2956#issuecomment-524881974 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDUyNDg4MTk3NA== | keewis 14808389 | 2019-08-26T14:26:34Z | 2019-08-26T14:26:34Z | MEMBER |
@shoyer: What kind of testing would that be? If this is something that would be removed again in the future and has no actual value, feel free to close (but don't forget the original PR). Otherwise, I'd be happy to continue working on this. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 | |
523242520 | https://github.com/pydata/xarray/pull/2956#issuecomment-523242520 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDUyMzI0MjUyMA== | keewis 14808389 | 2019-08-21T00:08:53Z | 2019-08-21T00:08:53Z | MEMBER | then I would like to finish this one and modify the tests in a new one. |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 | |
523231052 | https://github.com/pydata/xarray/pull/2956#issuecomment-523231052 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDUyMzIzMTA1Mg== | keewis 14808389 | 2019-08-20T23:15:00Z | 2019-08-20T23:15:00Z | MEMBER | I think I would like to tackle this, but I definitely will need some help with finding examples for some of the more advanced operations. @shoyer should I add these in a new PR or should I repurpose this one? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 | |
523229039 | https://github.com/pydata/xarray/pull/2956#issuecomment-523229039 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDUyMzIyOTAzOQ== | keewis 14808389 | 2019-08-20T23:06:27Z | 2019-08-20T23:07:04Z | MEMBER | Do you have a rough idea of which operations these tests should check? The ones currently implemented are |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 | |
523183790 | https://github.com/pydata/xarray/pull/2956#issuecomment-523183790 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDUyMzE4Mzc5MA== | keewis 14808389 | 2019-08-20T20:35:02Z | 2019-08-20T20:35:02Z | MEMBER | I'm not too sure about putting However, the question of whether or not this PR is obsolete now still remains: if one were to replace |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 | |
520632305 | https://github.com/pydata/xarray/pull/2956#issuecomment-520632305 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDUyMDYzMjMwNQ== | keewis 14808389 | 2019-08-12T23:32:50Z | 2019-08-12T23:32:50Z | MEMBER | now that #3117 has been merged, what is the status on this? From what I can tell from the few I looked at, most ndarray subclasses either already use NEP18 or will do so in the future. I guess what I want to know is: should I continue to work on it or does it overlap too much with that PR? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 | |
494151339 | https://github.com/pydata/xarray/pull/2956#issuecomment-494151339 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDQ5NDE1MTMzOQ== | keewis 14808389 | 2019-05-20T21:07:06Z | 2019-05-20T21:07:06Z | MEMBER | I added the option, though the documentation is minimal. It gets used by a function to decide between The Is the failure of the docs related to any of the changes I made? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 | |
493803589 | https://github.com/pydata/xarray/pull/2956#issuecomment-493803589 | https://api.github.com/repos/pydata/xarray/issues/2956 | MDEyOklzc3VlQ29tbWVudDQ5MzgwMzU4OQ== | keewis 14808389 | 2019-05-20T00:01:26Z | 2019-05-20T00:01:26Z | MEMBER | Most of the items above are implemented in unit-support, even support for array wrappers with units like I did not push these commits to this PR because most of it does not have anything to do with subclasses. I did not open a new PR for the branch either because it builds on this one and because I'd like to receive feedback on any of this first. Combining this with |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Picking up #1118: Do not convert subclasses of `ndarray` unless required 443157666 |
Advanced export
JSON shape: default, array, newline-delimited, object
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]);
user 1