[python] Add Paimon LeRobot map-style dataset - #9498
Conversation
JingsongLi
left a comment
There was a problem hiding this comment.
Two remaining issues after re-checking the updated head.
|
I think the imported dataset should be self-contained in Paimon, but separate typed tables would be clearer than either duplicating metadata in every frame or storing all metadata in a generic A possible logical dataset bundle is:
All component tables should use the same With this contract, The current PR leaves the source of truth split between Paimon frames and an external metadata directory. I suggest defining and implementing the self-contained multi-table import contract first, then keeping |
|
To clarify the business meaning of the three proposed metadata tables, they represent three different levels of the dataset rather than merely splitting one metadata blob by file type. 1.
|
f80e25b to
ac34c27
Compare
ac34c27 to
08ce1c8
Compare
Purpose
Add a map-style
PaimonLeRobotDatasetfor training from imported LeRobot v3 image datasets.Changes
tag_namefor reproducible reads, or the latest component snapshots when omitted.dataset.meta, including native task and subtask text indices.__getitems__and multi-workerDataLoaderreads without materializing frame payloads.torch.uint8image output.Scope
This version supports image-backed LeRobot v3 datasets imported by
load_from_lerobot(). Video-backed reads remain a follow-up to #9494.Tests
63 passed, 53 subtests passedinpypaimon/tests/multimodal_lerobot_test.py.57 passed, 20 subtests passedinpypaimon/tests/torch_read_test.py.py_compile, andgit diff --checkpass.