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

Using xarray instead of dask array #41

Open
pr4deepr opened this issue Aug 10, 2023 · 2 comments · May be fixed by #47
Open

Using xarray instead of dask array #41

pr4deepr opened this issue Aug 10, 2023 · 2 comments · May be fixed by #47
Labels
enhancement New feature or request

Comments

@pr4deepr
Copy link
Collaborator

Potentially use xarray instead of dask array to store data
xarray can have multiple attributes which could replace the need for setting a lot of properties in Latticedata class.
attributes can hold arbitrary metadata...

https://github.com/BiAPoL/scripts/blob/main/notebooks/xarray_example.ipynb
https://docs.xarray.dev/en/stable/user-guide/data-structures.html

@pr4deepr pr4deepr added the enhancement New feature or request label Aug 10, 2023
@pr4deepr
Copy link
Collaborator Author

It integrates with dask for parallel computations as well:

https://docs.xarray.dev/en/stable/user-guide/dask.html

@multimeric
Copy link
Collaborator

Nice, it would also make the code simpler if we could assume that we're always working with a single array type. If xarray can ingest all of the array types we want to support then this could be helpful.

@multimeric multimeric linked a pull request Sep 18, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants