Skip to content

Group.put() #81

Closed
Closed

Description

@alimanfoo

Implement Group.put() for convenience rather than Group.array(..., overwrite=True)?

Activity

  1. modified the milestone: v2.2 on Dec 15, 2016
  2. removed this from the v2.2 milestone on Oct 31, 2017
  3. hsadia538 commented on Oct 8, 2022

    @hsadia538

    Hi @alimanfoo I want to work on this issue. Can you give me some pointers as to where I should start from. Thanks

  4. zeelyha commented on Oct 14, 2022

    @zeelyha

    Hi @joshmoore @alimanfoo Is this issue still open? I am an Outreachy applicant and I would like to work on it.

  5. joshmoore commented on Oct 14, 2022

    @joshmoore
    Member

    As far is I know, no one has started working on it. I don't see any issue with both of you taking a look. Perhaps one could write tests and one the implementation. Looking for those locations where Group.array(, overwrite=True) is used:

    https://github.com/zarr-developers/zarr-python/blob/main/zarr/tests/test_hierarchy.py#L478

  6. zeelyha commented on Oct 14, 2022

    @zeelyha

    @joshmoore Alright, no problem. I'll look in to it.

    @hsadia538 I'm open to collaborating on this task. You can choose to test or implement, whichever you're comfortable with, but I'll start doing some work nonetheless.

  7. hsadia538 commented on Oct 14, 2022

    @hsadia538

    @zeelyha You can take up the task.

  8. zeelyha commented on Oct 14, 2022

    @zeelyha

    @hsadia538 Alright then, no problem.

  9. arkdong commented on Feb 25, 2025

    @arkdong
    Contributor

    @alimanfoo I would like to work on this issue, what I understand is that you want to implement a method where is replace the functionality of Group.array(..., overwrite=True), which is delete all pre-existing data in store at path before creating the array? Since Zarr is V3 now, the test file @joshmoore mentioned is gone. Should I look further in to test_group.py for v3?

  10. joshmoore commented on Feb 25, 2025

    @joshmoore
    Member

    Hi @arkdong. Apologies. There was a significant rewrite recently (https://zarr.dev/blog/zarr-python-3-release/) and so this issue is no longer up-to-date. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew features or improvementsgood-first-issueGood place to get started as a new contributor.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions