issue_comments
5 rows where issue = 430460404 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: reactions, created_at (date), updated_at (date)
issue 1
- Reduce length of cftime resample tests · 5 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
482080335 | https://github.com/pydata/xarray/pull/2879#issuecomment-482080335 | https://api.github.com/repos/pydata/xarray/issues/2879 | MDEyOklzc3VlQ29tbWVudDQ4MjA4MDMzNQ== | spencerkclark 6628425 | 2019-04-11T11:41:42Z | 2019-04-11T11:41:42Z | MEMBER | I'm going to go ahead and merge this; we can revisit things again if we want to continue to speed these tests up. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Reduce length of cftime resample tests 430460404 | |
481229888 | https://github.com/pydata/xarray/pull/2879#issuecomment-481229888 | https://api.github.com/repos/pydata/xarray/issues/2879 | MDEyOklzc3VlQ29tbWVudDQ4MTIyOTg4OA== | jwenfai 8708062 | 2019-04-09T12:26:43Z | 2019-04-09T12:26:43Z | CONTRIBUTOR | Wow, that's quick. The updated tests look fine to me so go ahead and merge it. |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Reduce length of cftime resample tests 430460404 | |
481227438 | https://github.com/pydata/xarray/pull/2879#issuecomment-481227438 | https://api.github.com/repos/pydata/xarray/issues/2879 | MDEyOklzc3VlQ29tbWVudDQ4MTIyNzQzOA== | spencerkclark 6628425 | 2019-04-09T12:19:01Z | 2019-04-09T12:19:01Z | MEMBER | Thanks for having a look @jwenfai! I updated my PR following your suggestions; this added some more tests so now the total time is around 9 seconds. I opted for listing the pairs of initial and resample frequencies explicitly rather than use a function to dynamically set one based on the other (I feel like it makes the tests easier to understand; also writing a function that converts one type of frequency to another similar-length-but-different-type frequency is messy). |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Reduce length of cftime resample tests 430460404 | |
481195895 | https://github.com/pydata/xarray/pull/2879#issuecomment-481195895 | https://api.github.com/repos/pydata/xarray/issues/2879 | MDEyOklzc3VlQ29tbWVudDQ4MTE5NTg5NQ== | jwenfai 8708062 | 2019-04-09T10:29:05Z | 2019-04-09T10:29:05Z | CONTRIBUTOR | Thanks for taking on the task of shortening test times! If the coverage is the same, I think the rewritten tests should be good. Just two things I feel I should mention:
- Testing even and odd multiples for resampling frequencies for a frequency class (e.g., '11MS' and '12M' for monthlies)
I don't quite remember what the issue was but there were tests that passed for even/odd resampling frequencies but fail for the other. Perhaps the tests could be rewritten to (1) switch Both of the problems I mentioned might have been from the earliest iteration of CFTimeIndex resampling so they might have no relevance now. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Reduce length of cftime resample tests 430460404 | |
480871486 | https://github.com/pydata/xarray/pull/2879#issuecomment-480871486 | https://api.github.com/repos/pydata/xarray/issues/2879 | MDEyOklzc3VlQ29tbWVudDQ4MDg3MTQ4Ng== | shoyer 1217238 | 2019-04-08T15:06:19Z | 2019-04-08T15:06:19Z | MEMBER | Looks good to me. Thank you for looking into this promptly! Let's wait for @jwenfai to review before merging. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Reduce length of cftime resample tests 430460404 |
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 3