Fix the new too-many-positional-arguments pylint errors

When appropriate, we just make arguments keyword-only, which is a good
practice anyway. For tests, in general we just disabled the check.

Signed-off-by: Leandro Lucarella <[email protected]>

Author: llucax

src/frequenz/sdk/microgrid/_power_distributing/_component_pool_status_tracker.py

@@ -33,6 +33,7 @@ class ComponentPoolStatusTracker:
 
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         component_ids: abc.Set[int],
         component_status_sender: Sender[ComponentPoolStatus],
         max_data_age: timedelta,

src/frequenz/sdk/microgrid/_power_distributing/_component_status/_battery_status_tracker.py

@@ -99,6 +99,7 @@ class BatteryStatusTracker(ComponentStatusTracker, BackgroundService):
     @override
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         component_id: int,
         max_data_age: timedelta,
         max_blocking_duration: timedelta,

src/frequenz/sdk/microgrid/_power_distributing/_component_status/_component_status.py

@@ -85,6 +85,7 @@ class ComponentStatusTracker(BackgroundService, ABC):
     @abstractmethod
     def __init__(  # pylint: disable=too-many-arguments,super-init-not-called
         self,
+        *,
         component_id: int,
         max_data_age: timedelta,
         max_blocking_duration: timedelta,

src/frequenz/sdk/microgrid/_power_distributing/_component_status/_ev_charger_status_tracker.py

@@ -46,6 +46,7 @@ class EVChargerStatusTracker(ComponentStatusTracker, BackgroundService):
     @override
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         component_id: int,
         max_data_age: timedelta,
         max_blocking_duration: timedelta,

src/frequenz/sdk/microgrid/_power_distributing/_component_status/_pv_inverter_status_tracker.py

@@ -43,6 +43,7 @@ class PVInverterStatusTracker(ComponentStatusTracker, BackgroundService):
     @override
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         component_id: int,
         max_data_age: timedelta,
         max_blocking_duration: timedelta,

src/frequenz/sdk/microgrid/_power_distributing/_distribution_algorithm/_battery_distribution_algorithm.py

@@ -449,6 +449,7 @@ def _compute_battery_availability_ratio(
 
     def _distribute_power(  # pylint: disable=too-many-arguments
         self,
+        *,
         components: list[InvBatPair],
         power_w: float,
         available_soc: dict[int, float],
@@ -730,7 +731,11 @@ def _distribute_consume_power(
         )
 
         return self._distribute_power(
-            components, power_w, available_soc, incl_bounds, excl_bounds
+            components=components,
+            power_w=power_w,
+            available_soc=available_soc,
+            incl_bounds=incl_bounds,
+            excl_bounds=excl_bounds,
         )
 
     def _distribute_supply_power(
@@ -761,7 +766,11 @@ def _distribute_supply_power(
         )
 
         result: DistributionResult = self._distribute_power(
-            components, -1 * power_w, available_soc, incl_bounds, excl_bounds
+            components=components,
+            power_w=-1 * power_w,
+            available_soc=available_soc,
+            incl_bounds=incl_bounds,
+            excl_bounds=excl_bounds,
         )
 
         for inverter_id in result.distribution.keys():

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

@@ -32,12 +32,12 @@ class PowerManagingActor(Actor):  # pylint: disable=too-many-instance-attributes
 
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         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

src/frequenz/sdk/timeseries/battery_pool/_battery_pool_reference_store.py

@@ -39,6 +39,7 @@ class BatteryPoolReferenceStore:  # pylint: disable=too-many-instance-attributes
 
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         channel_registry: ChannelRegistry,
         resampler_subscription_sender: Sender[ComponentMetricRequest],
         batteries_status_receiver: Receiver[ComponentPoolStatus],

src/frequenz/sdk/timeseries/ev_charger_pool/_ev_charger_pool_reference_store.py

@@ -35,6 +35,7 @@ class EVChargerPoolReferenceStore:
 
     def __init__(  # pylint: disable=too-many-arguments
         self,
+        *,
         channel_registry: ChannelRegistry,
         resampler_subscription_sender: Sender[ComponentMetricRequest],
         status_receiver: Receiver[ComponentPoolStatus],

src/frequenz/sdk/timeseries/formula_engine/_formula_engine_pool.py

@@ -80,12 +80,12 @@ def from_string(
             return self._string_engines[channel_key]
 
         builder = ResampledFormulaBuilder(
-            self._namespace,
-            formula,
-            self._channel_registry,
-            self._resampler_subscription_sender,
-            component_metric_id,
-            Quantity,
+            namespace=self._namespace,
+            formula_name=formula,
+            channel_registry=self._channel_registry,
+            resampler_subscription_sender=self._resampler_subscription_sender,
+            metric_id=component_metric_id,
+            create_method=Quantity,
         )
         formula_engine = builder.from_string(formula, nones_are_zeros=nones_are_zeros)
         self._string_engines[channel_key] = formula_engine