Repository navigation
Project completed hydroelastic contacts onto tactile pads - #1207
Open
han-xudong wants to merge 1 commit into
Open
han-xudong wants to merge 1 commit into
han-xudong wants to merge 1 commit into
Conversation
4 tasks done
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 opt-in
TactileIndentationCfg.surface. It projects the final external Newton collision pass of a completed step into the existing tactile probes. The grid reports face count and minimum signed, margin-relative separation. Per-arena fields expose collision generation, substep and capacity completeness.The sensor resolves physical pad and opposing actors through
SensorContext. It uses the collision-input pose for a moving pad, topology-scoped shape IDs, and owned SDK tensors. Regular flat and curved grids bin centroids in pad XY and check normal clearance. Explicit layouts use the nearest calibrated point inside the configured distance. Existing RGB, geometric depth and elastic shear keep their current inputs.Refs #1174. Requires a DexSim SDK with the completed surface bindings from DexSim !1614 and an external collision pipeline with
output_contact_surface=True. The option is disabled by default. A Scene with a missing or truncated snapshot reports an incomplete surface.Stack
codex/sensors-4g-shear.Type of change
Review focus
Check the collision-input frame, per-arena shape pairing, model rebuild and output invalidation. A completed empty pass has
surface_complete=True. A raw count exceeding capacity marks all arenas incomplete while retaining available bins.Validation
0.6.0rc2native libraries and NoRender. The installed release does not yet expose the new API, so native tests require the candidate.Full-step performance
RTX 5070, DexSim candidate on rc2 native libraries, Warp 1.17, Newton MuJoCo Warp, NoRender, single substep, CUDA Graph disabled. Every variant runs RGB and shear on 16 × 16 probes. The benchmark times pose and velocity writes, physics, sensors and GPU completion. Tables average the two run-level percentiles from forward and reverse order, with 25 warmup and 100 timed steps per variant. Depth, hit, shape IDs and RGB match across variants.
The 32-arena case exported 99,424 faces in a 524,288-face buffer and binned 90,464 centroids. Torch transient memory includes the independent snapshot copy. Run
python -m scripts.benchmark.sensors.hydroelastic_step --envs 1 8 32 --warmup 25 --repeats 100on the candidate SDK.Checklist