Michael Niklas

Results 194 comments of Michael Niklas

> I noticed that [DataArray.interp_like](https://docs.xarray.dev/en/stable/generated/xarray.DataArray.interp_like.html) takes an argument called `kwargs`, but not in the traditional manner which allows passing keyworded arguments (`**kwargs`). > > If this is the intended behavior...

The `QUANTILE_METHODS` has been moved and renamed to `core.types.QuantileMethods`. Probably something went wrong in the merge.

Since the merge apparently failed, I took the freedom to rebase this from the current master. Feel free to continue or we merge here :)

The PR doesn't really help. In the codecov website under flags the percentages are correctly displayed but in the mixed coverage the tests are still included leading to only 68%...

Is that something that will be deprecated or is it planned to keep the support for the kwargs dict forever?

> I think this is ready, unfortunately it does not make sense to merge it yet, and one test is failing, because netcdf-c 4.9.0 is not yet available in conda-forge...

Is that a valid syntax: `xr.Dataset({"a": [["x"], [1, 2, 3]]})`, or does one need to use a tuple? It seems to use the full list as data and then fails...

Ok, then I think using a list to create a DataArray is not supported by xarray. I never noticed before, haha!

It is not an easy fix since we do not simply want to return None, and returning an empty dict in this case without overwriting the internal one would break...

This has been fixed 1 year ago in xarray: https://github.com/pydata/xarray/pull/5959 Consider updating xarray.