HaddingtonDynamics / Dexter

GNU General Public License v3.0
374 stars 85 forks source link

Instruction number in status #18

Closed JamesNewton closed 5 years ago

JamesNewton commented 6 years ago

The firmware should return the current (or previous) instruction id number in the status data. This will help coordinate between DDE and Dexter. Scripts need to take actions when the robot has actually completed (or is working on) a previously send instruction.

The only trick is figuring out which item in the status data to replace with the instruction id. Documenting the actual status data #17 will help to pick one to replace.

JamesNewton commented 5 years ago

It already does... and always has... at least since we started using github: https://github.com/HaddingtonDynamics/Dexter/blame/add3e2e36c891be3e4853c9b1daac0d76035f576/Firmware/DexRun.c#L2776

I've just retested and verified via the onboard http proxy server /srv/samba/share/www/. The instruction numbers are absolutely being echoed.

JamesNewton commented 3 years ago

Kamino cloned this issue to HaddingtonDynamics/OCADO