Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do you really need the newest xarray>=2023 dependency? #35

Open
smartass101 opened this issue Jan 11, 2024 · 3 comments
Open

Do you really need the newest xarray>=2023 dependency? #35

smartass101 opened this issue Jan 11, 2024 · 3 comments

Comments

@smartass101
Copy link
Contributor

xrscipy used to work just fine with older xarray versions, so unless you really need xrscipy>=2023, please don't introduce such a strict dependency which prevents simple xrscipy installation in older environments (in scientific environments and clusters one does not always have a choice).

@hippalectryon-0
Copy link
Owner

Can you expand on which setting would not allow bumping xarray>=2023 ?

Unlike libraries like cartopy which rely on compiled libraries like libgeos, xarray is easily installed via pip/... without any special account rights.

Moreover, what's preventing the occasional users that really can't install this requirement to simply install an older version of xrscipy ?

xarray has undergone a number of improvements over the past years, some of which are breaking, and I can't guarantee that the current code would run exactly the same with older versions of the library.

@smartass101
Copy link
Contributor Author

xarray is easily installed via pip/... without any special account rights.

In typical circumstances yes, but not so easily on larger cluster systems or scientific environments where very often only certain Python environments and setups are supported and going outside them (e.g installing a a user pip or conda env) can be very difficult (loss of functionality in terms of specialized libraries). Often they may even have xarray installed and used by some software, but an older version than 2023.

Moreover, what's preventing the occasional users that really can't install this requirement to simply install an older version of xrscipy ?

This may indeed be a workaround for those that really need it, but such a hurdle needlessly lowers the possible appeal of xrscipy to such users.

xarray has undergone a number of improvements over the past years, some of which are breaking, and I can't guarantee that the current code would run exactly the same with older versions of the library.

If the xrscipy test suite passes with an older version of xarray, it should be fine. xrscipy does not really use the newest bells and whisles.

@hippalectryon-0
Copy link
Owner

If the xrscipy test suite passes with an older version of xarray, it should be fine. xrscipy does not really use the newest bells and whisles.

I'd agree with you if the test suite was actually complete, but as it stands only fft and integrate are properly tested.

If you feel like adding more test coverage (via PR), I'd be happy to integrate it and then find the oldest version of xarray that pass all tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants