Skip to content

Add option for custom filtering based on json#621

Merged
kpertsch merged 12 commits intoPhysical-Intelligence:mainfrom
verityw:main
Aug 31, 2025
Merged

Add option for custom filtering based on json#621
kpertsch merged 12 commits intoPhysical-Intelligence:mainfrom
verityw:main

Conversation

@verityw
Copy link
Contributor

@verityw verityw commented Aug 27, 2025

The current training code only uses the default filter. This PR adds the option to filter based off of JSONs that map episodes to ranges of timesteps to keep.

  • Changed config.py to also have the option to specify where that json filter file is.
  • Changed data_loader.py to make use of above.
  • Changed DroidRldsDataset to load that json, then filter out all frames not contained within that file.
  • Added examples/droid/get_droid_keep_ranges.py, which uses a more aggressive filter (created by Karl) to generate such a json.
  • Updated README_train.md to reflect these changes. Added reference to DROID hf repo, which contains a copy of that json run on droid/1.0.1.

@jimmyt857 jimmyt857 removed their request for review August 28, 2025 16:10
@kpertsch kpertsch enabled auto-merge August 31, 2025 20:00
@kpertsch kpertsch merged commit 255fe9b into Physical-Intelligence:main Aug 31, 2025
2 checks passed
ChenZhenY pushed a commit to ChenZhenY/openpi that referenced this pull request Sep 23, 2025
…l-Intelligence#621)

The current training code only uses the default filter. This PR adds the
option to filter based off of JSONs that map episodes to ranges of
timesteps to keep.

- Changed `config.py` to also have the option to specify where that
`json` filter file is.
- Changed `data_loader.py` to make use of above.
- Changed `DroidRldsDataset` to load that `json`, then filter out all
frames not contained within that file.
- Added `examples/droid/get_droid_keep_ranges.py`, which uses a more
aggressive filter (created by Karl) to generate such a json.
- Updated `README_train.md` to reflect these changes. Added reference to
DROID hf repo, which contains a copy of that json run on `droid/1.0.1`.
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.

2 participants