Skip to content

Public API for zarr.core.buffer #2871

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Open
TomAugspurger opened this issue Feb 27, 2025 · 1 comment · May be fixed by #2876
Open

Public API for zarr.core.buffer #2871

TomAugspurger opened this issue Feb 27, 2025 · 1 comment · May be fixed by #2876
Labels
enhancement New features or improvements

Comments

@TomAugspurger
Copy link
Contributor

TomAugspurger commented Feb 27, 2025 •
edited
Loading

Zarr version

v3

Numcodecs version

na

Python Version

na

Operating System

na

Installation

na

Description

Over in pydata/xarray#10079, I'd like to use zarr.core.buffer.cpu. But after #2621 zarr.core is offically private.

I'd suggest zarr.buffer.{cpu,gpu} as a public API for these modules. Similar-ish to zarr.storage.

Steps to reproduce

na

Additional output

xref #2641, but that covered the ABC. In this case I'll need the concrete implementation (CPU specifically).
@TomAugspurger TomAugspurger added the bug Potential issues with the zarr-python library label Feb 27, 2025
@jhamman
Copy link
Member

jhamman commented Feb 27, 2025

Yes! Let's do this.

@jhamman jhamman added enhancement New features or improvements and removed bug Potential issues with the zarr-python library labels Feb 27, 2025
@TomAugspurger TomAugspurger linked a pull request Feb 28, 2025 that will close this issue
Open
This was referenced Mar 1, 2025
Closed
Open
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement New features or improvements
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Public API for buffer objects TomAugspurger/zarr-python
2 participants
@TomAugspurger @jhamman