search

pulquero / BatteryAggregator

MIT License
36 stars 7 forks source link

V1.8 not working #5

Closed Scheppo closed 1 year ago

Scheppo commented 1 year ago

First of all thx for your fast programming.

when I add the „capacity“: 400 to config.json then the System don’t work. SSH connection is very slow and I can’t change the config.json anymore. Reboot don’t help

Scheppo commented 1 year ago

Ok, I have deleted the config.json file. Now the ssh connection is stable again. I have uninstalled BatteryAggregator befor but it was hard to come to the ssh console.

config.json: { "excludedServices": ["com.victronenergy.battery.socketcan_can1"] "capacity": 400 }

The min and maxcellvoltage is null: E25C2BC5-5AC3-467F-9346-E75DD98CB971

pulquero commented 1 year ago

The error is your json file is invalid, missing a comma separating the two entries. I suspect it got stuck in a loop trying to continually restart the service. Added a check to safely ignore invalid files. Note you can check the log for errors with cat /var/log/BatteryAggregator/current

Scheppo commented 1 year ago

Sorry iam Beginner in Linux so i don’t know the right Syntax in json file. It works now, but the Detail Page is empty: 64493CB7-F7DB-4F9B-BEC6-855F63888C5B

Scheppo commented 1 year ago

The detail Page is in 1.8.2 also empty, because i excluded the Can Battery in config file. Look at the log, can you see everything wrong? 1ABE0E21-5F71-4104-8FC5-D724ECD9F938

pulquero commented 1 year ago

Log looks ok. To confirm, the details page works if you don't exclude the can battery? So, I think you need to exclude the smart shunt for the can battery instead of the can battery.

Scheppo commented 1 year ago

Yes, details are working when can battery is not excluded. The reason why I use the Shunts for measure Volt and Ampere is the much better accuracy. The Bms don’t measure Current under 1 Ampere, the shunt can do till 0,1 Ampere.

pulquero commented 1 year ago

Ah, I see, so really want you need is a "virtual battery" that merges the shunt and can data. Then aggregate that with the other shunt?

Scheppo commented 1 year ago

Yes I need only MinCellVolt and MaxCellVolt from Can Battery in Batteryaggregator.

My System is follow: One 120Ah Battery with Smartshunt One 280Ah Battery with Smartshunt and also Can connected.

pulquero commented 1 year ago

I've now added virtual battery support. Note, there is an issue with stopping the service - you'll need to manually kill any child processes.