surfdado
surfdado
I agree that a little popup message would be a huge improvement and save many hours of trial and error, frustration, and self doubt :)
Wow, that's a lot of issues hahaha - but nonetheless this is amazing news, thanks guys!!!
I've now also added a get_values API to get the complete bms_values struct
Changes have been made, and commits updated get_version now uses 3 int pointers BMS_OP_STATE_UNKNOWN is now 0
Btw, reordering Kevin's existing enum is going to be a major PITA for him. This means that users running an older version of the tool will get wrong op-state reported....
Reading fresh settings when connecting would indeed be welcome, just today I accidentally wrote stale app settings to my VESC because I forgot to hit the "App-read" button first
I also use PC+Phone simultaneously but I'd say this could be considered a separate issue, affecting only a minority of users. The most important thing really is for the average...
Note: I have started working on this and will submit a pull request once I have a first working version
See RFQ pull request: https://github.com/vedderb/bldc/pull/622