From ab7e7b59bbc8b8f5c96002241bcaf82dd7da7177 Mon Sep 17 00:00:00 2001 From: Balduin Date: Fri, 10 Apr 2026 08:51:40 +0200 Subject: [PATCH] fix(battery_simulator): constrain param to min of 1 to avoid division by zero. This reverts commit 6380c4fdee38f9f0c04cb9b5e69c8675595b9967. --- src/modules/simulation/battery_simulator/BatterySimulator.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/modules/simulation/battery_simulator/BatterySimulator.cpp b/src/modules/simulation/battery_simulator/BatterySimulator.cpp index 4f2957cfea..36c9961bc1 100644 --- a/src/modules/simulation/battery_simulator/BatterySimulator.cpp +++ b/src/modules/simulation/battery_simulator/BatterySimulator.cpp @@ -85,7 +85,8 @@ void BatterySimulator::Run() const hrt_abstime now_us = hrt_absolute_time(); - const float discharge_interval_us = _param_sim_bat_drain.get() * 1000 * 1000; + // Limit to +1.0 s to guard against division by 0 + const float discharge_interval_us = math::max(_param_sim_bat_drain.get(), 1.0f) * 1000 * 1000; if (_armed) { if (_last_integration_us != 0) {