Merge pull request #161 from cyiallou/fix/alert-email

cyiallou · web-flow · commit 383a18e01bae · 2025-09-12T11:27:40.000Z
fix: render component IDs as plain ints in alert email summaries
diff --git a/src/frequenz/lib/notebooks/alerts/alert_email.py b/src/frequenz/lib/notebooks/alerts/alert_email.py
@@ -303,7 +303,7 @@ def generate_alert_summary(
                 "state_value",
                 lambda x: [html.escape(str(s)) for s in x.unique()],
             ),
-            unique_components=("component_id", lambda x: list(x.unique())),
+            unique_components=("component_id", lambda x: [int(c) for c in x.unique()]),
         )
         .reset_index()
     )

Original file line number	Diff line number	Diff line change
`@@ -303,7 +303,7 @@ def generate_alert_summary(`
`303`	`303`	`"state_value",`
`304`	`304`	`lambda x: [html.escape(str(s)) for s in x.unique()],`
`305`	`305`	`),`
`306`		`- unique_components=("component_id", lambda x: list(x.unique())),`
	`306`	`+ unique_components=("component_id", lambda x: [int(c) for c in x.unique()]),`
`307`	`307`	`)`
`308`	`308`	`.reset_index()`
`309`	`309`	`)`