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/6548#issuecomment-1117620357,https://api.github.com/repos/pydata/xarray/issues/6548,1117620357,IC_kwDOAMm_X85CnYiF,43316012,2022-05-04T17:33:07Z,2022-05-04T17:33:37Z,COLLABORATOR,"Personally I would allow coeffs without explicit index since I am a lazy person and would like to do `coeffs = xr.DataArray([1,2], dims=""degree"")`.
But I guess with the new indexing system you want to encourage people to use them.
But I am happy with this code and look forward to use it in my projects :)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1221848774
https://github.com/pydata/xarray/pull/6548#issuecomment-1115830683,https://api.github.com/repos/pydata/xarray/issues/6548,1115830683,IC_kwDOAMm_X85Cgjmb,43316012,2022-05-03T07:57:46Z,2022-05-03T08:06:29Z,COLLABORATOR,"One minor open point: what to do with a non-integer ""degree"" index?
Float type could be cast to integer (thats what is happening now).
But (nonsense) datetime etc. should raise an error?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1221848774
https://github.com/pydata/xarray/pull/6548#issuecomment-1114539954,https://api.github.com/repos/pydata/xarray/issues/6548,1114539954,IC_kwDOAMm_X85Cboey,43316012,2022-05-02T06:30:21Z,2022-05-02T09:50:49Z,COLLABORATOR,"Edit: nvmd, was only confusing output when the benchmark was failing. Now the benchmark looks good :)
First time working with asv...
It seems that module level variables affect all other peakmem tests (i guess memory usage of the phyton process is measured,).
We should refactor all dataarrays into the setup functions, otherwise O(n) memory algos will show wrong numbers and adding new tests will show regression on other tests.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1221848774
https://github.com/pydata/xarray/pull/6548#issuecomment-1114297460,https://api.github.com/repos/pydata/xarray/issues/6548,1114297460,IC_kwDOAMm_X85CatR0,43316012,2022-05-01T18:00:07Z,2022-05-01T18:00:07Z,COLLABORATOR,"Benchmark did not succeed since the inputs are not compatible with the old algorith...
Do we change it such that it is compatible?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1221848774
https://github.com/pydata/xarray/pull/6548#issuecomment-1114002143,https://api.github.com/repos/pydata/xarray/issues/6548,1114002143,IC_kwDOAMm_X85CZlLf,43316012,2022-04-30T14:57:58Z,2022-05-01T11:50:12Z,COLLABORATOR,"Several open points still:
1. [x] Unittests for datetime values are failing, I might need some help with that since I have no idea what this means for polynomials.
2. [x] Algorithm should work also with Datasets (any combination of DataArray and Dataset for `coord` and `coeffs` inputs). Still needs to be checked and tested (How does one define typing for such cases? I.e. DataArray + Dataset -> Dataset but DataArray + DataArray -> DataArray?)
3. [x] It uses Horners method instead of Vandermonde matrix, should be faster and consume less memory (unless the overhead of sorting index, isel etc. is too large). Maybe some performance comparisons should be done.
4. [ ] Instead of `coord` the input should simply be called `x` or similar, however this would break backwards compatibility, so maybe we just leave it.
5. [x] I had to add a `copy(deep=True)` since the broadcast returned a read-only DataArray. Any better ideas?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1221848774
https://github.com/pydata/xarray/pull/6548#issuecomment-1114212076,https://api.github.com/repos/pydata/xarray/issues/6548,1114212076,IC_kwDOAMm_X85CaYbs,43316012,2022-05-01T11:41:54Z,2022-05-01T11:41:54Z,COLLABORATOR,"I added a rough support for datetime values. Someone with more knowledge of handling them should take a look, the code seems too complicated and I am sure there is a more clever solution (I could not use `get_clean_interp_index` since it is not an index anymore).
> I agree I'm not sure whether we need to support them.
I think keeping support is nice, since they are a commonly occuring coordinates and we do not want to break anything if possible.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1221848774
https://github.com/pydata/xarray/pull/6548#issuecomment-1114196391,https://api.github.com/repos/pydata/xarray/issues/6548,1114196391,IC_kwDOAMm_X85CaUmn,43316012,2022-05-01T10:27:52Z,2022-05-01T10:27:52Z,COLLABORATOR,"Some performance comparison:
With 5th order polynomial and 10 x-values:
old: 1.05 ms ± 15.8 µs per loop
new: 1.41 ms ± 11.6 µs per loop
With 5th order polynomial and 10000 x-values:
old: 1.46 ms ± 10.5 µs per loop
new: 1.41 ms ± 14.5 µs per loop
With 5th order polynomial and 1mio x-values:
old: 65.1 ms ± 332 µs per loop
new: 6.99 ms ± 168 µs per loop
As expected for small arrays the new method creates some overhead, but for larger arrays the speedup is quite nice.
Also, it uses in-place operations with much less memory usage.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1221848774
https://github.com/pydata/xarray/pull/6548#issuecomment-1114028796,https://api.github.com/repos/pydata/xarray/issues/6548,1114028796,IC_kwDOAMm_X85CZrr8,43316012,2022-04-30T18:01:10Z,2022-04-30T18:01:10Z,COLLABORATOR,"I noticed that broadcasting Datasets behaves weird, see https://github.com/pydata/xarray/issues/6549, so I used a ""hack"" of adding an 0-valued DataArray/Dataset.
Anyone got a better idea?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1221848774