html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue https://github.com/pydata/xarray/pull/7862#issuecomment-1578777785,https://api.github.com/repos/pydata/xarray/issues/7862,1578777785,IC_kwDOAMm_X85eGjy5,43316012,2023-06-06T13:31:34Z,2023-06-06T13:31:34Z,COLLABORATOR,"If you want you can leave a comment But the mypy CI should fail on unused ignores, so we will notice :)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1720045908 https://github.com/pydata/xarray/pull/7891#issuecomment-1577364529,https://api.github.com/repos/pydata/xarray/issues/7891,1577364529,IC_kwDOAMm_X85eBKwx,43316012,2023-06-05T19:35:33Z,2023-06-05T19:35:33Z,COLLABORATOR,"> You could use `DataArray.round` to round to significant decimals. Better to use a tolerance in the assterion testing.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1740268634 https://github.com/pydata/xarray/pull/7862#issuecomment-1574278204,https://api.github.com/repos/pydata/xarray/issues/7862,1574278204,IC_kwDOAMm_X85d1ZQ8,43316012,2023-06-02T20:27:46Z,2023-06-02T20:28:29Z,COLLABORATOR,"This seems to be a numpy issue, mypy thinks that you cannot call np.dtype like you do. Might be worth an issue over at numpy with the example from the test. For now we can simply ignore this error.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1720045908 https://github.com/pydata/xarray/pull/7880#issuecomment-1572359754,https://api.github.com/repos/pydata/xarray/issues/7880,1572359754,IC_kwDOAMm_X85duE5K,43316012,2023-06-01T16:23:45Z,2023-06-01T16:23:45Z,COLLABORATOR,"Maybe you can add a test that creates a cachingFilemanager object, then deletes it, then run `gc.collect()` and check somehow if it works? But no idea how pytest interferes with this or how you can ensure that there are no references to the module anymore?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1730664352 https://github.com/pydata/xarray/pull/7670#issuecomment-1572259965,https://api.github.com/repos/pydata/xarray/issues/7670,1572259965,IC_kwDOAMm_X85dtsh9,43316012,2023-06-01T15:22:33Z,2023-06-01T15:22:33Z,COLLABORATOR,"> The cfgrib notebook in the documentation is broken. I guess it's related to this PR. > See: https://docs.xarray.dev/en/stable/examples/ERA5-GRIB-example.html Same problem with https://github.com/pydata/xarray/issues/7841","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1639732867 https://github.com/pydata/xarray/pull/7880#issuecomment-1567450094,https://api.github.com/repos/pydata/xarray/issues/7880,1567450094,IC_kwDOAMm_X85dbWPu,43316012,2023-05-29T19:28:21Z,2023-05-29T19:28:21Z,COLLABORATOR,"> I think this is intended (though certainly not very easy to get right): see the second part of the [warning in the `__del__` documentation](https://docs.python.org/3/reference/datamodel.html#object.__del__). You are right, that warning is exactly what is causing the issues.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1730664352 https://github.com/pydata/xarray/pull/7880#issuecomment-1567439206,https://api.github.com/repos/pydata/xarray/issues/7880,1567439206,IC_kwDOAMm_X85dbTlm,43316012,2023-05-29T19:09:41Z,2023-05-29T19:09:41Z,COLLABORATOR,"That's quite a weird bug. I would have thought that the global (or module level here) variable/function aquire should have at least one reference until after the deletion of the object. Is that a bug in pythons garbage collection? Or does the garbage collection already start when calling `del` and does not wait for the completion of the `__del__` method?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1730664352 https://github.com/pydata/xarray/issues/5644#issuecomment-1561096393,https://api.github.com/repos/pydata/xarray/issues/5644,1561096393,IC_kwDOAMm_X85dDHDJ,43316012,2023-05-24T12:56:48Z,2023-05-24T12:56:48Z,COLLABORATOR,"You can always reach out to the creator of the original PR and comment in the PR. But it looks like this particular PR was reaching a dead end and should be completely rewritten. But anyway the reviewers left helpful remarks on how to proceed. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,955043280 https://github.com/pydata/xarray/issues/7851#issuecomment-1554621577,https://api.github.com/repos/pydata/xarray/issues/7851,1554621577,IC_kwDOAMm_X85cqaSJ,43316012,2023-05-19T14:01:42Z,2023-05-19T14:01:42Z,COLLABORATOR,"I think this goes a bit against the rest of xarray which discourages in place operations. But on the other hand, both deleting a variable and extracting a DataArray exist on their own, so why not combine it.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1717209758 https://github.com/pydata/xarray/pull/7840#issuecomment-1547169473,https://api.github.com/repos/pydata/xarray/issues/7840,1547169473,IC_kwDOAMm_X85cN-7B,43316012,2023-05-15T04:15:53Z,2023-05-15T04:15:53Z,COLLABORATOR,"Well that's the point, ""x"" is a dimension as well as a coordinate. And since dimensions are names you cannot assign anything to them. If you want you can change coordinate by dimension coordinate.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1707774178 https://github.com/pydata/xarray/pull/7840#issuecomment-1546939363,https://api.github.com/repos/pydata/xarray/issues/7840,1546939363,IC_kwDOAMm_X85cNGvj,43316012,2023-05-14T16:29:13Z,2023-05-14T16:29:13Z,COLLABORATOR,"I agree that there is a lot of terminology that is somewhat similar and/or overlapping. In math you have the basis of a vector space which when plotting in a coordinate system you sometimes call coordinates or axes. Also you often use the phrase ""the x-coordinate of point p is ..."", So I understand why naming an axis ""coordinate"" might sound reasonable. However in xarray (or numerics in general) you often deal with data that is not aligned to any axis in a given coordinate system. Consider the following example: you have a time series of points that might be scattered randomly in your coordinate system. Now you can assign to each point a x and y coordinate. In xarray you would call the dimension of the data e.g. ""time"". You will need two additional time series of data points for the x and y coordinates. In xarray this would be something like that: ```python da = xr.DataArray( [5, 6, 7], dims=""time"", coords={ ""x"": (""time"", [1, 2, 3]), ""y"": (""time"", [9, 8, 7]), ""time"": [0, 1, 2] } ``` You can see that in this example we cannot easily define a x-y coordinate system since the data points are not on a grid. But still we can assign each data point a x and y coordinate. Now finding a naming convention that fits both, random data points and lattices that are aligned with the coordinate system is not trivial. That's why we choose to go with the following: - **dimension**: name of an axis of a nd-array, this might be a ""real"" axis that has any real world equivalent or something as trivial as ""order in which the data has been aquisited"". In the example this was ""time"". - **coordinate**: auxiliary data that can be used to identify the data values. In the example this was ""x"" and ""y"". (Basically it is a short name for ""coordinate variable"") - **dimension coordinate**: a coordinate that assigns values to the dimension directly if possible. This is the case when the name of a coordinate is the same as a dimension. In the example this was ""time"" with the timestamps 0, 1 and 2 (probably this should be real timestamps). Feel free to propose some changes to the documentation such that newcomers will find it easier to understand the terminology.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1707774178 https://github.com/pydata/xarray/pull/7840#issuecomment-1546029852,https://api.github.com/repos/pydata/xarray/issues/7840,1546029852,IC_kwDOAMm_X85cJosc,43316012,2023-05-12T16:57:53Z,2023-05-12T16:57:53Z,COLLABORATOR,Yes exactly. A dimension is only a name for an axis.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1707774178 https://github.com/pydata/xarray/pull/7840#issuecomment-1545930547,https://api.github.com/repos/pydata/xarray/issues/7840,1545930547,IC_kwDOAMm_X85cJQcz,43316012,2023-05-12T15:32:29Z,2023-05-12T15:32:29Z,COLLABORATOR,"This is a bit misleading. I think the term coordinate is correct, but you can be more precise and use dimension-coordinate, see: https://docs.xarray.dev/en/stable/user-guide/data-structures.html#coordinates Maybe we should start calling them index as well? But I think the concept of dimension-coordinates (when the coordinate has the same name as the dimension) is still relevant.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1707774178 https://github.com/pydata/xarray/pull/7676#issuecomment-1541835110,https://api.github.com/repos/pydata/xarray/issues/7676,1541835110,IC_kwDOAMm_X85b5olm,43316012,2023-05-10T09:56:52Z,2023-05-10T09:56:52Z,COLLABORATOR,@Mehak-4545 are you still working on this? You need to change the style of your docstrings and examples. See other docstrings formatting for inspiration.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1640453837 https://github.com/pydata/xarray/issues/7831#issuecomment-1540811425,https://api.github.com/repos/pydata/xarray/issues/7831,1540811425,IC_kwDOAMm_X85b1uqh,43316012,2023-05-09T19:57:48Z,2023-05-09T19:57:48Z,COLLABORATOR,"I don't know how we would implement that, it's probably not a good idea to special case all external backends within xarray. Either the package is installed and then it works or it is not installed and then we don't know which backend/package is missing.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1702025553 https://github.com/pydata/xarray/issues/7816#issuecomment-1535358946,https://api.github.com/repos/pydata/xarray/issues/7816,1535358946,IC_kwDOAMm_X85bg7fi,43316012,2023-05-04T20:18:51Z,2023-05-04T20:24:26Z,COLLABORATOR,"I am not sure what exactly is not working. The BACKEND_ENTRYPOINTS is not a public API and is only used for xarrays internal backends. In the documentation that you have linked it is explained on how to register your project, which is purely in the setup.cfg/pyproject.toml, no need to manipulate xarray internal objects.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1695809136 https://github.com/pydata/xarray/issues/7758#issuecomment-1509899180,https://api.github.com/repos/pydata/xarray/issues/7758,1509899180,IC_kwDOAMm_X85Z_zus,43316012,2023-04-15T17:19:38Z,2023-04-15T17:19:38Z,COLLABORATOR,"Maybe I was not clear enough, sry for that. With remote file system I meant that you mount e.g. an NFS drive. It seems that you are using a https protocol to transfer the data.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1668898601 https://github.com/pydata/xarray/issues/7754#issuecomment-1509585164,https://api.github.com/repos/pydata/xarray/issues/7754,1509585164,IC_kwDOAMm_X85Z-nEM,43316012,2023-04-15T06:33:50Z,2023-04-15T06:33:50Z,COLLABORATOR,What happens when you specify the netcdf4 engine directly? (Assuming it is a nc4 and not nc3 file).,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1667556281 https://github.com/pydata/xarray/issues/7758#issuecomment-1509537842,https://api.github.com/repos/pydata/xarray/issues/7758,1509537842,IC_kwDOAMm_X85Z-bgy,43316012,2023-04-15T05:35:50Z,2023-04-15T05:35:50Z,COLLABORATOR,"Which backend are you using? Is it a ""real"" server or simply a remote filesystem?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1668898601 https://github.com/pydata/xarray/issues/7754#issuecomment-1508940623,https://api.github.com/repos/pydata/xarray/issues/7754,1508940623,IC_kwDOAMm_X85Z8JtP,43316012,2023-04-14T16:43:59Z,2023-04-14T16:43:59Z,COLLABORATOR,"The error message tells that the backends are installed and can be found by xarray. What exactly did you try to open? A local file? If yes, can you provide the name of this file?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1667556281 https://github.com/pydata/xarray/issues/7748#issuecomment-1506550069,https://api.github.com/repos/pydata/xarray/issues/7748,1506550069,IC_kwDOAMm_X85ZzCE1,43316012,2023-04-13T08:20:24Z,2023-04-13T08:20:24Z,COLLABORATOR,See also somewhat related discussion here https://github.com/pydata/xarray/issues/6749,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1664193419 https://github.com/pydata/xarray/issues/7748#issuecomment-1506544631,https://api.github.com/repos/pydata/xarray/issues/7748,1506544631,IC_kwDOAMm_X85ZzAv3,43316012,2023-04-13T08:16:11Z,2023-04-13T08:16:11Z,COLLABORATOR,"Not sure what the correct way would be. If you consider that `d f(x) / d y = 0`, maybe all zeros could be considered correct as well. The same result can be obtained if one considers that the array might be broadcast to the y-dim. Also we have to check that happens in Datasets if one of the variables does not contain this dimension, do we raise an error as well?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1664193419 https://github.com/pydata/xarray/pull/7746#issuecomment-1502954308,https://api.github.com/repos/pydata/xarray/issues/7746,1502954308,IC_kwDOAMm_X85ZlUNE,43316012,2023-04-11T09:06:09Z,2023-04-11T09:06:09Z,COLLABORATOR,"By the way: even for such a small contribution you can add an entry into the whats-new changelog. If you wish to do so, you can open another PR and I will quickly merge it (I was too fast merging this one...).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1661789758 https://github.com/pydata/xarray/pull/7746#issuecomment-1502785268,https://api.github.com/repos/pydata/xarray/issues/7746,1502785268,IC_kwDOAMm_X85Zkq70,43316012,2023-04-11T07:04:01Z,2023-04-11T07:04:01Z,COLLABORATOR,Thanks for this nice little PR!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1661789758 https://github.com/pydata/xarray/issues/7742#issuecomment-1502746054,https://api.github.com/repos/pydata/xarray/issues/7742,1502746054,IC_kwDOAMm_X85ZkhXG,43316012,2023-04-11T06:20:08Z,2023-04-11T06:20:08Z,COLLABORATOR,"@ChristmasZCY does this answer your question? If yes, feel free to close this issue .","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1659786592 https://github.com/pydata/xarray/pull/7724#issuecomment-1501150705,https://api.github.com/repos/pydata/xarray/issues/7724,1501150705,IC_kwDOAMm_X85Zeb3x,43316012,2023-04-09T15:09:37Z,2023-04-09T15:09:37Z,COLLABORATOR,"Locally it always works, it segfaults in CI, which makes it impossible to debug.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1655782486 https://github.com/pydata/xarray/issues/620#issuecomment-1500937954,https://api.github.com/repos/pydata/xarray/issues/620,1500937954,IC_kwDOAMm_X85Zdn7i,43316012,2023-04-08T17:20:35Z,2023-04-08T17:20:50Z,COLLABORATOR,"Reviving this ancient issue here. We could add a deprecation cycle and issue a warning for dimensions with size 1.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,110820316 https://github.com/pydata/xarray/pull/7735#issuecomment-1500935213,https://api.github.com/repos/pydata/xarray/issues/7735,1500935213,IC_kwDOAMm_X85ZdnQt,43316012,2023-04-08T17:05:53Z,2023-04-08T17:05:53Z,COLLABORATOR,From 2.07±0.04ms to 369±10μs seems good :),"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1658287592 https://github.com/pydata/xarray/pull/7724#issuecomment-1500907146,https://api.github.com/repos/pydata/xarray/issues/7724,1500907146,IC_kwDOAMm_X85ZdgaK,43316012,2023-04-08T14:53:42Z,2023-04-08T14:53:42Z,COLLABORATOR,"> @Illviljan, @headtr1ck, I just noticed that the CI version of `mypy` is pinned to `<0.990`, but the `pre-commit` hook is at `1.1.1`. Is that intentional / known? Running the hook also exposes quite a few typing errors. I think this is because of https://github.com/pydata/xarray/issues/7270 The pre-commit hook of mypy should be disabled anyway because it takes too long to run (should be actually much faster since mypy >=1). I think we could check if the newest mypy still segfaults or not... ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1655782486 https://github.com/pydata/xarray/pull/7732#issuecomment-1500324581,https://api.github.com/repos/pydata/xarray/issues/7732,1500324581,IC_kwDOAMm_X85ZbSLl,43316012,2023-04-07T14:09:07Z,2023-04-07T14:11:35Z,COLLABORATOR,"Actually on top of examples maybe some simple text depictions could be useful? Something like this for aligning: Values: [ A A A ] + [ B B ] -> [ A B A B A ] Index:. [ 1 3 5 ] + [ 2 4 ] -> [ 1 2 3 4 5 ] But just throwing ideas around Edit: that's actually not what aligning does, haha. But you get the idea.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1657534038 https://github.com/pydata/xarray/issues/3921#issuecomment-1498503222,https://api.github.com/repos/pydata/xarray/issues/3921,1498503222,IC_kwDOAMm_X85ZUVg2,43316012,2023-04-06T05:14:46Z,2023-04-06T05:14:46Z,COLLABORATOR,"I think the last remaining item has been fixed by https://github.com/pydata/xarray/pull/7586, but not completely sure. Then we could close this issue :)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,590630281 https://github.com/pydata/xarray/pull/7635#issuecomment-1495843182,https://api.github.com/repos/pydata/xarray/issues/7635,1495843182,IC_kwDOAMm_X85ZKMFu,43316012,2023-04-04T11:54:17Z,2023-04-04T11:54:17Z,COLLABORATOR,@dsgreen2 if you apply the last review comments this is good to go.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1627298527 https://github.com/pydata/xarray/pull/7698#issuecomment-1495840833,https://api.github.com/repos/pydata/xarray/issues/7698,1495840833,IC_kwDOAMm_X85ZKLhB,43316012,2023-04-04T11:52:08Z,2023-04-04T11:52:08Z,COLLABORATOR,"I think this logic is done one level above in the call stack. But yes, maybe a different name for the argument would be better.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1646350377 https://github.com/pydata/xarray/issues/7711#issuecomment-1494151318,https://api.github.com/repos/pydata/xarray/issues/7711,1494151318,IC_kwDOAMm_X85ZDvCW,43316012,2023-04-03T11:29:01Z,2023-04-03T11:29:01Z,COLLABORATOR,"Not sure what you are proposing here. We have extensive documentation on data indexing and selection here https://docs.xarray.dev/en/stable/user-guide/indexing.html If this is not what you mean, please update your description and title to something more meaningful.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1651829181 https://github.com/pydata/xarray/pull/7706#issuecomment-1493396616,https://api.github.com/repos/pydata/xarray/issues/7706,1493396616,IC_kwDOAMm_X85ZA2yI,43316012,2023-04-02T17:25:02Z,2023-04-02T17:25:02Z,COLLABORATOR,"Oh, you can add an entry in whats-new if you want!","{""total_count"": 1, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 1, ""rocket"": 0, ""eyes"": 0}",,1650309361 https://github.com/pydata/xarray/issues/7703#issuecomment-1492167253,https://api.github.com/repos/pydata/xarray/issues/7703,1492167253,IC_kwDOAMm_X85Y8KpV,43316012,2023-03-31T15:45:01Z,2023-03-31T15:45:01Z,COLLABORATOR,Is working again :),"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1648748263 https://github.com/pydata/xarray/pull/7694#issuecomment-1491538813,https://api.github.com/repos/pydata/xarray/issues/7694,1491538813,IC_kwDOAMm_X85Y5xN9,43316012,2023-03-31T08:41:37Z,2023-03-31T08:41:37Z,COLLABORATOR,"> @headtr1ck Read the docs build is failing. Is it due to issue #7703. Or the fault is on my side. Yes, that is this issue.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1644759739 https://github.com/pydata/xarray/pull/7669#issuecomment-1491367287,https://api.github.com/repos/pydata/xarray/issues/7669,1491367287,IC_kwDOAMm_X85Y5HV3,43316012,2023-03-31T06:23:18Z,2023-03-31T06:23:38Z,COLLABORATOR," > The failing docs CI is caused by `pydata-sphinx-theme=0.13.2` being incompatible with `sphinx-book-theme=1.0.0`. I think we can pin `pydata-sphinx-theme!=0.13.2` until they fixed that (though maybe that should be done in a new PR). Just saw this comment. I have opened #7703 for this","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1639361476 https://github.com/pydata/xarray/pull/7623#issuecomment-1491365165,https://api.github.com/repos/pydata/xarray/issues/7623,1491365165,IC_kwDOAMm_X85Y5G0t,43316012,2023-03-31T06:22:28Z,2023-03-31T06:22:28Z,COLLABORATOR,"> I am not sure if pinning works as pydata-sphinx-theme had some private function which sphinx-book-theme was using but now it cannot. > Will probably have to wait for a new release of sphinx-book-theme. See #7703 ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1623776623 https://github.com/pydata/xarray/pull/7523#issuecomment-1491364430,https://api.github.com/repos/pydata/xarray/issues/7523,1491364430,IC_kwDOAMm_X85Y5GpO,43316012,2023-03-31T06:22:09Z,2023-03-31T06:22:09Z,COLLABORATOR,"Thanks for the fixes. I have opened an issue about the readthedocs failing #7703, it's not an issue of this PR.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1581313830 https://github.com/pydata/xarray/issues/7701#issuecomment-1490906724,https://api.github.com/repos/pydata/xarray/issues/7701,1490906724,IC_kwDOAMm_X85Y3W5k,43316012,2023-03-30T20:26:08Z,2023-03-30T20:26:08Z,COLLABORATOR,How did you test this? I just tried it with 2023.03.0 and 2022.06.0 with all other packages the same and I get the same results (the one you say is new).,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1647883619 https://github.com/pydata/xarray/issues/7697#issuecomment-1489146483,https://api.github.com/repos/pydata/xarray/issues/7697,1489146483,IC_kwDOAMm_X85YwpJz,43316012,2023-03-29T19:02:39Z,2023-03-29T19:02:39Z,COLLABORATOR,It seems that this problematic code is mostly used to determine the engine that is used to finally open it. Did you try specifying the correct engine directly?,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1646267547 https://github.com/pydata/xarray/pull/7698#issuecomment-1489144606,https://api.github.com/repos/pydata/xarray/issues/7698,1489144606,IC_kwDOAMm_X85Ywose,43316012,2023-03-29T19:01:21Z,2023-03-29T19:01:21Z,COLLABORATOR,"I think some backends rely on this magic number to determine the exact file format. Not sure if this change will cause problems if one doesn't get the full 8 bytes?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1646350377 https://github.com/pydata/xarray/pull/7553#issuecomment-1484860906,https://api.github.com/repos/pydata/xarray/issues/7553,1484860906,IC_kwDOAMm_X85YgS3q,43316012,2023-03-27T10:04:34Z,2023-03-27T10:04:34Z,COLLABORATOR,Thanks for this PR!,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1597118095 https://github.com/pydata/xarray/issues/6891#issuecomment-1484199176,https://api.github.com/repos/pydata/xarray/issues/6891,1484199176,IC_kwDOAMm_X85YdxUI,43316012,2023-03-26T19:30:08Z,2023-03-26T19:30:25Z,COLLABORATOR,"As said above, you don't have to change code to make it work. The intended way of using is `da.curvefit(...., kwargs={""method"": ""trf""})`. You can submit a PR which changes **kwargs to kwargs in the docstring (both, Dataset.curvefit and DataArray.curvefit) and probably add some more info there.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1331985070 https://github.com/pydata/xarray/pull/6515#issuecomment-1484196360,https://api.github.com/repos/pydata/xarray/issues/6515,1484196360,IC_kwDOAMm_X85YdwoI,43316012,2023-03-26T19:18:15Z,2023-03-26T19:18:15Z,COLLABORATOR,@StanczakDominik are you still motivated to work on these tests?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1215946244 https://github.com/pydata/xarray/issues/6891#issuecomment-1484195905,https://api.github.com/repos/pydata/xarray/issues/6891,1484195905,IC_kwDOAMm_X85YdwhB,43316012,2023-03-26T19:16:38Z,2023-03-26T19:16:38Z,COLLABORATOR,"Actually, as @kmuehlbauer pointed out, we can address this for now by updating the docstring of the function. It should get clear the the extra kwargs are passed as a dict and not as **kwargs.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1331985070 https://github.com/pydata/xarray/issues/7130#issuecomment-1484195481,https://api.github.com/repos/pydata/xarray/issues/7130,1484195481,IC_kwDOAMm_X85YdwaZ,43316012,2023-03-26T19:14:48Z,2023-03-26T19:15:00Z,COLLABORATOR,"> I am trying out open source for the first time through Outreachy. Could you please guide me a little so that I could proceed with this issue. Welcome to xarray! I think this issue requires some discussion before we can actually address code changes. It would be better to start with a different issue first.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1397104515 https://github.com/pydata/xarray/issues/6891#issuecomment-1484188179,https://api.github.com/repos/pydata/xarray/issues/6891,1484188179,IC_kwDOAMm_X85YduoT,43316012,2023-03-26T19:02:12Z,2023-03-26T19:02:12Z,COLLABORATOR,"I think we should continue the discussion in https://github.com/pydata/xarray/issues/7130 since it is not clear which function signature (kwargs vs **kwargs) is preferred.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1331985070 https://github.com/pydata/xarray/pull/7638#issuecomment-1484183805,https://api.github.com/repos/pydata/xarray/issues/7638,1484183805,IC_kwDOAMm_X85Ydtj9,43316012,2023-03-26T18:43:32Z,2023-03-26T18:43:32Z,COLLABORATOR,Thanks for this contribution!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1627983028 https://github.com/pydata/xarray/pull/7638#issuecomment-1484174808,https://api.github.com/repos/pydata/xarray/issues/7638,1484174808,IC_kwDOAMm_X85YdrXY,43316012,2023-03-26T18:07:05Z,2023-03-26T18:07:05Z,COLLABORATOR,"In https://github.com/pydata/xarray/blob/main/doc/whats-new.rst Add a line under the documentation section explaining what you have added. Use the rest of the file as a guideline on how to format.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1627983028 https://github.com/pydata/xarray/pull/7523#issuecomment-1484155605,https://api.github.com/repos/pydata/xarray/issues/7523,1484155605,IC_kwDOAMm_X85YdmrV,43316012,2023-03-26T16:50:35Z,2023-03-26T16:50:35Z,COLLABORATOR,Last call for reviews :),"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1581313830 https://github.com/pydata/xarray/pull/7638#issuecomment-1484151016,https://api.github.com/repos/pydata/xarray/issues/7638,1484151016,IC_kwDOAMm_X85Ydljo,43316012,2023-03-26T16:31:49Z,2023-03-26T16:31:49Z,COLLABORATOR,"@harshitha1201 feel free to add an entry in doc/whats-new.rst. Then we can merge this. Thanks!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1627983028 https://github.com/pydata/xarray/pull/7674#issuecomment-1483833424,https://api.github.com/repos/pydata/xarray/issues/7674,1483833424,IC_kwDOAMm_X85YcYBQ,43316012,2023-03-25T14:05:58Z,2023-03-25T14:05:58Z,COLLABORATOR,"About the logo, this might be relevant: https://github.com/xarray-contrib/xarray.dev/issues/66","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1640260666 https://github.com/pydata/xarray/pull/6956#issuecomment-1480142197,https://api.github.com/repos/pydata/xarray/issues/6956,1480142197,IC_kwDOAMm_X85YOS11,43316012,2023-03-22T19:31:00Z,2023-03-22T19:31:00Z,COLLABORATOR,"The error of mypy is real, but unfortunately this is a design error on our side, which is out of scope for this PR. see the mypy message: > xarray/backends/netCDF4_.py:563: error: Signature of ""open_dataset"" incompatible with supertype ""BackendEntrypoint"" [override] so simply add a `# type: ignore[override]` at the end of this line.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1352315409 https://github.com/pydata/xarray/pull/7657#issuecomment-1480136698,https://api.github.com/repos/pydata/xarray/issues/7657,1480136698,IC_kwDOAMm_X85YORf6,43316012,2023-03-22T19:26:18Z,2023-03-22T19:26:18Z,COLLABORATOR,"> Does anyone know why those take so long only on macos (and, partially, windows)? Does that have to do with the runner hardware? Some of there files are 10 numbers, so the netcdf should be ~50kb or so (mostly the header overhead). On no hardware this should take >1min to read and write?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1635470616 https://github.com/pydata/xarray/pull/7019#issuecomment-1477639520,https://api.github.com/repos/pydata/xarray/issues/7019,1477639520,IC_kwDOAMm_X85YEv1g,43316012,2023-03-21T11:03:05Z,2023-03-21T11:03:05Z,COLLABORATOR," > I just released Cubed 0.6.0 which doesn't have a dependency on Dask, so this should be possible now. Does this mean my comment https://github.com/pydata/xarray/pull/7019#discussion_r970713341 is valid again?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1368740629 https://github.com/pydata/xarray/issues/7646#issuecomment-1475399293,https://api.github.com/repos/pydata/xarray/issues/7646,1475399293,IC_kwDOAMm_X85X8M59,43316012,2023-03-19T21:13:48Z,2023-03-19T21:13:48Z,COLLABORATOR,"Can you see if you can run `import numba` in your environment? If this also raises an error, your numba installation is probably broken. If this works, then try `import sparse`. Anyway, please add the output of `xr.show_versions()` to your issue.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1630998880 https://github.com/pydata/xarray/pull/7521#issuecomment-1474861240,https://api.github.com/repos/pydata/xarray/issues/7521,1474861240,IC_kwDOAMm_X85X6Ji4,43316012,2023-03-18T14:08:20Z,2023-03-18T14:08:20Z,COLLABORATOR,"I guess we should revert this change and use our own definition as proposed by the numpy defs. If I find time I'll make a new PR such that we can be independent off numpys version in this regard.","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1580266844 https://github.com/pydata/xarray/issues/5985#issuecomment-1473820156,https://api.github.com/repos/pydata/xarray/issues/5985,1473820156,IC_kwDOAMm_X85X2LX8,43316012,2023-03-17T13:09:02Z,2023-03-17T13:10:06Z,COLLABORATOR,"Found as an answer to a proposal PR: https://github.com/pydata/xarray/pull/7628#discussion_r1140211918 The solution is to use `.str.format` the other way around as you propose: You can try `das.str.format(da)`. Maybe you need to use `{}` instead of `%` in your format array.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1052753606 https://github.com/pydata/xarray/pull/7604#issuecomment-1471425997,https://api.github.com/repos/pydata/xarray/issues/7604,1471425997,IC_kwDOAMm_X85XtC3N,43316012,2023-03-16T07:07:52Z,2023-03-16T07:07:52Z,COLLABORATOR,That issue will be fixed by #7619,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1618789537 https://github.com/pydata/xarray/pull/7628#issuecomment-1471413892,https://api.github.com/repos/pydata/xarray/issues/7628,1471413892,IC_kwDOAMm_X85Xs_6E,43316012,2023-03-16T06:53:52Z,2023-03-16T06:53:52Z,COLLABORATOR,"Again, I think that this implementation is too specific and the issue wanted a more general string formatting option. Also, auch a method is better kept as part of the StrAccessor.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1624520287 https://github.com/pydata/xarray/pull/7625#issuecomment-1470829677,https://api.github.com/repos/pydata/xarray/issues/7625,1470829677,IC_kwDOAMm_X85XqxRt,43316012,2023-03-15T20:56:37Z,2023-03-15T20:56:37Z,COLLABORATOR,Could you also merge main again?,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1623949333 https://github.com/pydata/xarray/pull/7586#issuecomment-1470782099,https://api.github.com/repos/pydata/xarray/issues/7586,1470782099,IC_kwDOAMm_X85XqlqT,43316012,2023-03-15T20:17:33Z,2023-03-15T20:17:33Z,COLLABORATOR,Could this potentially affect performance?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1610063645 https://github.com/pydata/xarray/pull/7628#issuecomment-1469697108,https://api.github.com/repos/pydata/xarray/issues/7628,1469697108,IC_kwDOAMm_X85XmcxU,43316012,2023-03-15T09:57:12Z,2023-03-15T09:57:12Z,COLLABORATOR,"By the way, this works (but probably slow?): ```python def tostr(x): return f""{:.2f}"" tostr_v = np.vectorize(tostr) da = xr.DataArray([1, 2, 3.456], dims=""x"") tostr_v(da) ```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1624520287 https://github.com/pydata/xarray/pull/7628#issuecomment-1469677146,https://api.github.com/repos/pydata/xarray/issues/7628,1469677146,IC_kwDOAMm_X85XmX5a,43316012,2023-03-15T09:43:57Z,2023-03-15T09:44:33Z,COLLABORATOR,"Wouldn't it be more useful to have something like `da.str.format(""sometext: {:2.2f}"")` Or maybe `da.as_str(...)`","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1624520287 https://github.com/pydata/xarray/issues/6914#issuecomment-1466890220,https://api.github.com/repos/pydata/xarray/issues/6914,1466890220,IC_kwDOAMm_X85Xbvfs,43316012,2023-03-13T20:11:15Z,2023-03-13T20:11:15Z,COLLABORATOR,@melioristic could you reproduce the matplotlib results by setting interpolation?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1338173609 https://github.com/pydata/xarray/issues/7372#issuecomment-1466868513,https://api.github.com/repos/pydata/xarray/issues/7372,1466868513,IC_kwDOAMm_X85XbqMh,43316012,2023-03-13T19:56:31Z,2023-03-13T19:56:31Z,COLLABORATOR,Then let's close this issue in favor of #4817,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1486990254 https://github.com/pydata/xarray/issues/7385#issuecomment-1466864506,https://api.github.com/repos/pydata/xarray/issues/7385,1466864506,IC_kwDOAMm_X85XbpN6,43316012,2023-03-13T19:53:06Z,2023-03-13T19:53:06Z,COLLABORATOR,@dopplershift does this answer fix your problem?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1499473190 https://github.com/pydata/xarray/pull/7612#issuecomment-1465733617,https://api.github.com/repos/pydata/xarray/issues/7612,1465733617,IC_kwDOAMm_X85XXVHx,43316012,2023-03-13T08:50:53Z,2023-03-13T08:50:53Z,COLLABORATOR,"> Nice! (at one point we can consider removing `infer_intervals` as this is now also done in matplotlib and cartopy). Yeah, I was wondering why we do this explicitly. Should we remove it now already, or is there something that still requires it (like older matplotlib versions etc)?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1620317764 https://github.com/pydata/xarray/issues/4114#issuecomment-1465302631,https://api.github.com/repos/pydata/xarray/issues/4114,1465302631,IC_kwDOAMm_X85XVr5n,43316012,2023-03-12T21:20:35Z,2023-03-12T21:20:35Z,COLLABORATOR,"I assume this has been resolved then? If not, feel free to reopen.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,627805952 https://github.com/pydata/xarray/issues/5666#issuecomment-1465290779,https://api.github.com/repos/pydata/xarray/issues/5666,1465290779,IC_kwDOAMm_X85XVpAb,43316012,2023-03-12T20:20:27Z,2023-03-12T20:20:27Z,COLLABORATOR,"Thanks for the report and sorry for the late reply :) This is not an issue of xarray but rather netCDF4. The corresponding issue can be found here: https://github.com/Unidata/netcdf4-python/issues/941 It seems that there is no solution yet. Since it seems that it is a HDF5 issue I don't know if any other backend could fix this issue for you (I tried h5netcdf without success). I will close this issue for now since we cannot fix it anyway.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,958557945 https://github.com/pydata/xarray/issues/5836#issuecomment-1465235251,https://api.github.com/repos/pydata/xarray/issues/5836,1465235251,IC_kwDOAMm_X85XVbcz,43316012,2023-03-12T15:59:13Z,2023-03-12T15:59:13Z,COLLABORATOR,"This is indeed intentional (Now with type hints even more obvious). I agree that in a jupyter notebook that automatically displays the last return value, this might look confusing.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1013881639 https://github.com/pydata/xarray/issues/6244#issuecomment-1465230423,https://api.github.com/repos/pydata/xarray/issues/6244,1465230423,IC_kwDOAMm_X85XVaRX,43316012,2023-03-12T15:38:37Z,2023-03-12T15:38:37Z,COLLABORATOR,Almost all files have `from __future__ import annotations` now and pyupgrade is doing its thing :),"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1125040125 https://github.com/pydata/xarray/issues/6870#issuecomment-1465228869,https://api.github.com/repos/pydata/xarray/issues/6870,1465228869,IC_kwDOAMm_X85XVZ5F,43316012,2023-03-12T15:32:01Z,2023-03-12T15:32:01Z,COLLABORATOR,"The logic of figuring out what stays and what goes is different in `DataArray` and `Dataset`. `DataArray` reduces the variable and then kicks out all dims, coords, indexes that don't fit anymore. `Dataset` tries to do the selection on the fly (and apparently fails to do so).","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1326238990 https://github.com/pydata/xarray/issues/6870#issuecomment-1465227486,https://api.github.com/repos/pydata/xarray/issues/6870,1465227486,IC_kwDOAMm_X85XVZje,43316012,2023-03-12T15:25:47Z,2023-03-12T15:27:31Z,COLLABORATOR,"This is actually a bug in `Dataset.reduce`, it will drop the time index as well. See this example: ```python da = xr.DataArray([[1, 2, 3], [4, 5, 6]], dims=[""x"", ""y""], coords={""x"": [5, 6]}) da.reduce(lambda x, *_, **__: x, dim=""x"") # has time coord ds = xr.Dataset({""a"": da}) ds.reduce(lambda x, *_, **__: x, dim=""x"") # lost time coord ``` Probably the assumtion when writing this code was that the passed function actually reduces the dim.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1326238990 https://github.com/pydata/xarray/issues/6990#issuecomment-1465150129,https://api.github.com/repos/pydata/xarray/issues/6990,1465150129,IC_kwDOAMm_X85XVGqx,43316012,2023-03-12T10:17:13Z,2023-03-12T10:17:13Z,COLLABORATOR,"Closing for now. Feel free to reopen or better to comment in the linked issues above.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1361989642 https://github.com/pydata/xarray/issues/6877#issuecomment-1465149441,https://api.github.com/repos/pydata/xarray/issues/6877,1465149441,IC_kwDOAMm_X85XVGgB,43316012,2023-03-12T10:14:37Z,2023-03-12T10:14:37Z,COLLABORATOR,"Closing for now. Seems like an upstream issue. Feel free to reopen if it is still relevant.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1329407773 https://github.com/pydata/xarray/issues/1004#issuecomment-1465000306,https://api.github.com/repos/pydata/xarray/issues/1004,1465000306,IC_kwDOAMm_X85XUiFy,43316012,2023-03-11T19:57:40Z,2023-03-11T19:57:40Z,COLLABORATOR,"Am I missing something, or is there no `IndexVariable.name` attribute anymore?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,176805500 https://github.com/pydata/xarray/issues/7611#issuecomment-1464948290,https://api.github.com/repos/pydata/xarray/issues/7611,1464948290,IC_kwDOAMm_X85XUVZC,43316012,2023-03-11T16:34:50Z,2023-03-11T16:34:50Z,COLLABORATOR,"Sounds like a good idea to mention the license and what it means in the documentation. We don't assign issues to people, so if you are interested in doing so feel free to open a PR.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1619980197 https://github.com/pydata/xarray/pull/7368#issuecomment-1464927490,https://api.github.com/repos/pydata/xarray/issues/7368,1464927490,IC_kwDOAMm_X85XUQUC,43316012,2023-03-11T14:50:02Z,2023-03-11T14:50:24Z,COLLABORATOR,"> DataAlignable = TypeVar(""DataAlignable"", bound=DataWithCoords | Coordinates) -> doesn't work since we cannot mix DataWithCoords and Coordinates when aligning each object (input type = output type) I think thats exactly correct. The whold idea of using a TypeVar for the inputs means that all inputs must have the same type. Consider the following example: ```python class A: ... class B(A): ... class C: ... T = TypeVar(""T"", bound=Union[A, C]) def f(*x: T) -> T: ... f(A(), A()) # OK f(A(), B()) # OK f(C(), C()) # OK f(A(), C()) # not ok, because the common type of A and C is object ``` Also: sorry for the late reply.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1485037066 https://github.com/pydata/xarray/issues/7336#issuecomment-1464386387,https://api.github.com/repos/pydata/xarray/issues/7336,1464386387,IC_kwDOAMm_X85XSMNT,43316012,2023-03-10T20:32:50Z,2023-03-10T20:32:50Z,COLLABORATOR,Closing this issue now as completed. Discussion about documentation can be continued in #7344 ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1468838643 https://github.com/pydata/xarray/issues/7344#issuecomment-1464384781,https://api.github.com/repos/pydata/xarray/issues/7344,1464384781,IC_kwDOAMm_X85XSL0N,43316012,2023-03-10T20:31:28Z,2023-03-10T20:31:28Z,COLLABORATOR,Maybe it is just a problem of documenting it more clearly?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1471685307 https://github.com/pydata/xarray/issues/5762#issuecomment-1464348270,https://api.github.com/repos/pydata/xarray/issues/5762,1464348270,IC_kwDOAMm_X85XSC5u,43316012,2023-03-10T20:01:18Z,2023-03-10T20:01:18Z,COLLABORATOR,Closing this issue in favor of #7593 since the provided MVCE in this issue now works.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,987551524 https://github.com/pydata/xarray/issues/7199#issuecomment-1464331562,https://api.github.com/repos/pydata/xarray/issues/7199,1464331562,IC_kwDOAMm_X85XR-0q,43316012,2023-03-10T19:49:10Z,2023-03-10T19:49:10Z,COLLABORATOR,"Any ideas on how to deprecate this? Do we raise a Deprecation warning if someone uses the open_dataset method of the internal backend? And then remove it some versions down the line?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1419825696 https://github.com/pydata/xarray/pull/7521#issuecomment-1462773338,https://api.github.com/repos/pydata/xarray/issues/7521,1462773338,IC_kwDOAMm_X85XMCZa,43316012,2023-03-09T20:44:04Z,2023-03-09T20:44:04Z,COLLABORATOR,"It seems that numpy is somewhat against this change, so maybe we are better off to restore the previous state to not have a mess with supporting different numpy versions again. Lets keep that in mind before the next release but wait if something changes on numpys side.","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1580266844 https://github.com/pydata/xarray/pull/7600#issuecomment-1462760816,https://api.github.com/repos/pydata/xarray/issues/7600,1462760816,IC_kwDOAMm_X85XL_Vw,43316012,2023-03-09T20:40:25Z,2023-03-09T20:40:25Z,COLLABORATOR,"Ok :) But I guess it is better to not merge this then, it will only create messy merge conflicts for other PRs.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1615980379 https://github.com/pydata/xarray/pull/3607#issuecomment-1462754116,https://api.github.com/repos/pydata/xarray/issues/3607,1462754116,IC_kwDOAMm_X85XL9tE,43316012,2023-03-09T20:37:18Z,2023-03-09T20:37:18Z,COLLABORATOR,"Is that still wanted? It seems like a useful feature to have.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,535686852 https://github.com/pydata/xarray/pull/7578#issuecomment-1462752419,https://api.github.com/repos/pydata/xarray/issues/7578,1462752419,IC_kwDOAMm_X85XL9Sj,43316012,2023-03-09T20:36:31Z,2023-03-09T20:36:31Z,COLLABORATOR,"Good spot. The fix seems good to me. If you fix the tests and add a whats-new this is good to go","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1607502760 https://github.com/pydata/xarray/pull/7598#issuecomment-1460859161,https://api.github.com/repos/pydata/xarray/issues/7598,1460859161,IC_kwDOAMm_X85XEvEZ,43316012,2023-03-08T20:50:46Z,2023-03-08T20:50:46Z,COLLABORATOR,"> As a newcomer, a short general question: The 'Contributing' guide mentions to document changes in `whats-new.rst`. Shall this be done by the developer or are you (the maintainers) doing this once the PR is merged into the main branch? > > (Since at the time of requesting, I can`t know when (or if) the issue will be merged). Usually this is done by the developer/contributor. You can just copy a similar entry and adopt it to yourself in the newest section, you have basically full freedom :)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1615645463 https://github.com/pydata/xarray/pull/7521#issuecomment-1450819830,https://api.github.com/repos/pydata/xarray/issues/7521,1450819830,IC_kwDOAMm_X85WecD2,43316012,2023-03-01T20:44:52Z,2023-03-01T20:44:52Z,COLLABORATOR,"I have opened https://github.com/numpy/numpy/issues/23308 It should be an easy change on numpys side if they decide to go for it. Only problem, we again have to special case this based on bumpy version, haha.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1580266844 https://github.com/pydata/xarray/issues/7519#issuecomment-1449430920,https://api.github.com/repos/pydata/xarray/issues/7519,1449430920,IC_kwDOAMm_X85WZI-I,43316012,2023-03-01T06:34:48Z,2023-03-01T06:35:08Z,COLLABORATOR,"> Should we special case a solution? Seems user-friendly to do so. I think dict_keys is its own type so with some type ignore this could work.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1579956621 https://github.com/pydata/xarray/pull/7521#issuecomment-1449428744,https://api.github.com/repos/pydata/xarray/issues/7521,1449428744,IC_kwDOAMm_X85WZIcI,43316012,2023-03-01T06:31:46Z,2023-03-01T06:31:46Z,COLLABORATOR," > Do you know if there's a plan to make something like this public on the numpy end? No idea, the static typing of numpy is still changing a lot and has many bugs (same with xarray, haha). I'm sure they will make it public is we request it. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1580266844 https://github.com/pydata/xarray/issues/3980#issuecomment-1449013427,https://api.github.com/repos/pydata/xarray/issues/3980,1449013427,IC_kwDOAMm_X85WXjCz,43316012,2023-02-28T22:24:56Z,2023-02-28T22:24:56Z,COLLABORATOR,"I will not be able to join tomorrow, so here are my thoughts on this topic: - How to work with subclassed Datasets that should return DataArrays? I think the pandas approach is good, we add `Dataset.dataarray_cls = DataArray` and `DataArray.dataset_cls = Dataset` which subclasses can overwrite. - How to deal with subclasses that change the constructor aka. `__init__`? Probably we need to introduce a kind of shortcut constructor that will be called instead in functions that create new instances (right now the subclass needs the same constructor signature as xarrays version). But then custom instance attributes will not be set etc. Not sure what is the best approach here. - Is the whole thing limited to DataArrays and Datasets or can one also subclass Variables?","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,602218021 https://github.com/pydata/xarray/issues/7552#issuecomment-1447067702,https://api.github.com/repos/pydata/xarray/issues/7552,1447067702,IC_kwDOAMm_X85WQIA2,43316012,2023-02-27T20:50:13Z,2023-02-27T20:50:13Z,COLLABORATOR,@liorella-qm in the meantime you can use `da.plot.line(col='y')` as a workaround,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1596890616 https://github.com/pydata/xarray/issues/7552#issuecomment-1447057892,https://api.github.com/repos/pydata/xarray/issues/7552,1447057892,IC_kwDOAMm_X85WQFnk,43316012,2023-02-27T20:43:00Z,2023-02-27T20:48:19Z,COLLABORATOR,"This is caused by the `squeeze()` here: https://github.com/pydata/xarray/blob/41949209b6cfb0d5c25d000b1e0c5739b513385c/xarray/plot/dataarray_plot.py#L267 @Illviljan do you know why this is required? I assume you want to remove singleton dimensions to better choose the correct plot method? Maybe we should exclude col, row dims from the squeeze? something like: `darray.squeeze(d for d, s in darray.sizes.items() if s == 1 and d not in (row, col, hue)) `","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1596890616 https://github.com/pydata/xarray/issues/7516#issuecomment-1447037080,https://api.github.com/repos/pydata/xarray/issues/7516,1447037080,IC_kwDOAMm_X85WQAiY,43316012,2023-02-27T20:27:52Z,2023-02-27T20:27:52Z,COLLABORATOR,"I am a bit puzzled here... The dask graph looks identical, so it must be the way the indexers are constructed. The major difference I can find is: The old version used `np.unique` while the new version uses xarrays `cond.any(..)` Maybe someone with more experience in dask can help out?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1575938277 https://github.com/pydata/xarray/issues/7516#issuecomment-1445469752,https://api.github.com/repos/pydata/xarray/issues/7516,1445469752,IC_kwDOAMm_X85WKB44,43316012,2023-02-26T21:16:35Z,2023-02-26T21:16:35Z,COLLABORATOR,"Git bisect pinpoints this to https://github.com/pydata/xarray/pull/6690 which funny enough, is my PR haha. I will look into it when I find time :)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1575938277 https://github.com/pydata/xarray/issues/7516#issuecomment-1445467918,https://api.github.com/repos/pydata/xarray/issues/7516,1445467918,IC_kwDOAMm_X85WKBcO,43316012,2023-02-26T21:07:56Z,2023-02-26T21:07:56Z,COLLABORATOR,"Can confirm, on my machine it went from 520ms to 5s","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1575938277 https://github.com/pydata/xarray/issues/7513#issuecomment-1445457887,https://api.github.com/repos/pydata/xarray/issues/7513,1445457887,IC_kwDOAMm_X85WJ-_f,43316012,2023-02-26T20:21:23Z,2023-02-26T20:21:23Z,COLLABORATOR,"Not sure if related, but this also randomly failed (different error though): https://github.com/pydata/xarray/actions/runs/4276072684/jobs/7443939252","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1574694462