issue_comments
46 rows where author_association = "CONTRIBUTOR" and user = 3958036 sorted by updated_at descending
This data as json, CSV (advanced)
Suggested facets: issue_url, reactions, created_at (date), updated_at (date)
user 1
- johnomotani · 46 ✖
id | html_url | issue_url | node_id | user | created_at | updated_at ▲ | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
1245036513 | https://github.com/pydata/xarray/issues/4417#issuecomment-1245036513 | https://api.github.com/repos/pydata/xarray/issues/4417 | IC_kwDOAMm_X85KNb_h | johnomotani 3958036 | 2022-09-13T07:54:32Z | 2022-09-13T07:54:32Z | CONTRIBUTOR | Thanks @benbovy - with explicit methods now to produce the result with or without index, I think we can close this now :smiley: |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Inconsistency in whether index is created with new dimension coordinate? 698577111 | |
1000305491 | https://github.com/pydata/xarray/issues/4456#issuecomment-1000305491 | https://api.github.com/repos/pydata/xarray/issues/4456 | IC_kwDOAMm_X847n3NT | johnomotani 3958036 | 2021-12-23T13:28:20Z | 2021-12-23T13:28:20Z | CONTRIBUTOR | I didn't find any solution in xarray - I think I ended up just dropping the conflicting variable... |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
workaround for file with variable and dimension having same name 708337538 | |
830637894 | https://github.com/pydata/xarray/pull/5153#issuecomment-830637894 | https://api.github.com/repos/pydata/xarray/issues/5153 | MDEyOklzc3VlQ29tbWVudDgzMDYzNzg5NA== | johnomotani 3958036 | 2021-05-01T14:01:36Z | 2021-05-01T14:01:36Z | CONTRIBUTOR | The vote seems to be for a separate |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
cumulative_integrate() method 857378504 | |
829239055 | https://github.com/pydata/xarray/pull/5101#issuecomment-829239055 | https://api.github.com/repos/pydata/xarray/issues/5101 | MDEyOklzc3VlQ29tbWVudDgyOTIzOTA1NQ== | johnomotani 3958036 | 2021-04-29T13:29:55Z | 2021-04-29T14:43:04Z | CONTRIBUTOR |
It seemed odd to add a |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Surface plots 847489988 | |
828034654 | https://github.com/pydata/xarray/pull/5101#issuecomment-828034654 | https://api.github.com/repos/pydata/xarray/issues/5101 | MDEyOklzc3VlQ29tbWVudDgyODAzNDY1NA== | johnomotani 3958036 | 2021-04-27T23:51:19Z | 2021-04-27T23:51:19Z | CONTRIBUTOR | Thanks @mathause - |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Surface plots 847489988 | |
823167082 | https://github.com/pydata/xarray/pull/5099#issuecomment-823167082 | https://api.github.com/repos/pydata/xarray/issues/5099 | MDEyOklzc3VlQ29tbWVudDgyMzE2NzA4Mg== | johnomotani 3958036 | 2021-04-20T10:31:19Z | 2021-04-20T10:31:19Z | CONTRIBUTOR | Thanks @mathause! I've used your suggestion to turn my MVCE into a test which fails on master, but passes with the fix in this PR. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Use broadcast_like for 2d plot coordinates 847199398 | |
819120258 | https://github.com/pydata/xarray/pull/5153#issuecomment-819120258 | https://api.github.com/repos/pydata/xarray/issues/5153 | MDEyOklzc3VlQ29tbWVudDgxOTEyMDI1OA== | johnomotani 3958036 | 2021-04-13T23:43:12Z | 2021-04-13T23:43:12Z | CONTRIBUTOR |
I did a quick check and it seems like applying BTW the test failures are in |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
cumulative_integrate() method 857378504 | |
812012953 | https://github.com/pydata/xarray/pull/5101#issuecomment-812012953 | https://api.github.com/repos/pydata/xarray/issues/5101 | MDEyOklzc3VlQ29tbWVudDgxMjAxMjk1Mw== | johnomotani 3958036 | 2021-04-01T16:10:01Z | 2021-04-01T16:10:01Z | CONTRIBUTOR | Tests pass! Think this is ready for review now @pydata/xarray |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Surface plots 847489988 | |
811935990 | https://github.com/pydata/xarray/pull/5101#issuecomment-811935990 | https://api.github.com/repos/pydata/xarray/issues/5101 | MDEyOklzc3VlQ29tbWVudDgxMTkzNTk5MA== | johnomotani 3958036 | 2021-04-01T14:11:35Z | 2021-04-01T14:11:35Z | CONTRIBUTOR | Seems like the way I've implemented surface plots requires a slightly newer version than the oldest supported matplotlib to use the |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Surface plots 847489988 | |
811519561 | https://github.com/pydata/xarray/pull/5101#issuecomment-811519561 | https://api.github.com/repos/pydata/xarray/issues/5101 | MDEyOklzc3VlQ29tbWVudDgxMTUxOTU2MQ== | johnomotani 3958036 | 2021-03-31T22:59:59Z | 2021-03-31T22:59:59Z | CONTRIBUTOR | Note, this PR includes the bugfix in #5099, because it needed to build on that change. #5099 is smaller, so I guess it will naturally be merged first. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Surface plots 847489988 | |
811261936 | https://github.com/pydata/xarray/issues/5084#issuecomment-811261936 | https://api.github.com/repos/pydata/xarray/issues/5084 | MDEyOklzc3VlQ29tbWVudDgxMTI2MTkzNg== | johnomotani 3958036 | 2021-03-31T17:12:23Z | 2021-03-31T17:12:23Z | CONTRIBUTOR | I started working on this, and it seems to be almost trivial, except that |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
plot_surface() wrapper 842583817 | |
809264761 | https://github.com/pydata/xarray/pull/5003#issuecomment-809264761 | https://api.github.com/repos/pydata/xarray/issues/5003 | MDEyOklzc3VlQ29tbWVudDgwOTI2NDc2MQ== | johnomotani 3958036 | 2021-03-29T10:22:57Z | 2021-03-29T10:22:57Z | CONTRIBUTOR | Thanks @mathause - I've merged |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Add Dataset.plot.streamplot() method 823290488 | |
735419937 | https://github.com/pydata/xarray/issues/3002#issuecomment-735419937 | https://api.github.com/repos/pydata/xarray/issues/3002 | MDEyOklzc3VlQ29tbWVudDczNTQxOTkzNw== | johnomotani 3958036 | 2020-11-29T16:28:32Z | 2020-11-29T16:28:32Z | CONTRIBUTOR | I had the same problem. I found that
I guess the problem is that 'gouraud' needs the coordinate positions at the grid points, and I assume |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
plot.pcolormesh fails with shading='gouraud' 453126577 | |
734487380 | https://github.com/pydata/xarray/issues/2779#issuecomment-734487380 | https://api.github.com/repos/pydata/xarray/issues/2779 | MDEyOklzc3VlQ29tbWVudDczNDQ4NzM4MA== | johnomotani 3958036 | 2020-11-26T21:44:09Z | 2020-11-26T21:44:09Z | CONTRIBUTOR | :+1: I ran into this issue today, and found the error message strange and confusing. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Bring xr.align objects specification in line with other top-level functions? 412080974 | |
691109135 | https://github.com/pydata/xarray/issues/4417#issuecomment-691109135 | https://api.github.com/repos/pydata/xarray/issues/4417 | MDEyOklzc3VlQ29tbWVudDY5MTEwOTEzNQ== | johnomotani 3958036 | 2020-09-11T13:54:01Z | 2020-09-11T13:54:01Z | CONTRIBUTOR |
:+1: On a quick glance it looks to me like #4108 would fix my issue here. |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Inconsistency in whether index is created with new dimension coordinate? 698577111 | |
691045790 | https://github.com/pydata/xarray/issues/4417#issuecomment-691045790 | https://api.github.com/repos/pydata/xarray/issues/4417 | MDEyOklzc3VlQ29tbWVudDY5MTA0NTc5MA== | johnomotani 3958036 | 2020-09-11T11:45:40Z | 2020-09-11T11:45:40Z | CONTRIBUTOR | I haven't managed to get ds = xr.Dataset() ds['a'] = ('x', np.linspace(0,1)) ds['b'] = ('x', np.linspace(3,4)) ds = ds.rename(b='x') ds = ds.set_coords('x') ds = ds.set_index(x='x') print(ds)
print('indexes', ds.indexes)
One suggested solution in #2461 was to use ds = xr.Dataset() ds['a'] = ('x', np.linspace(0,1)) ds['b'] = ('x', np.linspace(3,4)) ds = ds.swap_dims({'x': 'b'}) ds = ds.rename(b='x') print(ds)
print('indexes', ds.indexes)
I found the behaviour confusing because I wasn't aware of the index variables at all... I create a coordinate with I may well be missing other context that makes this an undesirable thing to do, but for my use-case at least, I think it would make more sense if |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Inconsistency in whether index is created with new dimension coordinate? 698577111 | |
690538286 | https://github.com/pydata/xarray/issues/4415#issuecomment-690538286 | https://api.github.com/repos/pydata/xarray/issues/4415 | MDEyOklzc3VlQ29tbWVudDY5MDUzODI4Ng== | johnomotani 3958036 | 2020-09-10T17:29:28Z | 2020-09-10T17:29:28Z | CONTRIBUTOR | Apparently it's possible to work around this by using ds = xr.Dataset() ds["a"] = xr.DataArray(np.linspace(0., 1.), dims="x") ds["x"] = xr.DataArray(np.linspace(0., 2., len(ds["x"])), dims="x") ds["x"].attrs["foo"] = "bar" print(ds["x"]) ds["b"] = xr.DataArray(np.linspace(0., 1.), dims="x")ds = ds.merge(xr.DataArray(np.linspace(0., 1.), dims="x", name="b")) print(ds["x"]) ``` |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Adding new DataArray to a Dataset removes attrs of existing coord 698263021 | |
690535154 | https://github.com/pydata/xarray/issues/4415#issuecomment-690535154 | https://api.github.com/repos/pydata/xarray/issues/4415 | MDEyOklzc3VlQ29tbWVudDY5MDUzNTE1NA== | johnomotani 3958036 | 2020-09-10T17:24:06Z | 2020-09-10T17:24:06Z | CONTRIBUTOR | Unfortunately I don't have time at the moment to dig into why this is happening, but it was confusing because I found a bug where attrs had disappeared, and eventually chased it down to where some new variables were added to my Dataset, which I would not expect to change existing coordinates! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Adding new DataArray to a Dataset removes attrs of existing coord 698263021 | |
651340318 | https://github.com/pydata/xarray/pull/3936#issuecomment-651340318 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDY1MTM0MDMxOA== | johnomotani 3958036 | 2020-06-29T20:22:49Z | 2020-06-29T20:22:49Z | CONTRIBUTOR | Thanks @dcherian :smile: You're very welcome! |
{ "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 | |
650303121 | https://github.com/pydata/xarray/pull/3936#issuecomment-650303121 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDY1MDMwMzEyMQ== | johnomotani 3958036 | 2020-06-26T17:29:15Z | 2020-06-26T17:29:15Z | CONTRIBUTOR | Thanks @keewis! I think we've addressed all the review comments now. |
{ "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 | |
650300879 | https://github.com/pydata/xarray/pull/3936#issuecomment-650300879 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDY1MDMwMDg3OQ== | johnomotani 3958036 | 2020-06-26T17:24:55Z | 2020-06-26T17:24:55Z | CONTRIBUTOR |
Some missing values should be OK. In your example though for example
I guess we could add some special handling for this (not sure what though, because we can't set a variable with type |
{ "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 | |
650272329 | https://github.com/pydata/xarray/pull/3936#issuecomment-650272329 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDY1MDI3MjMyOQ== | johnomotani 3958036 | 2020-06-26T16:29:35Z | 2020-06-26T16:29:35Z | CONTRIBUTOR |
I think this is a bug. |
{ "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 | |
649059087 | https://github.com/pydata/xarray/pull/3936#issuecomment-649059087 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDY0OTA1OTA4Nw== | johnomotani 3958036 | 2020-06-24T20:39:31Z | 2020-06-24T20:39:31Z | CONTRIBUTOR | Merge conflicts fixed, this PR should be ready to review/merge. |
{ "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 | |
621735196 | https://github.com/pydata/xarray/issues/4009#issuecomment-621735196 | https://api.github.com/repos/pydata/xarray/issues/4009 | MDEyOklzc3VlQ29tbWVudDYyMTczNTE5Ng== | johnomotani 3958036 | 2020-04-30T09:58:50Z | 2020-04-30T09:58:50Z | CONTRIBUTOR | Yes, should be simple to correct. I have a unit test now that reproduces the error. @pydata/xarray a question: for |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Incoherencies between docs in open_mfdataset and combine_by_coords and its behaviour. 607616849 | |
617375991 | https://github.com/pydata/xarray/pull/3936#issuecomment-617375991 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxNzM3NTk5MQ== | johnomotani 3958036 | 2020-04-21T19:46:09Z | 2020-04-21T19:46:09Z | CONTRIBUTOR |
|
{ "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 | |
616100298 | https://github.com/pydata/xarray/pull/3936#issuecomment-616100298 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxNjEwMDI5OA== | johnomotani 3958036 | 2020-04-19T10:42:06Z | 2020-04-19T10:42:06Z | CONTRIBUTOR | @pydata/xarray - I think this PR is ready to be merged, are there any changes I should make? Thanks! |
{ "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 | |
612143069 | https://github.com/pydata/xarray/pull/3936#issuecomment-612143069 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxMjE0MzA2OQ== | johnomotani 3958036 | 2020-04-10T17:51:13Z | 2020-04-10T17:51:13Z | CONTRIBUTOR | I eventually found that the cause of the errors I was getting was that the @shoyer I've kept the use of injected functions, because removing the injected I think this PR is ready for review now :smile: |
{ "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 | |
610418244 | https://github.com/pydata/xarray/issues/3949#issuecomment-610418244 | https://api.github.com/repos/pydata/xarray/issues/3949 | MDEyOklzc3VlQ29tbWVudDYxMDQxODI0NA== | johnomotani 3958036 | 2020-04-07T14:26:02Z | 2020-04-07T14:26:02Z | CONTRIBUTOR | I think |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Index 3D array with index of last axis stored in 2D array 595900209 | |
610414198 | https://github.com/pydata/xarray/issues/3948#issuecomment-610414198 | https://api.github.com/repos/pydata/xarray/issues/3948 | MDEyOklzc3VlQ29tbWVudDYxMDQxNDE5OA== | johnomotani 3958036 | 2020-04-07T14:18:36Z | 2020-04-07T14:18:36Z | CONTRIBUTOR | Sorry for the noise, but at least I will be glad of something to search for next time I forget how to do something like this! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Releasing memory? 595882590 | |
610413864 | https://github.com/pydata/xarray/issues/3948#issuecomment-610413864 | https://api.github.com/repos/pydata/xarray/issues/3948 | MDEyOklzc3VlQ29tbWVudDYxMDQxMzg2NA== | johnomotani 3958036 | 2020-04-07T14:18:01Z | 2020-04-07T14:18:01Z | CONTRIBUTOR | Thanks @lanougue, @dcherian I think I see the simple answer now: use a deepcopy first, to leave the Dataset as is, then da2 = ds["variable2"].copy(deep=True) ... do stuff with da2 ... del da2 ... etc. ``` |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Releasing memory? 595882590 | |
610404757 | https://github.com/pydata/xarray/issues/3948#issuecomment-610404757 | https://api.github.com/repos/pydata/xarray/issues/3948 | MDEyOklzc3VlQ29tbWVudDYxMDQwNDc1Nw== | johnomotani 3958036 | 2020-04-07T14:01:28Z | 2020-04-07T14:01:28Z | CONTRIBUTOR | Thanks @lanougue, but what if I might want to re-load |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Releasing memory? 595882590 | |
610403146 | https://github.com/pydata/xarray/issues/3948#issuecomment-610403146 | https://api.github.com/repos/pydata/xarray/issues/3948 | MDEyOklzc3VlQ29tbWVudDYxMDQwMzE0Ng== | johnomotani 3958036 | 2020-04-07T13:58:44Z | 2020-04-07T13:58:44Z | CONTRIBUTOR | OK, I think I've answered my own question. Looks like do_some_work does not call
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Releasing memory? 595882590 | |
610288219 | https://github.com/pydata/xarray/pull/3936#issuecomment-610288219 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxMDI4ODIxOQ== | johnomotani 3958036 | 2020-04-07T09:45:24Z | 2020-04-07T09:45:24Z | CONTRIBUTOR | These test failures seem to have uncovered a larger issue: overriding a method injected by |
{ "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 | |
610081717 | https://github.com/pydata/xarray/pull/3936#issuecomment-610081717 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxMDA4MTcxNw== | johnomotani 3958036 | 2020-04-06T23:06:11Z | 2020-04-06T23:06:11Z | CONTRIBUTOR | The rename to |
{ "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 | |
610079091 | https://github.com/pydata/xarray/pull/3936#issuecomment-610079091 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYxMDA3OTA5MQ== | johnomotani 3958036 | 2020-04-06T22:57:50Z | 2020-04-06T22:57:50Z | CONTRIBUTOR | I've updated so the new functionality is provided by |
{ "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 | |
609477080 | https://github.com/pydata/xarray/pull/3936#issuecomment-609477080 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYwOTQ3NzA4MA== | johnomotani 3958036 | 2020-04-05T20:26:12Z | 2020-04-05T20:50:03Z | CONTRIBUTOR | @shoyer I think your last option sounds good. Questions:
* What should |
{ "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 | |
609478091 | https://github.com/pydata/xarray/pull/3936#issuecomment-609478091 | https://api.github.com/repos/pydata/xarray/issues/3936 | MDEyOklzc3VlQ29tbWVudDYwOTQ3ODA5MQ== | johnomotani 3958036 | 2020-04-05T20:33:04Z | 2020-04-05T20:33:04Z | CONTRIBUTOR | Maybe worth noting, at the moment if you try to call
During handling of the above exception, another exception occurred: Traceback (most recent call last): File "<stdin>", line 1, in <module> File "/<...>/xarray/xarray/core/common.py", line 46, in wrapped_func return self.reduce(func, dim, axis, skipna=skipna, kwargs) File "/<...>/xarray/xarray/core/dataarray.py", line 2288, in reduce var = self.variable.reduce(func, dim, axis, keep_attrs, keepdims, kwargs) File "/<...>/xarray/xarray/core/variable.py", line 1579, in reduce data = func(input_data, axis=axis, kwargs) File "/<...>/xarray/xarray/core/duck_array_ops.py", line 304, in f return func(values, axis=axis, kwargs) File "/<...>/xarray/xarray/core/duck_array_ops.py", line 47, in f return wrapped(args, kwargs) File "<array_function internals>", line 6, in argmin File "/<...>/anaconda3/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 1267, in argmin return _wrapfunc(a, 'argmin', axis=axis, out=out) File "/<...>/anaconda3/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 70, in _wrapfunc return _wrapit(obj, method, args, kwds) File "/<...>/anaconda3/lib/python3.7/site-packages/numpy/core/fromnumeric.py", line 47, in _wrapit result = getattr(asarray(obj), method)(*args, kwds) TypeError: 'tuple' object cannot be interpreted as an integer ``` |
{ "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 | |
609084241 | https://github.com/pydata/xarray/pull/1469#issuecomment-609084241 | https://api.github.com/repos/pydata/xarray/issues/1469 | MDEyOklzc3VlQ29tbWVudDYwOTA4NDI0MQ== | johnomotani 3958036 | 2020-04-04T20:23:17Z | 2020-04-04T20:23:17Z | CONTRIBUTOR | Any plans to finish/merge this PR? |
{ "total_count": 1, "+1": 1, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Argmin indexes 239918314 | |
608324523 | https://github.com/pydata/xarray/pull/3923#issuecomment-608324523 | https://api.github.com/repos/pydata/xarray/issues/3923 | MDEyOklzc3VlQ29tbWVudDYwODMyNDUyMw== | johnomotani 3958036 | 2020-04-03T09:10:41Z | 2020-04-03T09:10:41Z | CONTRIBUTOR | The Linux-py38-upstream-dev test is failing in cftime tests at various places similar to this ``` with create_tmp_file() as tmp_file: original.to_netcdf(tmp_file) with pytest.warns(None) as record: with open_dataset(tmp_file, use_cftime=False) as ds: assert_identical(expected_x, ds.x) assert_identical(expected_time, ds.time)
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Add missing_dims argument allowing isel() to ignore missing dimensions 591471233 | |
603509086 | https://github.com/pydata/xarray/pull/3887#issuecomment-603509086 | https://api.github.com/repos/pydata/xarray/issues/3887 | MDEyOklzc3VlQ29tbWVudDYwMzUwOTA4Ng== | johnomotani 3958036 | 2020-03-24T21:16:08Z | 2020-03-24T21:16:08Z | CONTRIBUTOR | Doesn't look like it, but it seems like a thing that could be useful, e.g. for handling combining |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Rename ordered_dict_intersection -> compat_dict_intersection 587280307 | |
603134393 | https://github.com/pydata/xarray/pull/3877#issuecomment-603134393 | https://api.github.com/repos/pydata/xarray/issues/3877 | MDEyOklzc3VlQ29tbWVudDYwMzEzNDM5Mw== | johnomotani 3958036 | 2020-03-24T09:42:11Z | 2020-03-24T09:42:11Z | CONTRIBUTOR | For specifying which object, one possibility would be to pass an |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Control attrs of result in `merge()`, `concat()`, `combine_by_coords()` and `combine_nested()` 585868107 | |
602806659 | https://github.com/pydata/xarray/pull/3877#issuecomment-602806659 | https://api.github.com/repos/pydata/xarray/issues/3877 | MDEyOklzc3VlQ29tbWVudDYwMjgwNjY1OQ== | johnomotani 3958036 | 2020-03-23T19:23:47Z | 2020-03-23T19:23:47Z | CONTRIBUTOR |
That would actually be nice to have in |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Control attrs of result in `merge()`, `concat()`, `combine_by_coords()` and `combine_nested()` 585868107 | |
582538190 | https://github.com/pydata/xarray/pull/3601#issuecomment-582538190 | https://api.github.com/repos/pydata/xarray/issues/3601 | MDEyOklzc3VlQ29tbWVudDU4MjUzODE5MA== | johnomotani 3958036 | 2020-02-05T18:09:24Z | 2020-02-05T18:09:24Z | CONTRIBUTOR | I've added an extra test that checks the right thing happens when we do not change the bad, under or over colors. That test needs to set I think this is ready now. The unit test failures on my local computer appear to be unrelated to this PR (see earlier references to #3673 by @keewis and #3747 by @mathause). |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Fix contourf set under 533996523 | |
577418853 | https://github.com/pydata/xarray/pull/3601#issuecomment-577418853 | https://api.github.com/repos/pydata/xarray/issues/3601 | MDEyOklzc3VlQ29tbWVudDU3NzQxODg1Mw== | johnomotani 3958036 | 2020-01-22T22:37:01Z | 2020-01-22T22:37:01Z | CONTRIBUTOR | Not sure why some tests are failing... The ones I can see are to do with |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
Fix contourf set under 533996523 | |
562578478 | https://github.com/pydata/xarray/issues/3590#issuecomment-562578478 | https://api.github.com/repos/pydata/xarray/issues/3590 | MDEyOklzc3VlQ29tbWVudDU2MjU3ODQ3OA== | johnomotani 3958036 | 2019-12-06T13:49:36Z | 2019-12-06T13:49:36Z | CONTRIBUTOR | Looks like that was it, thanks @dcherian! |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
cmap.set_under() does not work as expected 532165408 | |
561358453 | https://github.com/pydata/xarray/issues/3590#issuecomment-561358453 | https://api.github.com/repos/pydata/xarray/issues/3590 | MDEyOklzc3VlQ29tbWVudDU2MTM1ODQ1Mw== | johnomotani 3958036 | 2019-12-03T21:10:58Z | 2019-12-03T21:10:58Z | CONTRIBUTOR | Might be somethnig to do with https://github.com/pydata/xarray/blob/ed05f9862622b00f40f7b9b99ccdb0ab3766ff0f/xarray/plot/dataset_plot.py#L137 ? The docstring for
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
cmap.set_under() does not work as expected 532165408 |
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]);
issue 20