-
Notifications
You must be signed in to change notification settings - Fork 2
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
Updated xarray breaks dscim #103
Comments
This is partly because binary operations respect This means that the binary operation to calculate the anomaly here can cause later problems when the anomaly shares attrs keys with data it will be combined with here. Keeping old behavior means ditching attrs during those binary operations, but maybe we should consider how |
Another problem seems to be that as of xarray v2023.3.0 bool arrays used for indexing can't be dask-backed. They need to be computed and in memory so their size can be comprehended. This is a problem in lines like this. |
Dependabot tried to bump xarray 2022.11.0 to 2023.7.0 in #101 and it failed tests in CI.
Here are the archived logs from that CI run in case the link expires: logs_381.zip
I haven't dug through this but should fix the package so it can run on the latest versions of xarray.
The text was updated successfully, but these errors were encountered: