protocol icon indicating copy to clipboard operation
protocol copied to clipboard

Incorrect value for "move complete command" in AccelStepper?

Open pavhofman opened this issue 5 years ago • 3 comments

The "move complete command" in AccelStepper lists value 0x0b in protocol/accelStepperFirmata.md . However, the implementations use 0x0a:

https://github.com/firmata/firmata.js/blob/488964da379b137da5ab3e7887161cbf31c60793/packages/firmata-io/lib/firmata.js#L400

https://github.com/firmata/ConfigurableFirmata/blob/master/src/AccelStepperFirmata.h#L39

Since the arduino firmware uses 0x0a, please can you fix the documentation?

Thanks a lot.

pavhofman avatar Nov 25 '19 21:11 pavhofman

You are absolutely correct. I must have removed a command at some point during the implementation and forgot to update that value. I'll get a PR in ASAP.

dtex avatar Nov 25 '19 21:11 dtex

Thanks a lot for instant response :-)

pavhofman avatar Nov 25 '19 21:11 pavhofman

Thank you for the catch!

It's all updated now.

dtex avatar Nov 25 '19 22:11 dtex