Add 3-phase voltage to inverter data

The voltage was missing from the inverter data.

Signed-off-by: Daniel Zullo <[email protected]>

Author: daniel-zullo-frequenz

src/frequenz/sdk/microgrid/component/_component_data.py

@@ -237,6 +237,11 @@ class InverterData(ComponentData):
             -ve current means supply into the grid.
     """
 
+    voltage_per_phase: tuple[float, float, float]
+    """The AC voltage in Volts (V) between the line and the neutral wire for
+       phase/line 1, 2 and 3 respectively.
+    """
+
     # pylint: disable=line-too-long
     active_power_inclusion_lower_bound: float
     """Lower inclusion bound for inverter power in watts.
@@ -312,6 +317,11 @@ def from_proto(cls, raw: microgrid_pb.ComponentData) -> InverterData:
                 raw.inverter.data.ac.phase_2.current.value,
                 raw.inverter.data.ac.phase_3.current.value,
             ),
+            voltage_per_phase=(
+                raw.inverter.data.ac.phase_1.voltage.value,
+                raw.inverter.data.ac.phase_2.voltage.value,
+                raw.inverter.data.ac.phase_3.voltage.value,
+            ),
             active_power_inclusion_lower_bound=raw_power.system_inclusion_bounds.lower,
             active_power_exclusion_lower_bound=raw_power.system_exclusion_bounds.lower,
             active_power_inclusion_upper_bound=raw_power.system_inclusion_bounds.upper,

tests/microgrid/test_component_data.py

@@ -52,12 +52,15 @@ def test_inverter_data() -> None:
                     ),
                     phase_1=electrical_pb2.AC.ACPhase(
                         current=metrics_pb2.Metric(value=12.3),
+                        voltage=metrics_pb2.Metric(value=229.8),
                     ),
                     phase_2=electrical_pb2.AC.ACPhase(
                         current=metrics_pb2.Metric(value=23.4),
+                        voltage=metrics_pb2.Metric(value=230.0),
                     ),
                     phase_3=electrical_pb2.AC.ACPhase(
                         current=metrics_pb2.Metric(value=34.5),
+                        voltage=metrics_pb2.Metric(value=230.2),
                     ),
                 ),
             ),
@@ -76,6 +79,7 @@ def test_inverter_data() -> None:
     assert inv_data.frequency == pytest.approx(50.1)
     assert inv_data.active_power == pytest.approx(100.2)
     assert inv_data.current_per_phase == pytest.approx((12.3, 23.4, 34.5))
+    assert inv_data.voltage_per_phase == pytest.approx((229.8, 230.0, 230.2))
     assert inv_data.active_power_inclusion_lower_bound == pytest.approx(-51_000.0)
     assert inv_data.active_power_inclusion_upper_bound == pytest.approx(51_000.0)
     assert inv_data.active_power_exclusion_lower_bound == pytest.approx(-501.0)

tests/utils/component_data_wrapper.py

@@ -102,6 +102,7 @@ def __init__(  # pylint: disable=too-many-arguments
         timestamp: datetime,
         active_power: float = math.nan,
         current_per_phase: tuple[float, float, float] | None = None,
+        voltage_per_phase: tuple[float, float, float] | None = None,
         active_power_inclusion_lower_bound: float = math.nan,
         active_power_exclusion_lower_bound: float = math.nan,
         active_power_inclusion_upper_bound: float = math.nan,
@@ -126,6 +127,11 @@ def __init__(  # pylint: disable=too-many-arguments
                 if current_per_phase
                 else (math.nan, math.nan, math.nan)
             ),
+            voltage_per_phase=(
+                voltage_per_phase
+                if voltage_per_phase
+                else (math.nan, math.nan, math.nan)
+            ),
             active_power_inclusion_lower_bound=active_power_inclusion_lower_bound,
             active_power_exclusion_lower_bound=active_power_exclusion_lower_bound,
             active_power_inclusion_upper_bound=active_power_inclusion_upper_bound,