xgcm / xrft

Fourier transforms on xarray data structures
https://xrft.readthedocs.io/en/latest/
MIT License
154 stars 45 forks source link

Linting for a new version release #186

Closed roxyboy closed 2 years ago

roxyboy commented 2 years ago

This PR removes the deprecated False flags from the functions and sets them to True as default. This addresses issue #177 .

roxyboy commented 2 years ago

@lanougue Would you be ok with the linting I've done?

lanougue commented 2 years ago

Hi @roxyboy , In cross_phase calculation, the default phase seems to be still "False". To be coherent, it should be changed to True, right ?

roxyboy commented 2 years ago

Thanks for the review @lanougue :) I'll merge this and release a new version.