With "temp" off, unable to start

[nicobresseleers]

I ran into issue/bug mr-manuel/venus-os_dbus-serialbattery#145

to resolve it , i temporary disabled:
CCCM_T_ENABLE = True -> False
DCCM_T_ENABLE = True -> False

Now aggregate will not start:

INFO:root:Sun Dec 29 21:16:01 2024: MultiPlus-II 48/3000/35-32 found.
ERROR:root:Sun Dec 29 21:16:02 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read temperatures, Battery SerialBattery(JKBMS)C2.
ERROR:root:Read trial nr. 1
.....
ERROR:root:Sun Dec 29 21:16:12 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read temperatures, Battery SerialBattery(JKBMS)C2.
ERROR:root:Read trial nr. 11
ERROR:root:Sun Dec 29 21:16:12 2024: DBus read failed. Exiting.

Can you please check?

n

[nicobresseleers]

PS:
the temp values are still there:

Seems you are reading in the wrong place or something?

[Dr-Gigavolt]

Hi Nico,
I'm sorry, I have no time to follow the issues of Serial Battery. Does my program raise an exception when the temperature is negative? According the log it receives 'None' value. Please make a screenshot of dbus-spy of the temperatures. Perhaps there is some format problem.

[nicobresseleers]

Issue with Serial Battery has been identified and resolved.. ( nothing to do with negative temp .. it's when min voltage is not set), many people complained about unidentified "low battery" errors... seems we found that one.( and is also reported to victron)

Aggregates works, till i change these 2 parameters in serial: ( as a workaround to ignore a negative value -> Serial works)

CCCM_T_ENABLE = True -> False
DCCM_T_ENABLE = True -> False

and/or

read 1 temperature from the batteries ( to ignore the negative)

So only 1 measurement for temperature (14) is available as Temp 1

and

once i start aggregate now it stops, with "NONE", seems it expects more.. quite sure if i disconnect the temp sensors ( or turn of the temperature on the BMS via the IOS app , it will also fail to start.

Can you replicate the issue?

Nico

[nicobresseleers]

UPDATE:

Could replicate myself:

I can confirm it's only the reading 1 temperature that makes aggregate stop:

; Battery temperature
; Specify how the battery temperature is assembled.
; 0 Get mean of temperature sensor 1 to sensor 4
; 1 Get only temperature from temperature sensor 1
; 2 Get only temperature from temperature sensor 2
; 3 Get only temperature from temperature sensor 3
; 4 Get only temperature from temperature sensor 4
TEMP_BATTERY = 0

when TEMP_BATTERY = 0 ( means all) , aggregates runs again even with parameters

CCCM_T_ENABLE = True -> False
DCCM_T_ENABLE = True -> False

set to
TEMP_BATTERY = 1

and aggregates stops since it's missing a temp value ... and honestly with LTO, i don't care about temp :-)

hope this helps.

nico

[Dr-Gigavolt]

Hi Nico, sorry, I still have no time. I'll reply in couple of days.

[nicobresseleers]

Hi Nico, sorry, I still have no time. I'll reply in couple of days.

No worries.. i have temporary set TEMP_BATTERY = 0 and it works.. but it's still a bug ( i need to set to 1 since i have a defect sensor that i cannot replace in 123).. for the time being: all OK!