issue_comments
8 rows where author_association = "MEMBER" and issue = 308039063 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: reactions, created_at (date), updated_at (date)
issue 1
- rolling: periodic · 8 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
482403368 | https://github.com/pydata/xarray/pull/2011#issuecomment-482403368 | https://api.github.com/repos/pydata/xarray/issues/2011 | MDEyOklzc3VlQ29tbWVudDQ4MjQwMzM2OA== | shoyer 1217238 | 2019-04-12T01:43:07Z | 2019-04-12T01:43:07Z | MEMBER |
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
rolling: periodic 308039063 | |
482402058 | https://github.com/pydata/xarray/pull/2011#issuecomment-482402058 | https://api.github.com/repos/pydata/xarray/issues/2011 | MDEyOklzc3VlQ29tbWVudDQ4MjQwMjA1OA== | shoyer 1217238 | 2019-04-12T01:36:11Z | 2019-04-12T01:36:11Z | MEMBER | It might make sense to start with the higher level |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
rolling: periodic 308039063 | |
482143007 | https://github.com/pydata/xarray/pull/2011#issuecomment-482143007 | https://api.github.com/repos/pydata/xarray/issues/2011 | MDEyOklzc3VlQ29tbWVudDQ4MjE0MzAwNw== | rabernat 1197350 | 2019-04-11T14:41:23Z | 2019-04-11T14:41:23Z | MEMBER | Rather than letting this PR lie dormant, maybe we should just move forward without dask support and raise an appropriate error if the user tries it with dask arrays. |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
rolling: periodic 308039063 | |
376145446 | https://github.com/pydata/xarray/pull/2011#issuecomment-376145446 | https://api.github.com/repos/pydata/xarray/issues/2011 | MDEyOklzc3VlQ29tbWVudDM3NjE0NTQ0Ng== | fujiisoup 6815844 | 2018-03-26T12:16:12Z | 2018-03-26T12:16:12Z | MEMBER | Ah, you are right. The only way to realize rolling.periodic with dask would be to manually attach a chunk in the other side to the boundaries, construct rolling window array and then remove unnecessary boundaries by slicing. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
rolling: periodic 308039063 | |
376122203 | https://github.com/pydata/xarray/pull/2011#issuecomment-376122203 | https://api.github.com/repos/pydata/xarray/issues/2011 | MDEyOklzc3VlQ29tbWVudDM3NjEyMjIwMw== | mathause 10194086 | 2018-03-26T10:37:00Z | 2018-03-26T10:37:00Z | MEMBER | Thanks for the discussion. I could not get it to work with dask - the problem is the 'padding before ghosting'. I see no way how to get the |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
rolling: periodic 308039063 | |
375835173 | https://github.com/pydata/xarray/pull/2011#issuecomment-375835173 | https://api.github.com/repos/pydata/xarray/issues/2011 | MDEyOklzc3VlQ29tbWVudDM3NTgzNTE3Mw== | fujiisoup 6815844 | 2018-03-24T01:07:32Z | 2018-03-24T01:07:32Z | MEMBER | Some comments.
We use this here https://github.com/pydata/xarray/blob/7c2c43ce6b1792e1193635ab9b64fd248266f632/xarray/core/dask_array_ops.py#L78-L79 I suspect this would be already working, but it would be nice if we can add some test cases.
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
rolling: periodic 308039063 | |
375833339 | https://github.com/pydata/xarray/pull/2011#issuecomment-375833339 | https://api.github.com/repos/pydata/xarray/issues/2011 | MDEyOklzc3VlQ29tbWVudDM3NTgzMzMzOQ== | fujiisoup 6815844 | 2018-03-24T00:45:25Z | 2018-03-24T01:03:50Z | MEMBER | Thanks for sending a PR.
Maybe better API would be
I don't think so. I think we can keep rolling as similar to that of pandas,
and this option should be specific to
I think at least we need to add this to the docstring of |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
rolling: periodic 308039063 | |
375677083 | https://github.com/pydata/xarray/pull/2011#issuecomment-375677083 | https://api.github.com/repos/pydata/xarray/issues/2011 | MDEyOklzc3VlQ29tbWVudDM3NTY3NzA4Mw== | mathause 10194086 | 2018-03-23T14:10:08Z | 2018-03-23T14:50:02Z | MEMBER | We would probably have to test how edit: or not, as |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
rolling: periodic 308039063 |
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 4