You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am using the simp BMS option in the VCU. The BMS_ChargeLim value in the web interface seems wrong. Looking in the simpbms.cpp, the chargeCurrentLimit is divided by 1000. The simp_BMS datasheet states that this value is scaled by 10.
The text was updated successfully, but these errors were encountered:
@Tom-evnut
You're likely the most active contributor. :)
Just wondering if you can roll this into an upcoming release to purge it from the issue list.
Can confirm that the scaling is currently incorrect. I've corrected it in my offline build on the bench.
If not, I'll submit a PR whenever I get my build up to date with the list of recent fixes.
I am using the simp BMS option in the VCU. The BMS_ChargeLim value in the web interface seems wrong. Looking in the simpbms.cpp, the chargeCurrentLimit is divided by 1000. The simp_BMS datasheet states that this value is scaled by 10.
The text was updated successfully, but these errors were encountered: