From 6380c4fdee38f9f0c04cb9b5e69c8675595b9967 Mon Sep 17 00:00:00 2001 From: Balduin Date: Thu, 9 Apr 2026 09:05:21 +0200 Subject: [PATCH] fix(battery_simulator): remove constraining again now that SIM_BAT_DRAIN=0 means the module is not started we are safe from division by zero again (param compare has a tolerance of 1e-7) --- src/modules/simulation/battery_simulator/BatterySimulator.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/modules/simulation/battery_simulator/BatterySimulator.cpp b/src/modules/simulation/battery_simulator/BatterySimulator.cpp index 36c9961bc1..4f2957cfea 100644 --- a/src/modules/simulation/battery_simulator/BatterySimulator.cpp +++ b/src/modules/simulation/battery_simulator/BatterySimulator.cpp @@ -85,8 +85,7 @@ void BatterySimulator::Run() const hrt_abstime now_us = hrt_absolute_time(); - // 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; + const float discharge_interval_us = _param_sim_bat_drain.get() * 1000 * 1000; if (_armed) { if (_last_integration_us != 0) {