Skip to content

feat(enclosure): flush-at-rim pedal seating (case top on the slot's upper rim) - #376

Open
tomassasovsky wants to merge 1 commit into
feat/led-pills-all-pedals-366from
feat/pedal-flush-373
Open

feat(enclosure): flush-at-rim pedal seating (case top on the slot's upper rim)#376
tomassasovsky wants to merge 1 commit into
feat/led-pills-all-pedals-366from
feat/pedal-flush-373

Conversation

@tomassasovsky

Copy link
Copy Markdown
Owner

Closes #373

Stacked on #368 (which stacks on #361) — merge order: #361#368 → this; retarget after each parent lands. CI note: the test matrix only triggers on PRs into master, so only GitGuardian runs until retargeted.

User call: the Cherub pedals read too low — sunken against the slot's upper (rear) rim, because the sloped plate keeps rising past the slot centre while the old rule fixed the pad +12 over the centre-line.

  • platform_h() re-derived: pedal case top flush with the plate surface at the slot's upper rim, pad above the metal. Uses the new face_drift() doc-calibrated seating model (bare frame was 1.4 mm short at row 1)
  • Pedestals: front 10.9 → 15.2, mid 56.0 → 59.3
  • Perimeter relief on the tubs: the footprint strips overhanging the slot opening are shaved with the skirt's under-plate plane — the raised deck box otherwise poked the plate bands and row-1's front lip fold (caught by doc interference probes)
  • SKIRT_DRIFT_ROW2 0.7 → 0.5 (re-measured at the feat(enclosure): LED indicators for REC/PLAY, STOP, UNDO, MODE (design trial) #366 position)
  • Trade-off to be aware of: side-screw boss clearance under the faceplate drops to ~1.0 mm (row 1) / ~0.95 mm (row 2) real — static parts, no relative motion, but tighter than the old ~4 mm; assert now checks the drift-corrected value ≥ 0.8
  • Front pedestals now take the standard M3 5×5 inserts (full 6.0 pilot depth) — short M3×3 obsolete, insert BOM unified
  • Mini console inherits via platform_h and stays a faithful metal-order fit test

Verified in "VAMP console (populated)": flush within 0.15 mm on both rows, zero interference beyond the known-benign internal pedal slivers; VSM doc pedestals replaced too. Generator assertions ALL PASS.

… upper rim

Issue #373 (user call: pedals read too low, sunken against the rear lip):
- platform_h() re-derived: pedal BODY TOP lands flush with the faceplate
  surface at the slot's UPPER (rear) rim; only the pad stands above the
  metal (+2.2 at the rim, ~15 over the centre-line). Includes the new
  face_drift() doc-calibrated seating model -- without it flush was still
  1.4 mm short at row 1 (probed in the populated doc)
- pedestals grow: front 10.9 -> 15.2, mid 56.0 -> 59.3
- PERIMETER RELIEF: tub strips overhanging the slot opening (1.25 front/
  rear, ~5.2 per side) are shaved with the skirt's under-plate plane --
  the raised deck box would otherwise poke the plate bands and the row-1
  front lip fold (found via doc interference probes, 8x153 + 2x124 mm3)
- SKIRT_DRIFT_ROW2 0.7 -> 0.5 (re-measured at the #366 rearward position;
  0.3 mm to the real plate at the tub's rear band)
- screw-boss clearance assert now uses face_drift (real ~1.0 row 1 /
  ~0.95 row 2, threshold 0.8) -- the price of flush seating, static parts
- front pedestals now take the standard M3 5x5 inserts (deck 13.2 gives
  full 6.0 pilot depth) -- short M3x3 obsolete, BOM unified
- mini console inherits via platform_h (stays a faithful fit test)

Both Fusion docs updated (pedestals v373c/v373 replaced in populated +
VSM, pedals re-seated) and verified: flush within 0.15 mm on both rows,
ZERO interference above the known-benign pedal slivers. Assertions PASS.
@tomassasovsky tomassasovsky added stage:in-review Pipeline: PR open, CI + code-review iterating autonomy:merge-gate I build + go green; you click merge review:pending code-review skill not yet run / has findings labels Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autonomy:merge-gate I build + go green; you click merge review:pending code-review skill not yet run / has findings stage:in-review Pipeline: PR open, CI + code-review iterating

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant