HaddingtonDynamics / Dexter

GNU General Public License v3.0
363 stars 84 forks source link

`S 51` does not work as `S LinkLengths`. #100

Closed JamesNewton closed 3 years ago

JamesNewton commented 3 years ago

Need to update the code to allow the number to trigger it as well, like S RunFile will also work as S 30. All the other SetParameter values work because they are set by looking up the number in a table of names, but if you give it a number, it just takes that as if it was the result of that looking.

Just need to add || !strcmp("51",p1) to the else if(!strcmp("LinkLengths",p1)){ line in DexRun.c

Watch out for this in adding ServoSetX support as well, unless that ends up replacing "ServoSet" on Branch XL-430-Support at: https://github.com/HaddingtonDynamics/Dexter/blob/XL-430-Support/Firmware/DexRun.c#L5558