Skip to content

Commit b16e2df

Browse files
committed
Don't default to zero target power before processing the proposals
Signed-off-by: Sahas Subramanian <[email protected]>
1 parent 044cbe6 commit b16e2df

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/frequenz/sdk/microgrid/_power_managing/_matryoshka.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ def _calc_target_power(
4949
self,
5050
proposals: set[Proposal],
5151
system_bounds: SystemBounds,
52-
) -> Power:
52+
) -> Power | None:
5353
"""Calculate the target power for the given components.
5454
5555
Args:
@@ -80,7 +80,7 @@ def _calc_target_power(
8080
):
8181
exclusion_bounds = system_bounds.exclusion_bounds
8282

83-
target_power = Power.zero()
83+
target_power = None
8484
for next_proposal in sorted(proposals, reverse=True):
8585
if upper_bound < lower_bound:
8686
break
@@ -200,7 +200,9 @@ def calculate_target_power(
200200

201201
target_power = self._calc_target_power(proposals, system_bounds)
202202

203-
self._target_power[component_ids] = target_power
203+
if target_power is not None:
204+
self._target_power[component_ids] = target_power
205+
204206
return target_power
205207

206208
@override

0 commit comments

Comments
 (0)