Use abc.Set for component_ids in EVChargerPool

`abc.Set` allows us to treat `set`s as read-only, and interchangably
with `frozenset`s.  Many of the SDK's methods that give out component
IDs produce `abc.Set` values, and with this change, such values can
directly be used in calls to `ev_charger_pool`.

This is also to be consistent with `battery_pool`, which already
accepts `abc.Set`.

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

Author: shsms

src/frequenz/sdk/microgrid/_data_pipeline.py

@@ -154,7 +154,7 @@ def producer(self) -> Producer:
 
     def ev_charger_pool(
         self,
-        ev_charger_ids: set[int] | None = None,
+        ev_charger_ids: abc.Set[int] | None = None,
     ) -> EVChargerPool:
         """Return the corresponding EVChargerPool instance for the given ids.
 
@@ -350,7 +350,7 @@ def producer() -> Producer:
     return _get().producer()
 
 
-def ev_charger_pool(ev_charger_ids: set[int] | None = None) -> EVChargerPool:
+def ev_charger_pool(ev_charger_ids: abc.Set[int] | None = None) -> EVChargerPool:
     """Return the corresponding EVChargerPool instance for the given ids.
 
     If an EVChargerPool instance for the given ids doesn't exist, a new one is

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

@@ -77,7 +77,7 @@ def __init__(
         self,
         channel_registry: ChannelRegistry,
         resampler_subscription_sender: Sender[ComponentMetricRequest],
-        component_ids: set[int] | None = None,
+        component_ids: abc.Set[int] | None = None,
         repeat_interval: timedelta = timedelta(seconds=3.0),
     ) -> None:
         """Create an `EVChargerPool` instance.
@@ -103,7 +103,7 @@ def __init__(
         self._resampler_subscription_sender: Sender[ComponentMetricRequest] = (
             resampler_subscription_sender
         )
-        self._component_ids: set[int] = set()
+        self._component_ids: abc.Set[int] = set()
         if component_ids is not None:
             self._component_ids = component_ids
         else:

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

@@ -6,6 +6,7 @@
 from __future__ import annotations
 
 import asyncio
+from collections import abc
 from enum import Enum
 
 from frequenz.channels import Merger, Receiver, merge
@@ -78,7 +79,7 @@ def is_ev_connected(self) -> bool:
 class StateTracker:
     """A class for keeping track of the states of all EV Chargers in a pool."""
 
-    def __init__(self, component_ids: set[int]) -> None:
+    def __init__(self, component_ids: abc.Set[int]) -> None:
         """Create a `_StateTracker` instance.
 
         Args: