hytech-racing / MCU

https://hytech-racing.github.io/MCU/index.html
GNU General Public License v3.0
0 stars 0 forks source link

Update telem interface to use new CAN library #42

Closed walkermburns closed 5 months ago

walkermburns commented 7 months ago

In preparation for new TCU, the telem interface will need to be updated to use the new CAN library to allow for correct parsing. @CL16gtgh I know you will have a lot to say about this, but if old TCU simply logs the CAN messages and does not parse, this should not affect it (the CAN messages should be interoperable anyway) and will make sure that any minute changes we make to the CAN parsing from now on are captured for all systems on the car (new TCU, dash, and ECU will be using the new lib).

The fact of the matter is that the telem interface is an ideal candidate for this change as there are not many other systems on the car that rely on this library (only TCU, and now dash since we are adding pedal graphics). If old TCU needs any changes to support this, I will personally make sure that they are made.

This being said, this is a low-priority issue until new TCU is up and running and can be completed as time allows, or can wait until after our first tests with the car.