shining-man / bsc_fw

Firmware battery safety controller (BSC)
MIT License
96 stars 18 forks source link

BMS Temperatur Sensor #143

Closed a-wolter closed 1 month ago

a-wolter commented 1 month ago

Laut WebUI kann ich bei Auswahl "BMS" nur 0-2 als Temp. Sensor festlegen. Ich konnte problemlos "4" eintragen, der Wert welcher an VenusOS gesendet wird ändert sich allerdings nicht. Zudem ist nicht ganz klar, welcher Sensor aus welchem Pack nun übertragen wird. Ich vermute es ist ein Wert aus dem Pack mit der niedrigsten ID, da die Temperatur passen würde.

Mit z.B. 5 JK Inverter BMS im Daisy-Chain an Serial 2 sollten min. auch 5 Sensoren zur Verfügung stehen. Evtl. sogar mehr, da über die JK App pro Pack 4 Sensoren (5 mit MOS Temp) angezeigt werden

grafik

Der Wert für die min cell temp ist falsch grafik

Getestet mit JK Inverter BMS an Serial2 im Daisy-Chain. FW 15.17

shining-man commented 1 month ago

Werte kannst du von 0-63 eintragen. Aber für BMS ist nur 0-2 gültig. Dies hat den Grund, dass in der internen Datenstruktur des BSC platz für 3 Temperaturen ist. Wenn ein BMS mehr als drei Temperaturen liefert, dann nehme ich immer die ersten drei. Aus diesen drei Temperaturen wird dann auch die Min./Max.-Temp genommen. Ich vermute, dass eine der Temperaturen 0 ist und daher ist die Min.-Temp auch 0. Das wäre von daher erst einmal richtig.

Man muss jetzt schauen, welche drei Temperaturen sinn ergeben.

a-wolter commented 1 month ago

Das kann ich bestätigen grafik

a-wolter commented 1 month ago

Hier der Vergleich von JK App zu /restapi

IMG_4203 IMG_4202

a-wolter commented 1 month ago

Funktioniert ab der 0.6.1_T7