Remove unused paranthesis in formula generators (#761)

Author: shsms

src/frequenz/sdk/timeseries/formula_engine/_formula_generators/_battery_power_formula.py

@@ -70,12 +70,9 @@ def generate(
                 "Can't find inverters for all batteries from the component graph."
             )
 
-        builder.push_oper("(")
-        builder.push_oper("(")
         for idx, comp in enumerate(battery_inverters):
             if idx > 0:
                 builder.push_oper("+")
             builder.push_component_metric(comp.component_id, nones_are_zeros=True)
-        builder.push_oper(")")
 
         return builder.build()

src/frequenz/sdk/timeseries/formula_engine/_formula_generators/_chp_power_formula.py

@@ -50,13 +50,10 @@ def generate(  # noqa: DOC502 (FormulaGenerationError is raised indirectly by _g
             )
             return builder.build()
 
-        builder.push_oper("(")
-        builder.push_oper("(")
         for idx, chp_meter_id in enumerate(chp_meter_ids):
             if idx > 0:
                 builder.push_oper("+")
             builder.push_component_metric(chp_meter_id, nones_are_zeros=False)
-        builder.push_oper(")")
 
         return builder.build()
 

src/frequenz/sdk/timeseries/formula_engine/_formula_generators/_ev_charger_power_formula.py

@@ -41,12 +41,9 @@ def generate(self) -> FormulaEngine[Power]:
             )
             return builder.build()
 
-        builder.push_oper("(")
-        builder.push_oper("(")
         for idx, component_id in enumerate(component_ids):
             if idx > 0:
                 builder.push_oper("+")
             builder.push_component_metric(component_id, nones_are_zeros=True)
-        builder.push_oper(")")
 
         return builder.build()

src/frequenz/sdk/timeseries/formula_engine/_formula_generators/_grid_power_formula.py

@@ -40,8 +40,6 @@ def generate(  # noqa: DOC502
         #  - `PASSIVE_SIGN_CONVENTION`: `(grid-successor-1 + grid-successor-2 + ...)`
         #  - `PRODUCTION`: `max(0, -(grid-successor-1 + grid-successor-2 + ...))`
         #  - `CONSUMPTION`: `max(0, (grid-successor-1 + grid-successor-2 + ...))`
-        builder.push_oper("(")
-        builder.push_oper("(")
         for idx, comp in enumerate(grid_successors):
             if idx > 0:
                 builder.push_oper("+")
@@ -65,6 +63,5 @@ def generate(  # noqa: DOC502
             builder.push_component_metric(
                 comp.component_id, nones_are_zeros=nones_are_zeros
             )
-        builder.push_oper(")")
 
         return builder.build()

src/frequenz/sdk/timeseries/formula_engine/_formula_generators/_pv_power_formula.py

@@ -57,8 +57,6 @@ def generate(  # noqa: DOC502
             )
             return builder.build()
 
-        builder.push_oper("(")
-        builder.push_oper("(")
         for idx, component in enumerate(pv_components):
             if idx > 0:
                 builder.push_oper("+")
@@ -68,6 +66,5 @@ def generate(  # noqa: DOC502
                 component.component_id,
                 nones_are_zeros=component.category != ComponentCategory.METER,
             )
-        builder.push_oper(")")
 
         return builder.build()