Checklist for LeRobot dataloader #8949

teh-cmc · 2025-02-06T10:54:04Z

It's a bit of an odd one because of the way the dataloader works, but I would still feel much better with an odd test than no tests at all.

I've uploaded a heavily sampled down apple_storage dataset to LFS in order to make this work.

github-actions · 2025-02-06T10:54:42Z

Web viewer built successfully. If applicable, you should also test it:

Result	Commit	Link	Manifest
✅	`dcbf2df`	https://rerun.io/viewer/pr/8949	`+nightly` `+main`

^{Note: This comment is updated whenever you push a commit.}

teh-cmc · 2025-02-10T09:03:23Z

I'll sample even more aggressively (not sure why I kept 10 rather than.. I dunno, 3 maybe?) then merge this.

teh-cmc force-pushed the cmc/lerobot_checklist branch 2 times, most recently from 5e9b389 to 5639d32 Compare February 6, 2025 11:19

teh-cmc added 🔨 testing testing and benchmarks exclude from changelog PRs with this won't show up in CHANGELOG.md labels Feb 6, 2025

teh-cmc force-pushed the cmc/lerobot_checklist branch from 5639d32 to 840031a Compare February 6, 2025 11:20

teh-cmc added do-not-merge Do not merge this PR and removed do-not-merge Do not merge this PR labels Feb 6, 2025

teh-cmc marked this pull request as ready for review February 6, 2025 11:27

Base automatically changed from cmc/lfs_assets to main February 7, 2025 08:42

teh-cmc force-pushed the cmc/lerobot_checklist branch from 840031a to 0e97711 Compare February 7, 2025 08:43

teh-cmc removed the do-not-merge Do not merge this PR label Feb 7, 2025

teh-cmc added 3 commits February 10, 2025 10:24

add heavily sampled LeRobot's apple_storage dataset

9201afd

check_lerobot_dataloader

d3fc9c6

lints

1817306

teh-cmc force-pushed the cmc/lerobot_checklist branch from 0e97711 to 1817306 Compare February 10, 2025 09:32

format

dcbf2df

teh-cmc merged commit 87564f7 into main Feb 10, 2025
34 of 35 checks passed

teh-cmc deleted the cmc/lerobot_checklist branch February 10, 2025 09:47

Provide feedback