-
Notifications
You must be signed in to change notification settings - Fork 32
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
Run on Cubed #908
Comments
I've managed to get some basic aggregation tests in They rely on some changes in xarray too: pydata/xarray#7067. |
Also, this example shows that Cubed works with Numba (locally at least), which answers @hammer's question here: https://github.com/pystatgen/sgkit/issues/885#issuecomment-1209288596. |
Since I opened this issue almost two years ago, Xarray has added a chunk manager abstraction (https://docs.xarray.dev/en/stable/internals/chunked-arrays.html), which makes it a lot easier to switch from Dask to Cubed as the backend computation engine without changing the code to express the computation. The nice thing about this approach is that we can use Dask or Cubed or any other distributed array engine that Xarray might support in the future (such as Arkouda). I've started to explore what this might look like in https://github.com/tomwhite/sgkit/tree/xarray-apply-ufunc, but the two main ideas are:
The code in the branch does this for This commit (da8657e) shows the new pytest command-line option to run on cubed: I would be interested in any thoughts on this direction @jeromekelleher, @hammer, @timothymillar, @benjeffery, @ravwojdyla, @eric-czech. I'd like to set up a CI workflow that adds |
Here's a successful run for the |
This sounds like an excellent approach +1 |
This is an umbrella issue to track the work needed to run sgkit on Cubed.
This is possible because Cubed exposes the Python array API standard as well as common Dask functions and methods like
map_blocks
andArray.compute
. Also, there is ongoing work to Integrate cubed in xarray, as a part of exploring alternative parallel execution frameworks in xarray.The text was updated successfully, but these errors were encountered: