The deserialize functions as well as readShortFromCommBuffer (btw I'd rename this to deserialCommBufferShort), use a for loop to parse the bytes in the byte_array commBuffer in a big-endian manner when in reality, all values from the ODrive usb protocol are sent using in a little-endian format. Because the for loop is reversed, the boost endian reversal is needed at the end of the function. If we fix the for loop, then we would no longer need the boost library which would be a huge plus!
The deserialize functions as well as readShortFromCommBuffer (btw I'd rename this to deserialCommBufferShort), use a for loop to parse the bytes in the
byte_arraycommBufferin a big-endian manner when in reality, all values from the ODrive usb protocol are sent using in a little-endian format. Because the for loop is reversed, the boost endian reversal is needed at the end of the function. If we fix the for loop, then we would no longer need the boost library which would be a huge plus!