Pass through zero-power requests from PowerManager

Instead of adjusting them to the exclusion bounds, because the API
accepts zero power values even when there are exclusion bounds.

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

Author: shsms

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

@@ -138,7 +138,7 @@ def clamp_to_bounds(  # pylint: disable=too-many-return-statements
 
     # If the given value is within the exclusion bounds and the exclusion bounds are
     # within the given bounds, clamp the given value to the closest exclusion bound.
-    if exclusion_bounds is not None:
+    if exclusion_bounds is not None and not value.isclose(Power.zero()):
         if exclusion_bounds.lower < value < exclusion_bounds.upper:
             return exclusion_bounds.lower, exclusion_bounds.upper
 

tests/actor/_power_managing/test_matryoshka.py

@@ -313,6 +313,13 @@ async def test_matryoshka_with_excl_3() -> None:
     )
 
     tester = StatefulTester(batteries, system_bounds)
+    tester.tgt_power(priority=2, power=10.0, bounds=(None, None), expected=30.0)
+    tester.tgt_power(priority=2, power=-10.0, bounds=(None, None), expected=-30.0)
+    tester.tgt_power(priority=2, power=0.0, bounds=(None, None), expected=0.0)
+    tester.tgt_power(priority=3, power=20.0, bounds=(None, None), expected=None)
+    tester.tgt_power(priority=1, power=-20.0, bounds=(None, None), expected=-30.0)
+    tester.tgt_power(priority=3, power=None, bounds=(None, None), expected=None)
+    tester.tgt_power(priority=1, power=None, bounds=(None, None), expected=0.0)
 
     tester.tgt_power(priority=2, power=25.0, bounds=(25.0, 50.0), expected=30.0)
     tester.bounds(priority=2, expected_power=30.0, expected_bounds=(-200.0, 200.0))

tests/actor/_power_managing/test_report.py

@@ -58,7 +58,7 @@ def test_adjust_to_bounds() -> None:
         inclusion_bounds=(-200.0, 200.0),
         exclusion_bounds=(-30.0, 30.0),
     )
-    tester.case(0.0, -30.0, 30.0)
+    tester.case(0.0, 0.0, 0.0)
     tester.case(-210.0, -200.0, None)
     tester.case(220.0, None, 200.0)
     tester.case(-20.0, -30.0, 30.0)