xarray-contrib / xeofs

Comprehensive EOF analysis in Python with xarray: A versatile, multidimensional, and scalable tool for advanced climate data analysis
https://xeofs.readthedocs.io/
MIT License
105 stars 20 forks source link

Using `normalize=False` as default for all scores #203

Closed nicrie closed 2 months ago

nicrie commented 2 months ago

With the upcoming major release, should we consider changing the default of the normalize parameter in the various (PC) scores from True to False? This would better align with the scikit-learn API, potentially reducing confusion for users familiar with that framework. Do you have strong opinions on this @slevang ?

slevang commented 2 months ago

That seems reasonable but no strong opinions from me.