Make component_category and component_type parameters kw-only

This affects the PowerManager, PowerDistributor and PowerWrapper.  The
`component_type` paramater now also has a default value of `None`.

Signed-off-by: Sahas Subramanian <[email protected]>

Author: shsms

src/frequenz/sdk/actor/_power_managing/_power_managing_actor.py

@@ -32,20 +32,28 @@ class PowerManagingActor(Actor):
 
     def __init__(  # pylint: disable=too-many-arguments
         self,
-        component_category: ComponentCategory,
-        component_type: ComponentType | None,
         proposals_receiver: Receiver[Proposal],
         bounds_subscription_receiver: Receiver[ReportRequest],
         power_distributing_requests_sender: Sender[power_distributing.Request],
         power_distributing_results_receiver: Receiver[power_distributing.Result],
         channel_registry: ChannelRegistry,
+        *,
+        component_category: ComponentCategory,
+        component_type: ComponentType | None = None,
         # arguments to actors need to serializable, so we pass an enum for the algorithm
         # instead of an instance of the algorithm.
         algorithm: Algorithm = Algorithm.MATRYOSHKA,
     ):
         """Create a new instance of the power manager.
 
         Args:
+            proposals_receiver: The receiver for proposals.
+            bounds_subscription_receiver: The receiver for bounds subscriptions.
+            power_distributing_requests_sender: The sender for power distribution
+                requests.
+            power_distributing_results_receiver: The receiver for power distribution
+                results.
+            channel_registry: The channel registry.
             component_category: The category of the component this power manager
                 instance is going to support.
             component_type: The type of the component of the given category that this
@@ -55,13 +63,6 @@ def __init__(  # pylint: disable=too-many-arguments
                 the inverter as a solar inverter or a battery inverter.  This can be
                 `None` when the component category is enough to uniquely identify the
                 component.
-            proposals_receiver: The receiver for proposals.
-            bounds_subscription_receiver: The receiver for bounds subscriptions.
-            power_distributing_requests_sender: The sender for power distribution
-                requests.
-            power_distributing_results_receiver: The receiver for power distribution
-                results.
-            channel_registry: The channel registry.
             algorithm: The power management algorithm to use.
 
         Raises:

src/frequenz/sdk/actor/power_distributing/power_distributing.py

@@ -57,18 +57,25 @@ class PowerDistributingActor(Actor):
 
     def __init__(  # pylint: disable=too-many-arguments
         self,
-        component_category: ComponentCategory,
-        component_type: ComponentType | None,
         requests_receiver: Receiver[Request],
         results_sender: Sender[Result],
         component_pool_status_sender: Sender[ComponentPoolStatus],
         wait_for_data_sec: float,
         *,
+        component_category: ComponentCategory,
+        component_type: ComponentType | None = None,
         name: str | None = None,
     ) -> None:
         """Create class instance.
 
         Args:
+            requests_receiver: Receiver for receiving power requests from the power
+                manager.
+            results_sender: Sender for sending results to the power manager.
+            component_pool_status_sender: Channel for sending information about which
+                components are expected to be working.
+            wait_for_data_sec: How long actor should wait before processing first
+                request. It is a time needed to collect first components data.
             component_category: The category of the components that this actor is
                 responsible for.
             component_type: The type of the component of the given category that this
@@ -78,13 +85,6 @@ def __init__(  # pylint: disable=too-many-arguments
                 the inverter as a solar inverter or a battery inverter.  This can be
                 `None` when the component category is enough to uniquely identify the
                 component.
-            requests_receiver: Receiver for receiving power requests from the power
-                manager.
-            results_sender: Sender for sending results to the power manager.
-            component_pool_status_sender: Channel for sending information about which
-                components are expected to be working.
-            wait_for_data_sec: How long actor should wait before processing first
-                request. It is a time needed to collect first components data.
             name: The name of the actor. If `None`, `str(id(self))` will be used. This
                 is used mostly for debugging purposes.
 

src/frequenz/sdk/microgrid/_data_pipeline.py

@@ -98,13 +98,15 @@ def __init__(
         self._resampling_actor: _ActorInfo | None = None
 
         self._battery_power_wrapper = PowerWrapper(
-            ComponentCategory.BATTERY, None, self._channel_registry
+            self._channel_registry, component_category=ComponentCategory.BATTERY
         )
         self._ev_power_wrapper = PowerWrapper(
-            ComponentCategory.EV_CHARGER, None, self._channel_registry
+            self._channel_registry, component_category=ComponentCategory.EV_CHARGER
         )
         self._pv_power_wrapper = PowerWrapper(
-            ComponentCategory.INVERTER, InverterType.SOLAR, self._channel_registry
+            self._channel_registry,
+            component_category=ComponentCategory.INVERTER,
+            component_type=InverterType.SOLAR,
         )
 
         self._logical_meter: LogicalMeter | None = None

src/frequenz/sdk/microgrid/_power_wrapper.py

@@ -38,13 +38,15 @@ class PowerWrapper:
 
     def __init__(
         self,
-        component_category: ComponentCategory,
-        component_type: ComponentType | None,
         channel_registry: ChannelRegistry,
+        *,
+        component_category: ComponentCategory,
+        component_type: ComponentType | None = None,
     ):
         """Initialize the power control.
 
         Args:
+            channel_registry: A channel registry for use in the actors.
             component_category: The category of the components that actors started by
                 this instance of the PowerWrapper will be responsible for.
             component_type: The type of the component of the given category that this
@@ -54,7 +56,6 @@ def __init__(
                 the inverter as a solar inverter or a battery inverter.  This can be
                 `None` when the component category is enough to uniquely identify the
                 component.
-            channel_registry: A channel registry for use in the actors.
         """
         self._component_category = component_category
         self._component_type = component_type