-
Notifications
You must be signed in to change notification settings - Fork 21
Open
Labels
part:microgridAffects the interactions with the microgridAffects the interactions with the microgridtype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
Milestone
Description
There are a couple of issue with the current stand for inverters.
- The battery manager logic in
src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_battery_manager.pywill pick up anyInverter, but it should probably restrict inverters toBatteryInverterorHybridInverterwhen searching for predecessors. - The
PowerWrapperin_data_pipeline.pyandPVManagersrc/frequenz/sdk/microgrid/_power_distributing/_component_managers/_pv_inverter_manager/_pv_inverter_manager.pyare usingSolarInverteras type, but it should probably useSolarInverter | HybridInverter.
In general, we might be more tolerant and accept any inverter, but print a warning if the inverter is of an unexpected type (as long as we can still try to work with that type of inverter).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
part:microgridAffects the interactions with the microgridAffects the interactions with the microgridtype:enhancementNew feature or enhancement visitble to usersNew feature or enhancement visitble to users
Type
Projects
Status
To do