Louisvdw / dbus-serialbattery

Battery Monitor driver for serial battery in VenusOS GX systems
MIT License
541 stars 165 forks source link

Exception in battery.py line #937 #1084

Closed chaerle closed 5 months ago

chaerle commented 5 months ago

Describe the bug

After some time running, the drivers starts logging the following error and becomes instable, so that also AggregateBatteries does not work any more with the instance.

Log:

@40000000666dcfd211538df4 ERROR:SerialBattery:Exception occurred: TypeError("'<=' not supported between instances of 'NoneType' and 'float'") of type <class 'TypeError'> in /opt/victronenergy/dbus-serialbattery/battery.py line #937
@40000000666dcfd2116c6554 WARNING:SerialBattery:Error while executing calcMaxDischargeCurrentReferringToCellVoltage(). Using default value instead.
@40000000666dcfd21179c71c WARNING:SerialBattery:CELL_VOLTAGES_WHILE_DISCHARGING: [2.7, 2.8, 2.9, 3.1] • MAX_DISCHARGE_CURRENT_CV: [0.0, 6.0, 30.0, 60.0]

How to reproduce

Unknown, just wait

Expected behavior

no error

Driver version

v1.2.20240408

Venus OS device type

Raspberry Pi 3

Venus OS version

v3.31

BMS type

JKBMS / Heltec BMS

Cell count

16

Battery count

2

Connection type

Serial USB adapter to TTL

Config file

empty (default values)

Relevant log output

@40000000666dcfd211538df4 ERROR:SerialBattery:Exception occurred: TypeError("'<=' not supported between instances of 'NoneType' and 'float'") of type <class 'TypeError'> in /opt/victronenergy/dbus-serialbattery/battery.py line #937
@40000000666dcfd2116c6554 WARNING:SerialBattery:Error while executing calcMaxDischargeCurrentReferringToCellVoltage(). Using default value instead.
@40000000666dcfd21179c71c WARNING:SerialBattery:CELL_VOLTAGES_WHILE_DISCHARGING: [2.7, 2.8, 2.9, 3.1] • MAX_DISCHARGE_CURRENT_CV: [0.0, 6.0, 30.0, 60.0]

Any other information that may be helpful

No response

mr-manuel commented 5 months ago

Did you read the message when opening an issue?

grafik

chaerle commented 5 months ago

Does not happen with v1.3.20240606dev any more.