Skip to content

Add optional CPU PhysX support for conveyor racetrack transfer - #8794

Draft
maxkra15 wants to merge 71 commits into
isaac-sim:mainfrom
maxkra15:maximiliank/conveyor-physx-cpu
Draft

maxkra15 wants to merge 71 commits into
isaac-sim:mainfrom
maxkra15:maximiliank/conveyor-physx-cpu

Conversation

@maxkra15

@maxkra15 maxkra15 commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Description

Adds optional IsaacContrib-Conveyor-Racetrack-Transfer-PhysX-CPU-v0 using native PhysxSurfaceVelocityAPI. CPU-only validation rejects GPU dynamics, which can lose belt contacts in the supported runtime.

Owns PhysX configuration, registration and dependencies while sharing the racetrack task logic. Uses the standard in-memory mesh spawner, retaining the existing curved-belt SDF settings, fixed speeds, startup ramp and stationary pre-parsing authoring. No custom mesh prerequisite.

Depends on #8331/#8327; independent of warehouse sorting. Review the PhysX addition. Review last. #6952 records combined validation.

Validation

  • Five focused adapter/configuration cases passed.
  • Isaac Sim 6.2 native probe: two CPU environments, 135 steps, 16 surfaces, finite observations/rewards and no fall-through.
  • Partial/full/hard resets and repeated close passed. SDF collision settings remain unchanged.
  • Full formatting/lint/changelog/LFS, browser and lock checks passed.

Matching policy shapes do not establish numerical parity between backends.

Checklist

  • Followed contribution and test-audit guidance.
  • Ran focused tests and repository formatting/lint checks.
  • Updated documentation and package changelog.
  • Contributor already listed in CONTRIBUTORS.md.

Keep the custom mesh branch current with develop at 9b6d2d8.\n\nThe unchanged feature diff passed the mesh spawner tests and the\nrepository-wide validation gate before this merge was committed.
Integrate the recorded main revision while preserving the PR scope.

Validate the merged tree with focused tests and full repository hooks.
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

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation isaac-lab Related to Isaac Lab team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant