Skip to content

Use self consume as part of idle strategy when actual pv and/or consu… #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+9 −9
Open

Use self consume as part of idle strategy when actual pv and/or consu… #10

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions delegates/dynamicess.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
from enum import Enum

NUM_SCHEDULES = 12
INTERVAL = 5
INTERVAL = 2
SELLPOWER = -32000
HUB4_SERVICE = 'com.victronenergy.hub4'

Expand Down Expand Up @@ -60,7 +60,7 @@ def charge(self, flags, restrictions, rate, allow_feedin):
def discharge(self, flags, restrictions, rate, allow_feedin):
raise NotImplementedError("discharge")

def idle(self, allow_feedin):
def idle(self, restrictions, allow_feedin):
raise NotImplementedError("idle")

def self_consume(self, restrictions, allow_feedin):
Expand Down Expand Up @@ -177,21 +177,21 @@ def discharge(self, flags, restrictions, rate, allow_feedin):
self.monitor.set_value_async(HUB4_SERVICE, '/Overrides/MaxDischargePower', -1)
return rate

def idle(self, allow_feedin):
def idle(self, restrictions, allow_feedin):
self._set_feedin(allow_feedin)
self._set_charge_power(None)
self.monitor.set_value_async(HUB4_SERVICE, '/Overrides/ForceCharge', 0)

if allow_feedin:
# When idle only configure setpoint and maxDischarge overrides if consumption < pv. Otherwise, use standard ESS
# (which is also quicker to react) to avoid importing when pv is lower, or consumption higher, than forecast.
if allow_feedin and self.consumption < self.acpv + self.pvpower:
# This keeps battery idle by not allowing more power to be taken
# from the DC bus than what DC-coupled PV provides.
self.monitor.set_value_async(HUB4_SERVICE, '/Overrides/MaxDischargePower',
max(1.0, round(0.9*self.pvpower)))
self.monitor.set_value_async(HUB4_SERVICE, '/Overrides/Setpoint', self.maxfeedinpower)
else:
self.monitor.set_value_async(HUB4_SERVICE, '/Overrides/Setpoint', 0) # Normal ESS
self.monitor.set_value_async(HUB4_SERVICE, '/Overrides/MaxDischargePower', max(1.0, self.pvpower))

self.self_consume(restrictions, allow_feedin)
return None

def self_consume(self, restrictions, allow_feedin):
Expand Down Expand Up @@ -276,7 +276,7 @@ def discharge(self, flags, restrictions, rate, allow_feedin):

return rate

def idle(self, allow_feedin):
def idle(self, restrictions, allow_feedin):
self.monitor.set_value_async(self.service, '/Ess/DisableFeedIn', int(not allow_feedin))
self.monitor.set_value_async(self.service, '/Ess/UseInverterPowerSetpoint', 1)
self.monitor.set_value_async(self.service, '/Ess/InverterPowerSetpoint', -max(0, self.pvpower))
Expand Down Expand Up @@ -583,7 +583,7 @@ def bail(code):
# zone
self.discharge_hysteresis = 1
self._dbusservice['/DynamicEss/ChargeRate'] = \
self._device.idle(w.allow_feedin)
self._device.idle(restrictions, w.allow_feedin)

break # out of for loop
else:
Expand Down