Skip to content

Zarr writing utils#198

Merged
SimarKareer merged 1 commit intomainfrom
aidan/zarr-writing
Feb 24, 2026
Merged

Zarr writing utils#198
SimarKareer merged 1 commit intomainfrom
aidan/zarr-writing

Conversation

@aidang3019
Copy link
Contributor

@aidang3019 aidang3019 commented Feb 9, 2026

No description provided.

@aidang3019 aidang3019 marked this pull request as ready for review February 9, 2026 21:07
@aidang3019 Graphite App
Copy link
Contributor Author

aidang3019 commented Feb 9, 2026 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@aidang3019 aidang3019 mentioned this pull request Feb 9, 2026
Merged
SimarKareer
SimarKareer reviewed Feb 9, 2026
View reviewed changes
egomimic/scripts/eva_process/zarr_utils.py
Array2D = Union[np.ndarray, torch.Tensor]


def _to_numpy(x):
Copy link
Collaborator

@SimarKareer SimarKareer Feb 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A lot of this file is copying stuff from other files, just import instead of copying

Copy link
Collaborator

@SimarKareer SimarKareer Feb 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete this entire file actually

@aidang3019 aidang3019 mentioned this pull request Feb 10, 2026
Closed
@rl2aloha rl2aloha mentioned this pull request Feb 11, 2026
Open
@ElmoPA ElmoPA force-pushed the aidan/zarr-writing branch from d8e0333 to 1826215 Compare February 11, 2026 05:52
@ElmoPA ElmoPA force-pushed the aidan/zarr-reading branch from 790eac4 to 907e5e1 Compare February 11, 2026 05:52
@ElmoPA ElmoPA force-pushed the aidan/zarr-writing branch from 466906c to d11ada3 Compare February 12, 2026 01:22
@RyanPCo RyanPCo mentioned this pull request Feb 12, 2026
Open
@ElmoPA ElmoPA force-pushed the aidan/zarr-writing branch from ae528e7 to d11ada3 Compare February 12, 2026 05:22
@ElmoPA ElmoPA force-pushed the aidan/zarr-reading branch from 4b4203d to 1847509 Compare February 12, 2026 05:22
@rl2aloha rl2aloha mentioned this pull request Feb 12, 2026
Merged
@ryanthecreator ryanthecreator mentioned this pull request Feb 13, 2026
Merged
@aidang3019 aidang3019 mentioned this pull request Feb 21, 2026
Closed
@AnikethCheluva AnikethCheluva mentioned this pull request Feb 21, 2026
Closed
@RyanPCo RyanPCo force-pushed the aidan/zarr-writing branch 2 times, most recently from 43d95f1 to a5a98f1 Compare February 21, 2026 23:45
@aidang3019 aidang3019 mentioned this pull request Feb 22, 2026
Merged
@SimarKareer SimarKareer force-pushed the aidan/zarr-writing branch 2 times, most recently from 2b61031 to f68233d Compare February 24, 2026 00:54
@AnikethCheluva AnikethCheluva mentioned this pull request Feb 24, 2026
Merged
@SimarKareer SimarKareer mentioned this pull request Feb 24, 2026
Merged
@SimarKareer Graphite App
Copy link
Collaborator

SimarKareer commented Feb 24, 2026 •
edited
Loading

Merge activity

  • Feb 24, 11:20 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Feb 24, 11:23 PM UTC: Graphite rebased this pull request as part of a merge.
  • Feb 24, 11:23 PM UTC: @SimarKareer merged this pull request with Graphite.

@ryanthecreator ryanthecreator mentioned this pull request Feb 25, 2026
Merged
This was referenced Feb 27, 2026
Open
Draft
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SimarKareer SimarKareer SimarKareer left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aidang3019 @SimarKareer