Éric Dupuis
Éric Dupuis
@SarahG-579462 Thoughts on these changes? Especially `transform` -> `normalization` which is a matter of taste.
It's a step in the right direction, but I realze that's not even what I had in mind for the standardization. Right now, we just remove the mean/std of the...
Maybe the dOTC evolution through cells (more fidel to the paper) can wait another PR. But, as I was refering in my last comment, I don't think what we were...
I'll just merge this PR now, other changes can be in another PR.
Very nice! And thanks for the heads up, I will definitely give it a try.
On first sight, it seems the same as: #1276 lazy indexing doesn't work with many xarray function `doymax` uses `da.argmax(dim="time")`, etc.
Hi! Indeed, this indicator requires xarray.DataArray.rolling, so it makes sense it fails too. I can't reproduce your problem with some testdata with xclim==0.56.0 and xarray==2025.6.1 ```python import xclim import xarray...
Hi! We can add this indeed. Just a couple caveats: 1. It will only work with standard calendars, as `.resample(time="W")` and `.groupby("time.week")` is not yet compatible with cf-time (https://github.com/pydata/xarray/discussions/6375) 2....
@C15H24 We decided to support the weekly frequency after all, this will be in xclim's next release. Enjoy!
The SPI functions were only the canary in the mine... indices assigning "" (dimensionless units) are concerned, you can also test "test_ffdi.py" to see this. It comes from pint2cfunits, here...