Skip to content

Commit e8e9629

Browse files
committed
WIP H1: stiffness 1.0->50.0 / damping 0.1->5.0 diagnostic
Diagnostic probe to discriminate two hypotheses for the Newton ~10x reward gap on Shadow-Hand-Over MAPPO at the develop pin: - If reward jumps sharply (e.g. mean from 27 -> 100+), the actuator wiring works and the baseline gains were just too low; sweep for sweet spot. - If reward stays ~27, the wiring is broken (e.g. Newton issue #2585 imports MjcActuators as CTRL_DIRECT, fixed in PR #2598 which is 51 commits past our pin a27277ed). Switch to IdealPDActuatorCfg or bump Newton. Will revert/replace this commit once the diagnostic is interpreted.
1 parent ccac624 commit e8e9629

1 file changed

Lines changed: 12 additions & 8 deletions

File tree

source/isaaclab_tasks/isaaclab_tasks/direct/shadow_hand_over/shadow_hand_over_env_cfg.py

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -180,17 +180,21 @@ def _newton_shadow_hand_cfg(
180180
"robot0_THJ(2|1)": 0.99,
181181
"robot0_THJ0": 0.81,
182182
},
183+
# H1 sweep: stiffness 1.0 → 50.0 / damping 0.1 → 5.0 to test whether
184+
# the actuator wiring is broken (Newton issue #2585) or just under-tuned.
185+
# If reward jumps sharply, wiring works and base gains were just too low.
186+
# If reward stays at ~27 (the baseline), wiring is broken — need PR #2598.
183187
stiffness={
184-
"robot0_WRJ.*": 5.0,
185-
"robot0_(FF|MF|RF|LF|TH)J(3|2|1)": 1.0,
186-
"robot0_(LF|TH)J4": 1.0,
187-
"robot0_THJ0": 1.0,
188+
"robot0_WRJ.*": 50.0,
189+
"robot0_(FF|MF|RF|LF|TH)J(3|2|1)": 50.0,
190+
"robot0_(LF|TH)J4": 50.0,
191+
"robot0_THJ0": 50.0,
188192
},
189193
damping={
190-
"robot0_WRJ.*": 0.5,
191-
"robot0_(FF|MF|RF|LF|TH)J(3|2|1)": 0.1,
192-
"robot0_(LF|TH)J4": 0.1,
193-
"robot0_THJ0": 0.1,
194+
"robot0_WRJ.*": 5.0,
195+
"robot0_(FF|MF|RF|LF|TH)J(3|2|1)": 5.0,
196+
"robot0_(LF|TH)J4": 5.0,
197+
"robot0_THJ0": 5.0,
194198
},
195199
friction=1e-2,
196200
armature=2e-3,

0 commit comments

Comments
 (0)