Repository navigation
Conversation
…conveyor-foundations
…eyor-racetrack-transfer
…nk/conveyor-warehouse-sorting
4 tasks done
Integrate upstream main e907f82 to keep this branch current while preserving its feature scope.
Use the clone configuration owner for existing conveyor namespace defaults. This preserves the public values while satisfying the current clone-ownership check across the integrated task stack. Recorded upstream main: e907f82
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds
IsaacContrib-Conveyor-Warehouse-Sorting-v0: a 24-parcel pool uses four policy slots and class metadata to route four colors onto two circulating conveyors. Reuses the racetrack observations/actions and an authored USD warehouse.A standard hidden ground plane replaces the custom terrain importer. Physics stays in the policy workspace; presentation roots and camera share one translation. This also fixes a persistent reset bug that applied the warehouse height twice to the robot. Asset spawning uses stock mesh/cuboid APIs, visual physics stripping has one pass, and arrivals use batched goal assignment.
NVIDIA geometry and textures remain remote references under their original licenses. Depends on #8331/#8327. Review warehouse changes. PhysX is independent and optional (#8794).
Validation and limitation
Warehouse policy success is not established. With
model_1100.pt, seed 41, two environments and 2,400 steps, both the historical scene with corrected robot height and the simplified scene completed zero transfers, with zero failures/non-finite observations. Arrivals and targets were active. This PR remains a draft; these checks establish scene/dispatch contracts, not successful batch sorting.Checklist
CONTRIBUTORS.md.