issue_comments
3 rows where author_association = "MEMBER", issue = 271017521 and user = 10050469 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: created_at (date), updated_at (date)
issue 1
- Remove netCDF dependency from rasterio backend tests · 3 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
341994764 | https://github.com/pydata/xarray/pull/1687#issuecomment-341994764 | https://api.github.com/repos/pydata/xarray/issues/1687 | MDEyOklzc3VlQ29tbWVudDM0MTk5NDc2NA== | fmaussion 10050469 | 2017-11-05T18:38:15Z | 2017-11-05T18:38:15Z | MEMBER | Looks good, thanks! Waiting for the tests to pass |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Remove netCDF dependency from rasterio backend tests 271017521 | |
341746579 | https://github.com/pydata/xarray/pull/1687#issuecomment-341746579 | https://api.github.com/repos/pydata/xarray/issues/1687 | MDEyOklzc3VlQ29tbWVudDM0MTc0NjU3OQ== | fmaussion 10050469 | 2017-11-03T15:59:30Z | 2017-11-03T15:59:30Z | MEMBER |
No I don't, but it's more a test for the backend than for the rasterio files. See e.g. https://github.com/pydata/xarray/blob/master/xarray/backends/rasterio_.py#L154 : this line was added by @gbrener and the roundtrip tests obviously forced him to cast the bool to an int . I don't know if its good or bad ;) |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Remove netCDF dependency from rasterio backend tests 271017521 | |
341740088 | https://github.com/pydata/xarray/pull/1687#issuecomment-341740088 | https://api.github.com/repos/pydata/xarray/issues/1687 | MDEyOklzc3VlQ29tbWVudDM0MTc0MDA4OA== | fmaussion 10050469 | 2017-11-03T15:38:52Z | 2017-11-03T15:38:52Z | MEMBER | My reasoning (I guess) was that we shouldn't create Dataset objects which cannot be serialized to NetCDF (for whatever reason, for example because we store rasterio objects of some kind), since NetCDF is the preferred way to serialize xarray objects. I don't mind removing these, but then the test names should be renamed too ( |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Remove netCDF dependency from rasterio backend tests 271017521 |
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