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.

Sign up for GitHub

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

Jump to bottom

GroupMetaData has no info property #1934

Closed
dstansby opened this issue May 31, 2024 · 0 comments · Fixed by #1936
Closed

GroupMetaData has no info property #1934

dstansby opened this issue May 31, 2024 · 0 comments · Fixed by #1936

Comments

@dstansby
Copy link
Contributor

AsyncGroup tries to access the .metadata.info property:

zarr-python/src/zarr/group.py

Lines 277 to 279 in 9ad01f1

@property
def info(self):
return self.metadata.info

but GroupMetaData doesn't hava a .info property...

zarr-python/src/zarr/group.py

Line 76 in 9ad01f1

class GroupMetadata(Metadata):

This is one of the last typing errors that needs fixing, but it also looks like a bug too? Does anyone know if there are plans to implement GroupMetaData.info, or should AsyncGroup.info be removed?
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(types): Group.info -> NotImplementedError jhamman/zarr-python
1 participant
@dstansby