Niclas Rieger

I had a look at the `experimental-rotation` branch and it indeed looks quite good. The design of using a rotator class makes sense to me, and the calculations are accurate....

@ajdawson I just opened a PR adding Varimax-rotated PCA functionality to the `standard` interface. Currently implemented for rotated PCA is: - `eofs` - `pcs` - `eigenvalues` (representing variance of each...

In case anyone is interested: Varimax and Promax rotated EOF analysis is now available in [xeofs](https://github.com/nicrie/xeofs).

same error message here when running ```py regridder = xe.Regridder(ds, ds_out, "bilinear") ``` where `ds` is a `dask` `Dataset`

dask support works fine for real numbers, both with and without rotation. However, in case of complex input for `dask.array.linalg.svd_compressed()` there seems to be a bug. This project will be...

Just to be sure, xMCA only works with `xr.DataArray` (in xeofs Dataset will be allowed soon), so be sure to convert your Dataset before the analysis. Also , please ensure...

I must admit I'm not sure of a package that can automatically apply this kind of weighting for you, but that's not to say it doesn't exist! However, I'm a...

Hi zhiweijt - sorry for the late reply! Upon examining the figures, the results do appear to be consistent. The PCs/EOFs are only defined up to a sign, meaning that...

Agree with you @slevang , that'd be nice. Currently, the rank is only computed when we fit the `Decomposer` https://github.com/xarray-contrib/xeofs/blob/a3cb204171dbbd1f09f2c083099cf3b67f625495/xeofs/models/decomposer.py#L72-L74 The first opportunity to compute the rank is probably already...

The error is due to an unfortunate limitation of this package that the spatial coordinates have to be named `lon` and `lat`. Currently, your main spatial coordinates are called `west_east`...