pulquero / BatteryAggregator

MIT License
32 stars 8 forks source link

CCL & DCL are not aggregated #25

Closed johnboyi closed 5 months ago

johnboyi commented 5 months ago

Hi!

I have 2 Systems running, and on both (both 3xMultiplus5000, one system with 2 Basengreen batteries, one with 3 Basengreen batteries) CCL & DCL are not aggregated? Any idea?

2024-03-06

dbus-spy 2024-03-06 (0)

Aggregator: 2024-03-06 (1)

1st battery: 2024-03-06 (2) 2024-03-06 (3) 2024-03-06 (5)

pulquero commented 5 months ago

Check Io/AllowToCharge on each battery, if it is zero/off then its CCL is ignored (it should probably be taken to be zero, but I can fix that if this is the case). Also, check the log cat /var/log/BatteryAggregator/current

johnboyi commented 5 months ago

Screenshot (1) Can this be changed?

here´s the log: Screenshot (2)

pulquero commented 5 months ago

If we are talking about a battery then the logic in dbus-serialbattery is

    self._dbusservice["/Io/AllowToCharge"] = (
        1
        if self.battery.charge_fet
        and self.battery.control_allow_charge
        and self.block_because_disconnect is False
        else 0
    )

It looks like these variables are not supported by lifepower bms hence it is 0. I might try to contact dbus-serialbattery author to try to understand why this is the case.

pulquero commented 5 months ago

Has the latest version helped?

johnboyi commented 5 months ago

Hi! Yes! It works great! I thank you very very much!!! Robert -------- Ursprüngliche Nachricht --------Von: pulquero @.> Datum: 14.03.24 17:31 (GMT+01:00) An: pulquero/BatteryAggregator @.> Cc: johnboyi @.>, Author @.> Betreff: Re: [pulquero/BatteryAggregator] CCL & DCL are not aggregated (Issue

25)

Has the latest version helped?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

johnboyi commented 5 months ago

Now it works! Thank you!