issue_comments
9 rows where author_association = "CONTRIBUTOR" and issue = 656163384 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: created_at (date), updated_at (date)
issue 1
- Change isinstance checks to duck Dask Array checks #4208 · 9 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
685920915 | https://github.com/pydata/xarray/pull/4221#issuecomment-685920915 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY4NTkyMDkxNQ== | rpmanser 19578931 | 2020-09-02T18:32:45Z | 2020-09-02T18:32:45Z | CONTRIBUTOR | @dcherian No problem! Thanks for all your help! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 | |
684052741 | https://github.com/pydata/xarray/pull/4221#issuecomment-684052741 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY4NDA1Mjc0MQ== | rpmanser 19578931 | 2020-08-31T21:31:51Z | 2020-08-31T21:31:51Z | CONTRIBUTOR |
No worries, thanks for the reviews! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 | |
682306752 | https://github.com/pydata/xarray/pull/4221#issuecomment-682306752 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY4MjMwNjc1Mg== | rpmanser 19578931 | 2020-08-28T03:37:36Z | 2020-08-28T03:37:36Z | CONTRIBUTOR | I meant |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 | |
681182191 | https://github.com/pydata/xarray/pull/4221#issuecomment-681182191 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY4MTE4MjE5MQ== | rpmanser 19578931 | 2020-08-26T23:54:18Z | 2020-08-26T23:54:18Z | CONTRIBUTOR | Tests are failing in I'm not sure why FormattingBlack failed. It's claiming that 45 files would be reformatted. I used the pre-commit hook, which passed. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 | |
661226607 | https://github.com/pydata/xarray/pull/4221#issuecomment-661226607 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY2MTIyNjYwNw== | rpmanser 19578931 | 2020-07-20T17:35:31Z | 2020-07-20T17:35:31Z | CONTRIBUTOR | I have erred on the side of leaving the On another note, if it is appropriate to apply these changes:
I can do that as well. If not, should a new issue be opened for this? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 | |
660157829 | https://github.com/pydata/xarray/pull/4221#issuecomment-660157829 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY2MDE1NzgyOQ== | jthielen 3460034 | 2020-07-17T15:04:50Z | 2020-07-17T15:04:50Z | CONTRIBUTOR | @rpmanser For what it's worth, I'd think doing tests like |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 | |
658946116 | https://github.com/pydata/xarray/pull/4221#issuecomment-658946116 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY1ODk0NjExNg== | rpmanser 19578931 | 2020-07-15T19:01:17Z | 2020-07-15T19:01:17Z | CONTRIBUTOR | I'm getting a bit lost within the testing suite while trying to figure out the implications of this change and what kinds of tests are necessary for it. Would a test module for this
- Individually test all functions that include the newly implemented Any suggestions would be greatly appreciated. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 | |
658140708 | https://github.com/pydata/xarray/pull/4221#issuecomment-658140708 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY1ODE0MDcwOA== | jthielen 3460034 | 2020-07-14T12:03:05Z | 2020-07-14T12:03:19Z | CONTRIBUTOR |
As long as that doesn't break any of the current uses, I think that would be the best way forwards. This would require xarray to be on NumPy 1.16+ (in order to ensure
Although, while we're at it, do we also need more careful handling of upcast types? I'm not sure if there even are any out there right now (not sure if a HoloViews Dataset counts here), but that doesn't necessarily mean there never will be any. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 | |
657895835 | https://github.com/pydata/xarray/pull/4221#issuecomment-657895835 | https://api.github.com/repos/pydata/xarray/issues/4221 | MDEyOklzc3VlQ29tbWVudDY1Nzg5NTgzNQ== | jthielen 3460034 | 2020-07-14T00:22:09Z | 2020-07-14T00:22:09Z | CONTRIBUTOR | Also, a broader discussion that's I've seen hinted to in the past, but is brought to the forefront by this PR: how should xarray be checking for general duck array types it can wrap? Right now, it looks like there is a mix of
and it would be nice to bring consistency. However, both these checks seem like they'd let too many types through. For example, |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Change isinstance checks to duck Dask Array checks #4208 656163384 |
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 2