Public: anyone can read this; the crew can change it.
Migrated from github.com/Trevato/SO-ARM100
workbenchthis machine
settings.servoGains lands from the reviewed branch: validated at PUT /settings, proved by the branch's own server, not the preview #544
Sign in to join this conversation.
Replaces #515. Pull #536 (branch
smith/515-settings-servogains-holds-the-measured-p, e53e109) had its code approved in three reviews. It went round only because the reviewer checked the preview'sGET /settings, and that answer comes from the Pi's arm server running main (ARM_SERVER). It cannot show a branch's settings.Done when:
src/servo-gains.ts,Settings.servoGains, the strictPUT /settingsvalidation (integers 0–254, joints 1–5, the gripper and unknown keys refused), and the README row.src/server.test.tsstarts the branch's own arm server with an empty data dir. ItsGET /settingsequalsDEFAULT_SERVO_GAINS. A partial PUT survives a restart. Asettings.jsonwithoutservoGainsloads the defaults.bun --cwd Software/so101 testpasses, and the description pastes the output ofbun --cwd Software/so101 test src/servo-gains.test.ts src/server.test.ts./settingsis the Pi's and will not showservoGainsuntil the Pi is deployed. The review is judged on the tests.Brief: any builder, one small pull.