-
Notifications
You must be signed in to change notification settings - Fork 15
Inverter type WIND_TURBINE is redundant #393
Copy link
Copy link
Closed
Labels
part:❓We need to figure out which part is affectedWe need to figure out which part is affectedpriority:❓We need to figure out how soon this should be addressedWe need to figure out how soon this should be addressedtype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
Milestone
Metadata
Metadata
Assignees
Labels
part:❓We need to figure out which part is affectedWe need to figure out which part is affectedpriority:❓We need to figure out how soon this should be addressedWe need to figure out how soon this should be addressedtype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
What's needed?
Wind turbines were already specified in the protobuf definitions as an inverter type, which is now here:
frequenz-api-common/proto/frequenz/api/common/v1alpha7/microgrid/electrical_components/electrical_components.proto
Lines 545 to 546 in 7953a91
Wind turbines are now also specified as an electrical component category.
The latter can be considered redundant, and can be confusing to users.
On the other hand, wind turbines typically do AC->DC->AC conversion. They typically use variable-speed drives to produce variable frequency AC, which then gets converted into DC, which then gets converted to AC with a stable frequency matching that of the local grid. From this perspective, classifying wind turbines as inverters would be technically incorrect, and could lead to confusions.
Proposed solution
Prioritizing technical correctness, the inverter type wind turbine should be removed.
Use cases
No response
Alternatives and workarounds
No response
Additional context
No response