pulquero / BatteryAggregator

MIT License
32 stars 8 forks source link

[Question] SOC to calculate CVL and DCL from the shunt #20

Closed entropia1ac closed 6 months ago

entropia1ac commented 6 months ago

I have 3 virtual batteries from 3 BMSes, each in series with a shunt.

"virtualBatteries": {
    "com.victronenergy.battery.virtual1": ["com.victronenergy.battery.ttyS7", "com.victronenergy.battery.ttyUSB4"],
    "com.victronenergy.battery.virtual2": ["com.victronenergy.battery.ttyS6", "com.victronenergy.battery.ttyUSB3"],

"com.victronenergy.battery.virtual3": ["com.victronenergy.battery.ttyS5", "com.victronenergy.battery.ttyUSB1"] } }

Is it possible that for the calculation of CCL and DCL the SOC is taken from the shunt and not from the BMS ?

pulquero commented 6 months ago

First entry in the array has precedence, so ttyS data will be preferred over ttyUSB. You can swap the order and/or use the more advanced syntax:

"virtualBatteries": {
    "com.victronenergy.battery.virtual1": {"com.victronenergy.battery.ttyS6": ["/Soc"], "com.victronenergy.battery.ttyUSB0": []},
    "com.victronenergy.battery.virtual2": {"com.victronenergy.battery.ttyS7": ["/Soc"], "com.victronenergy.battery.ttyUSB1": []]
}

to specify only some dbus paths to take precedence.

entropia1ac commented 6 months ago

Great.Thanks

entropia1ac commented 6 months ago

BA does not accept a new config.json file : { "virtualBatteries": { "com.victronenergy.battery.virtual1": {"com.victronenergy.battery.ttyS7" : ["/Soc"], "com.victronenergy.battery.ttyUSB4":[]}, "com.victronenergy.battery.virtual2": {"com.victronenergy.battery.ttyS6" : ["/Soc"], "com.victronenergy.battery.ttyUSB2": []}, "com.victronenergy.battery.virtual3": {"com.victronenergy.battery.ttyS5" : ["/Soc"], "com.victronenergy.battery.ttyUSB1": []] } } }

SmartSelect_20240218_154453_Termius

pulquero commented 6 months ago

Last ] should be a }

entropia1ac commented 6 months ago

This option does not work: { "virtualBatteries": { "com.victronenergy.battery.virtual1": {"com.victronenergy.battery.ttyS7" : ["/Soc"], "com.victronenergy.battery.ttyUSB4":[]}, "com.victronenergy.battery.virtual2": {"com.victronenergy.battery.ttyS6" : ["/Soc"], "com.victronenergy.battery.ttyUSB2": []}, "com.victronenergy.battery.virtual3": {"com.victronenergy.battery.ttyS5" : ["/Soc"], "com.victronenergy.battery.ttyUSB1": []} } }

SmartSelect_20240218_212620_Opera SmartSelect_20240218_212504_Termius

pulquero commented 6 months ago

Try restarting the service, using svc. I'm guessing it is an ordering issue where BA is started before the settings service.

entropia1ac commented 6 months ago

Ok. I' check it out

entropia1ac commented 6 months ago

With this file config.json BA sees 4 batteries instead of 3 SmartSelect_20240219_154448_Opera SmartSelect_20240219_154327_Opera SmartSelect_20240219_154237_Termius

pulquero commented 6 months ago

dbus-spy BA, look at /System/Batteries, it will list the 4 batteries it thinks it has found.