Skip to content

Conversation

@tomwhite
Copy link
Member

@tomwhite tomwhite commented Sep 2, 2024

See #908

@tomwhite
Copy link
Member Author

tomwhite commented Sep 2, 2024

This is passing all tests, but when producing a visualization of the Dask graph I noticed that it reintroduces the problem in #871, so shouldn't be merged. (cc @timothymillar)

@tomwhite
Copy link
Member Author

tomwhite commented Sep 2, 2024

In the meeting @jeromekelleher suggested that we could abstract the map_blocks bit into a function in sgkit that dispatches either to da.map_blocks (for Dask arrays) or xr.apply_ufunc (for everything else, including Cubed). This would keep the Dask path as it is today, while allowing other array libraries to work.

@mergify
Copy link
Contributor

mergify bot commented Sep 10, 2024

This PR has conflicts, @tomwhite please rebase and push updated version 🙏

@mergify mergify bot added the conflict PR conflict label Sep 10, 2024
@tomwhite
Copy link
Member Author

Superseded by #1254

@tomwhite tomwhite closed this Sep 10, 2024
@mergify mergify bot removed the conflict PR conflict label Sep 10, 2024
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

Successfully merging this pull request may close these issues.

1 participant