What's needed?
ATM, inverters are allowed to have only one DC battery, and only one PV array. However, we can encounter inverters that support multiple batteries and multiple PV arrays. For this inverters, the current definition is not enough. Here is how it looks now:
|
// DC metrics for the inverter-battery linkage. |
|
// This is applicable to `BATTERY` and `HYBRID` inverters only. |
|
frequenz.api.common.metrics.electrical.DC dc_battery = 4; |
|
|
|
// DC metrics for the inverter-PV linkage. |
|
// This is applicable to `SOLAR` and `HYBRID` inverters only. |
|
frequenz.api.common.metrics.electrical.DC dc_solar = 5; |
Proposed solution
The DC metrics need to be changed to arrays.
// DC metrics for the inverter-battery linkage.
// This is applicable to `BATTERY` and `HYBRID` inverters only.
- frequenz.api.common.metrics.electrical.DC dc_battery = 4;
+ repeated frequenz.api.common.metrics.electrical.DC dc_battery = 4;
// DC metrics for the inverter-PV linkage.
// This is applicable to `SOLAR` and `HYBRID` inverters only.
- frequenz.api.common.metrics.electrical.DC dc_solar = 5;
+ repeated frequenz.api.common.metrics.electrical.DC dc_solar = 5;
Use cases
No response
Alternatives and workarounds
No response
Additional context
No response
What's needed?
ATM, inverters are allowed to have only one DC battery, and only one PV array. However, we can encounter inverters that support multiple batteries and multiple PV arrays. For this inverters, the current definition is not enough. Here is how it looks now:
frequenz-api-microgrid/proto/frequenz/api/microgrid/inverter.proto
Lines 101 to 107 in b355c8b
Proposed solution
The DC metrics need to be changed to arrays.
Use cases
No response
Alternatives and workarounds
No response
Additional context
No response