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

[WIP] added protect_dataset_variables_inplace to open_zarr #2813

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rabernat
Copy link
Contributor

This adds the same call to _protect_dataset_variables_inplace to open_zarr which we find in open_dataset. It wraps the arrays with indexing.MemoryCachedArray.

As far as I can tell, it does not work, in the sense that nothing is cached.

@rabernat rabernat changed the title added protect_dataset_variables_inplace to open_zarr [WIP] added protect_dataset_variables_inplace to open_zarr Mar 14, 2019
@dcherian dcherian added topic-backends topic-zarr Related to zarr storage library labels Aug 14, 2020
@kmuehlbauer
Copy link
Contributor

@rabernat Given the age of this, is this still in the works?

@rabernat
Copy link
Contributor Author

Same answer as on #2814. Not quite sure whether there is interest in this feature.

@kmuehlbauer
Copy link
Contributor

Thanks @rabernat for the quick reply. I was skimming old issues/pr to see if there are some which might be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic-backends topic-zarr Related to zarr storage library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

expose zarr caching from xarray
3 participants