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.

Sign up for GitHub

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

Jump to bottom

use dask.array functions instead of numpy #367

Merged

fneum merged 3 commits into master from use-dask.array

Sep 8, 2024

Member

fneum commented Aug 5, 2024 •

edited

Loading

I had hoped for some performance gains, but this is not so clear.

fneum and others added 2 commits

August 5, 2024 09:09


          use dask.array functions instead of numpy

75ea941


          [pre-commit.ci] auto fixes from pre-commit.com hooks

5d6aa4b

for more information, see https://pre-commit.ci

Collaborator

euronion commented Aug 6, 2024

NB:
Most of the dask functions you have changed are just calls to the same numpy functions using ufunc, see the "direct" entries here: https://docs.dask.org/en/stable/array-numpy-compatibility.html#compatibility-with-numpy-functions .

Changing does give a bit of consistency, I'm not opposed.


          Merge branch 'master' into use-dask.array

90bec1d

fneum merged commit 74e961f into master

2 of 6 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet