Skip to content

Ignore this PR, folded branch#285

Closed
aidang3019 wants to merge 1 commit intographite-base/285from
aidan/bounds-check
Closed

Ignore this PR, folded branch#285
aidang3019 wants to merge 1 commit intographite-base/285from
aidan/bounds-check

Conversation

@aidang3019
Copy link
Copy Markdown
Contributor

No description provided.

@aidang3019 aidang3019 marked this pull request as ready for review March 13, 2026 16:07
Copy link
Copy Markdown
Contributor Author

aidang3019 commented Mar 13, 2026

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

Comment thread egomimic/rldb/zarr/zarr_dataset_multi.py Outdated
Comment thread egomimic/rldb/zarr/zarr_dataset_multi.py Outdated
Comment thread egomimic/rldb/zarr/zarr_dataset_multi.py Outdated
Comment thread egomimic/trainHydra.py Outdated
),
)

# Propagate norm stats to all zarrdatasets out of bounds check in getitem
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is there a reason we can't just directly set dataset.norm_stats = data_schematic.norm_stats. I wish we didn't have to add norm stats as an attribute after creating the dataset, but I'm not sure if there's a clean pattern to avoid this

Copy link
Copy Markdown
Collaborator

@SimarKareer SimarKareer left a comment

Choose a reason for hiding this comment

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

see comments

@RyanPCo RyanPCo force-pushed the aniketh/bad-idx-fallback branch from 525dd6d to a099aad Compare March 20, 2026 14:40
@RyanPCo RyanPCo force-pushed the aidan/bounds-check branch from 49ef806 to b54c305 Compare March 20, 2026 14:40
This was referenced Mar 20, 2026
@RyanPCo RyanPCo force-pushed the aidan/bounds-check branch from b54c305 to 0b6765d Compare March 20, 2026 18:31
@RyanPCo RyanPCo force-pushed the aniketh/bad-idx-fallback branch from a099aad to 38d6b12 Compare March 20, 2026 18:31
@AnikethCheluva AnikethCheluva force-pushed the aniketh/bad-idx-fallback branch from 38d6b12 to 774c9b3 Compare March 23, 2026 19:52
@AnikethCheluva AnikethCheluva mentioned this pull request Mar 23, 2026
@AnikethCheluva AnikethCheluva force-pushed the aidan/bounds-check branch 2 times, most recently from a73267a to b34056d Compare March 23, 2026 20:17
@AnikethCheluva AnikethCheluva force-pushed the aniketh/bad-idx-fallback branch from 774c9b3 to 3b6cc3c Compare March 23, 2026 20:17
@SimarKareer SimarKareer force-pushed the aniketh/bad-idx-fallback branch from 3b6cc3c to a4a7300 Compare March 24, 2026 07:27
@SimarKareer SimarKareer changed the base branch from aniketh/bad-idx-fallback to graphite-base/285 March 24, 2026 07:50
@SimarKareer SimarKareer changed the title norm stats bound check Ignore this PR Mar 24, 2026
@SimarKareer SimarKareer changed the title Ignore this PR Ignore this PR, folded branch Mar 24, 2026
@SimarKareer SimarKareer mentioned this pull request Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants