Fix consumer power formula

matthias-wende-frequenz · matthias-wende-frequenz · commit 6d068b1e79c5 · 2023-09-08T11:48:11.000+02:00
All grid meters have to be summed for this formula but in the current
implemantation there was minus pushed to the operator stack. In this
commit the sign error has been corrected.

Signed-off-by: Matthias Wende &lt;matthias.wende@frequenz.com&gt;
diff --git a/src/frequenz/sdk/timeseries/_formula_engine/_formula_generators/_consumer_power_formula.py b/src/frequenz/sdk/timeseries/_formula_engine/_formula_generators/_consumer_power_formula.py
@@ -106,7 +106,7 @@ def non_consumer_component(component: Component) -> bool:
         # push all grid meters
         for idx, grid_meter in enumerate(grid_meters):
             if idx > 0:
-                builder.push_oper("-")
+                builder.push_oper("+")
             builder.push_component_metric(
                 grid_meter.component_id, nones_are_zeros=False
             )

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ def non_consumer_component(component: Component) -> bool:`
`106`	`106`	`# push all grid meters`
`107`	`107`	`for idx, grid_meter in enumerate(grid_meters):`
`108`	`108`	`if idx > 0:`
`109`		`- builder.push_oper("-")`
	`109`	`+ builder.push_oper("+")`
`110`	`110`	`builder.push_component_metric(`
`111`	`111`	`grid_meter.component_id, nones_are_zeros=False`
`112`	`112`	`)`