From a0726398651b7b1c20cb91f46b283b049a264088 Mon Sep 17 00:00:00 2001 From: robbederks Date: Fri, 17 Apr 2020 14:05:43 -0700 Subject: [PATCH] Added value-error for negative integration (#1390) old-commit-hash: 15f57c4c22bb8e1f091344891e5b7d88443683d1 --- selfdrive/thermald/power_monitoring.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/selfdrive/thermald/power_monitoring.py b/selfdrive/thermald/power_monitoring.py index fe6e8ec2d4..378bee6f91 100644 --- a/selfdrive/thermald/power_monitoring.py +++ b/selfdrive/thermald/power_monitoring.py @@ -152,7 +152,10 @@ class PowerMonitoring: try: if self.last_measurement_time: integration_time_h = (t - self.last_measurement_time) / 3600 - self.power_used_uWh += (current_power * 1000000) * integration_time_h + power_used = (current_power * 1000000) * integration_time_h + if power_used < 0: + raise ValueError(f"Negative power used! Integration time: {integration_time_h} h Current Power: {power_used} uWh") + self.power_used_uWh += power_used self.last_measurement_time = t except Exception: cloudlog.exception("Integration failed")