Skip to content

Rewrite da.fromfunction w/ da.blockwise#7704

Merged
jrbourbeau merged 1 commit intodask:mainfrom
jakirkham:rewrite_fromfunction_blockwise
Jun 3, 2021
Merged

Rewrite da.fromfunction w/ da.blockwise#7704
jrbourbeau merged 1 commit intodask:mainfrom
jakirkham:rewrite_fromfunction_blockwise

Conversation

@jakirkham
Copy link
Member

@jakirkham jakirkham commented May 26, 2021

This simplifies the implementation of da.fromfunction by rewriting it in terms of da.blockwise. As a result da.fromfunction can take advantage of HLG by default. This also makes it more natural to feed into other HLG operations.

  • Closes #xxxx
  • Tests added / passed
  • Passes black dask / flake8 dask / isort dask

Also addresses the fromfunction task in issue ( #6791 )

@github-actions github-actions bot added the array label May 26, 2021
@jakirkham jakirkham force-pushed the rewrite_fromfunction_blockwise branch 2 times, most recently from 8427ed0 to c9e134a Compare May 26, 2021 03:25
@jakirkham jakirkham marked this pull request as draft May 26, 2021 03:49
@jakirkham jakirkham changed the title Rewrite da.fromfunction w/ da.blockwise WIP: Rewrite da.fromfunction w/ da.blockwise May 26, 2021
@jakirkham jakirkham force-pushed the rewrite_fromfunction_blockwise branch 2 times, most recently from b7e2b08 to 20a400d Compare May 26, 2021 03:56
This simplifies the implementation of `da.fromfunction` by rewriting it
in terms of `da.blockwise`. As a result `da.fromfunction` can take
advantage of HLG by default. This also makes it more natural to feed
into other HLG operations.
@jakirkham jakirkham force-pushed the rewrite_fromfunction_blockwise branch from 20a400d to 8efa8b8 Compare May 26, 2021 04:14
@jakirkham jakirkham changed the title WIP: Rewrite da.fromfunction w/ da.blockwise Rewrite da.fromfunction w/ da.blockwise May 26, 2021
@jakirkham jakirkham marked this pull request as ready for review May 26, 2021 04:15
@jakirkham
Copy link
Member Author

cc @jrbourbeau

@jakirkham
Copy link
Member Author

Planning on merging tomorrow if no comments

Copy link
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jakirkham!

@jrbourbeau jrbourbeau merged commit dc9746f into dask:main Jun 3, 2021
@jakirkham jakirkham deleted the rewrite_fromfunction_blockwise branch June 3, 2021 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants