Bug fix: Print Power as str instead of decimal (#1232)

ela-kotulska-frequenz · web-flow · commit 5f5c5407d179 · 2025-06-11T09:17:07.000Z
Commit 41d4543 changed `float` to `Power` in PowerDistributor's battery manager, but left %d in formatting.
diff --git a/RELEASE_NOTES.md b/RELEASE_NOTES.md
@@ -14,4 +14,4 @@
 
 ## Bug Fixes
 
-<!-- Here goes notable bug fixes that are worth a special mention or explanation -->
+* Fix TypeError raised by BatteryManager when distributing power
diff --git a/src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_battery_manager.py b/src/frequenz/sdk/microgrid/_power_distributing/_component_managers/_battery_manager.py
@@ -273,7 +273,7 @@ async def _distribute_power(
                     battery_distribution.get(battery_id, Power.zero()) + dist
                 )
         _logger.debug(
-            "Distributing power %d between the batteries %s",
+            "Distributing power %s between the batteries %s",
             distributed_power_value,
             str(battery_distribution),
         )

Original file line number	Diff line number	Diff line change
`@@ -14,4 +14,4 @@`
`14`	`14`
`15`	`15`	`## Bug Fixes`
`16`	`16`
`17`		`-<!-- Here goes notable bug fixes that are worth a special mention or explanation -->`
	`17`	`+* Fix TypeError raised by BatteryManager when distributing power`
Original file line number	Diff line number	Diff line change
`@@ -273,7 +273,7 @@ async def _distribute_power(`
`273`	`273`	`battery_distribution.get(battery_id, Power.zero()) + dist`
`274`	`274`	`)`
`275`	`275`	`_logger.debug(`
`276`		`- "Distributing power %d between the batteries %s",`
	`276`	`+ "Distributing power %s between the batteries %s",`
`277`	`277`	`distributed_power_value,`
`278`	`278`	`str(battery_distribution),`
`279`	`279`	`)`