Skip to content

chore(assets): keep only the objects the scenarios load - #10

Merged
ZJLi2013 merged 1 commit into
mainfrom
chore/prune-assets
Oct 20, 2026
Merged

ZJLi2013 merged 1 commit into
mainfrom
chore/prune-assets

Conversation

@ZJLi2013

Copy link
Copy Markdown
Owner

Keeps only the objects that the four scenarios load, plus the scene base assets:

  • Kept: apple_01, die_01, lego_02, mug_02, plane, table_simple; assets/articraft/ unchanged.
  • Removed: apple_02, block_blue, carton_01, carton_02, fruit_03, and their catalog.json entries.
  • Tests that seeded the library with block_blue now use die_01; the name-derived grasp category test uses apple_01.
  • Default asset lists in validate_asset_picks.py and run_metadata_guided_rollout.py now name only assets that exist.

Asset management gets its own rework later; this keeps the library at the minimum the demos need.

Test: pytest tests — 135 passed, 1 skipped.

AI use

  • Tool/model: Cursor agent (Claude)
  • Extent: finding which assets the scenarios reference, the deletions and test/script edits, PR text
  • Human verification: pending review by the contributor

Co-authored-by: Cursor <cursoragent@cursor.com>
@ZJLi2013
ZJLi2013 merged commit dcd65e9 into main Oct 20, 2026
4 checks passed
@ZJLi2013
ZJLi2013 deleted the chore/prune-assets branch October 22, 2026 02:16
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.

1 participant