muexxl / batcontrol

MIT License
18 stars 6 forks source link

Nachfrage, funktioniert always_allow_discharge_limit richtig? #18

Closed MaStr closed 2 weeks ago

MaStr commented 2 weeks ago
    def is_discharge_allowed(self, net_consumption: np.ndarray, prices: dict):
        # always allow discharging when battery is >90% maxsoc
        discharge_limit = self.get_max_capacity() * self.always_allow_discharge_limit
        soc = self.get_SOC()
        if soc > discharge_limit:
            logger.debug(
                f'[BatCTRL] Battery level ({soc}) above discharge limit {discharge_limit}')
            return True

Ich nahm in meiner unendlichen Naivität an, dass get_SOC() den % Wert liefern soll, das scheint aber nicht korrekt zu sein, da das IF hier eigentlich den current_capacaty Wert braucht. Das wäre dann eigentlich "get_stored_energy()" was verlgichen wird, wenn ich mir die Berechnung vorher anschaue.

Ich hatte den get_SOC(), so wie oben im Testtreiber implementiert. Deswegen könnte das eine oder andere in der Umsellung auf die API Funktion gelitten haben.

Kannst du @muexxl mir da bitte Feedback geben?

muexxl commented 2 weeks ago

Ja nee, das macht so keinen Sinn hier wird SOC in der Range (0-100) mit dem discharge_limit in kWh verglichen. Muss man fixen. Ich könnt das morgen vormittag erledigen.

muexxl commented 2 weeks ago

Ok.. du warst einfach schneller :)

MaStr commented 2 weeks ago

Hatte ich schon fertig. ;-) Am 23. Okt. 2024, 10:42 +0200 schrieb Stephan Mükusch @.***>:

Ok.. du warst einfach schneller :) — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you modified the open/close state.Message ID: @.***>