From 0c3655b01f9ceed468a4bbdeef5af9e2cba8a1a4 Mon Sep 17 00:00:00 2001 From: Andrew Tridgell Date: Thu, 14 Jan 2021 12:16:20 +1100 Subject: [PATCH] autotest: fixed epsilon handling in set_parameters --- Tools/autotest/common.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Tools/autotest/common.py b/Tools/autotest/common.py index bfc35878df..683f39190f 100644 --- a/Tools/autotest/common.py +++ b/Tools/autotest/common.py @@ -3521,7 +3521,7 @@ class AutoTest(ABC): self.set_parameters({name: value }, **kwargs) - def set_parameters(self, parameters, add_to_context=True, epsilon=0.000012, retries=None, verbose=True): + def set_parameters(self, parameters, add_to_context=True, epsilon_pct=0.00001, retries=None, verbose=True): """Set parameters from vehicle.""" want = copy.copy(parameters) self.progress("set_parameters: (%s)" % str(want)) @@ -3556,7 +3556,7 @@ class AutoTest(ABC): # self.progress("Requesting (%s) (retry=%u)" % (name, i)) continue delta = abs(autopilot_values[name] - value) - if delta <= epsilon: + if delta <= epsilon_pct*0.01*abs(value): # correct value self.progress("%s is now %f" % (name, autopilot_values[name])) if add_to_context: