Skip to content

Consider HybridInverter for battery and PV managers #1285

@llucax

Description

@llucax

There are a couple of issue with the current stand for inverters.

  1. The battery manager logic in src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_battery_manager.py will pick up any Inverter, but it should probably restrict inverters to BatteryInverter or HybridInverter when searching for predecessors.
  2. The PowerWrapper in _data_pipeline.py and PVManager src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_pv_inverter_manager/_pv_inverter_manager.py are using SolarInverter as type, but it should probably use SolarInverter | 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).

Metadata

Metadata

Assignees

No one assigned

    Labels

    part:microgridAffects the interactions with the microgridtype:enhancementNew feature or enhancement visitble to users

    Projects

    Status

    To do

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions