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

Update to fix deprecation warning from dask regarding atop #608

Closed
djhoese opened this issue Feb 6, 2019 · 2 comments
Closed

Update to fix deprecation warning from dask regarding atop #608

djhoese opened this issue Feb 6, 2019 · 2 comments

Comments

@djhoese
Copy link
Member

djhoese commented Feb 6, 2019

A user reported an error on slack that also showed a deprecation warning in dask. This may affect both satpy and pyresample:

/home/raghav/anaconda2/lib/python2.7/site-packages/dask/array/blockwise.py:204: UserWarning: The da.atop function has moved to da.blockwise
  warnings.warn("The da.atop function has moved to da.blockwise")

We should switch to using blockwise instead of atop when possible.

@mraspaud Maybe we should make all UserWarning/DeprecationWarnings raise errors on travis.

@mraspaud
Copy link
Member

mraspaud commented Feb 6, 2019

Raising on travis sounds good.

@djhoese
Copy link
Member Author

djhoese commented Jun 1, 2021

I think we updated these usages a long time ago. And we can raise all warnings on travis (now github actions) as we sometimes deliberately test deprecated interfaces of other libraries. We do have an "unstable" environment which should catch things when they are completely removed from upstream packages.

@djhoese djhoese closed this as completed Jun 1, 2021
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