Skip to content

Commit b6d1159

Browse files
committed
Rename _add_bounds_tracker_add_system_bounds_tracker
The older name was a bit ambiguous, because there are also bounds calculated by the power manager for use by actors. This change clears this ambiguity. Signed-off-by: Sahas Subramanian <[email protected]>
1 parent 2ad660f commit b6d1159

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ async def _bounds_tracker(
130130
await self._send_updated_target_power(component_ids, None)
131131
await self._send_reports(component_ids)
132132

133-
def _add_bounds_tracker(self, component_ids: frozenset[int]) -> None:
133+
def _add_system_bounds_tracker(self, component_ids: frozenset[int]) -> None:
134134
"""Add a bounds tracker.
135135
136136
Args:
@@ -225,7 +225,7 @@ async def _run(self) -> None:
225225
if selected_from(selected, self._proposals_receiver):
226226
proposal = selected.message
227227
if proposal.component_ids not in self._bound_tracker_tasks:
228-
self._add_bounds_tracker(proposal.component_ids)
228+
self._add_system_bounds_tracker(proposal.component_ids)
229229

230230
# TODO: must_send=True forces a new request to # pylint: disable=fixme
231231
# be sent to the PowerDistributor, even if there's no change in power.
@@ -260,7 +260,7 @@ async def _run(self) -> None:
260260
)
261261

262262
if sub.component_ids not in self._bound_tracker_tasks:
263-
self._add_bounds_tracker(sub.component_ids)
263+
self._add_system_bounds_tracker(sub.component_ids)
264264

265265
elif selected_from(selected, self._power_distributing_results_receiver):
266266
from .. import ( # pylint: disable=import-outside-toplevel

0 commit comments

Comments
 (0)