issue_comments
5 rows where issue = 594594646 and user = 1217238 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: created_at (date), updated_at (date)
issue 1
- Support multiple dimensions in DataArray.argmin() and DataArray.argmax() methods · 5 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
649204113 | https://github.com/pydata/xarray/pull/3936#issuecomment-649204113 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDY0OTIwNDExMw== | shoyer 1217238 | 2020-06-25T04:10:50Z | 2020-06-25T04:10:50Z | MEMBER | I'm going to wait a little while before merging in case anyone else has comment, but otherwise will merge in a day or two (definitely before the next release) |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Support multiple dimensions in DataArray.argmin() and DataArray.argmax() methods 594594646 | |
617002459 | https://github.com/pydata/xarray/pull/3936#issuecomment-617002459 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxNzAwMjQ1OQ== | shoyer 1217238 | 2020-04-21T07:23:11Z | 2020-04-21T07:23:27Z | MEMBER | I notice that some of your new deprecation warnings get issued when running your tests. Please silence these methods, e.g., using |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Support multiple dimensions in DataArray.argmin() and DataArray.argmax() methods 594594646 | |
616998622 | https://github.com/pydata/xarray/pull/3936#issuecomment-616998622 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxNjk5ODYyMg== | shoyer 1217238 | 2020-04-21T07:14:10Z | 2020-04-21T07:14:10Z | MEMBER |
I guess this would happen if somebody calls If it's the former, then we should just fix xarray not to do this. If it's only in external code, I would still consider breaking this behavior. The override mechanism that NumPy uses in argmin/argmax is really old/fragile, and it's not something we ever intended to support. There's no way we could write an The better way to do overrides in NumPy is probably with |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Support multiple dimensions in DataArray.argmin() and DataArray.argmax() methods 594594646 | |
610516784 | https://github.com/pydata/xarray/pull/3936#issuecomment-610516784 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxMDUxNjc4NA== | shoyer 1217238 | 2020-04-07T17:23:24Z | 2020-04-07T17:23:24Z | MEMBER | Also, feel free to rewrite to avoid using inject_all_ops_and_reduce_methods() for argmin/argmax at all. Method injection is pretty hacky, and generally not worthwhile, e.g., see this note about removing it. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Support multiple dimensions in DataArray.argmin() and DataArray.argmax() methods 594594646 | |
609468040 | https://github.com/pydata/xarray/pull/3936#issuecomment-609468040 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYwOTQ2ODA0MA== | shoyer 1217238 | 2020-04-05T19:19:41Z | 2020-04-05T19:19:41Z | MEMBER | This looks really comprehensive, thank you! Before doing a really careful review here, I'd like to try to work out the full API design we want. I'll write out some of my thoughts here, but your thoughts would also be very welcome! Here's my summary of the current situation:
1. We have two pairs of methods, This PR implements the multidimensional equivalent of My first concern is about the name: it isn't obvious to me whether Another option would be to overload I think I like this last option best but I would be curious what others think! @pydata/xarray any thoughts on this? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Support multiple dimensions in DataArray.argmin() and DataArray.argmax() methods 594594646 |
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