added/fixed thruster current collection

osu-uwrt / maelstrom_firmware

Everything electrical and non-hardware

1 stars 13 forks source link

added/fixed thruster current collection #59

Closed gworkman closed 5 years ago

gworkman commented 5 years ago

Addresses #58. It currently is not completely functional yet - the current value (a float) needs to be encoded as an integer or byte for sending data back through the socket.

To do so, I'm planning on multiplying by 10 or 100 and truncating to an integer. @blaine141 what is the maximum current consumption value that it is expected/should be able to read? This will determine how much precision is necessary and how many bytes the number will take up

blaine141 commented 5 years ago

Each thruster has a 10A max. 0A to 10A

gworkman commented 5 years ago

Going to multiply by a factor of 25 then - 10*25 = 250, keeps us below the max value of 255 for 1 byte. On the receiving end, can simply pull that out to get the most precision possible