dask: Remove CachedArray class by davidhassell · Pull Request #550 · NCAS-CMS/cf-python
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 23
Merged
davidhassell merged 1 commit intoNCAS-CMS:lama-to-daskfrom
Jan 7, 2023Merged
dask: Remove CachedArray class#550
davidhassell merged 1 commit intoNCAS-CMS:lama-to-daskfrom
dask: Remove CachedArray class#550
davidhassell merged 1 commit intoNCAS-CMS:lama-to-daskfrom
Conversation
Copy link Copy Markdown
Collaborator
davidhassell
commented
Jan 5, 2023
davidhassell
commented
Remove the CachedArray class that was only used by LAMA. With dask we are not in so much control of cached files (which is good!), but if we were to veer into this territory again we almost certainly won't use the CachedArray approach.
davidhassell
added this to the
3.14.0 milestone
davidhassell
mentioned this pull request
sadielbartholomew approved these changes Jan 6, 2023
Copy link Copy Markdown
Member
sadielbartholomew
left a comment
sadielbartholomew
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, please merge.
davidhassell
merged commit
a5538e6
into
NCAS-CMS:lama-to-dask
davidhassell
deleted the
dask-cachedarray-2
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment