rusefi
/
rusefi-1
mirror of https://github.com/rusefi/rusefi-1.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

32 bit vs 64 bit compiler challenge for floats in unit tests

This commit is contained in:
rusefi 2019-11-10 13:57:27 -05:00
parent b996190707
commit 25298e2392
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
unit_tests/tests/test_pid.cpp
View File

@ -138,11 +138,11 @@ TEST(util, pidIndustrial) {
pid.derivativeFilterLoss = 0.1;
// now the first value is much less due to the derivative filtering
ASSERT_EQ(10.5288095f, pid.getOutput(1, 0));
ASSERT_NEAR(10.5288095f, pid.getOutput(1, 0), EPS4D);
// here we still have some leftovers of the initial D-term
ASSERT_EQ(10.0802946f, pid.getOutput(1, 0));
ASSERT_NEAR(10.0802946f, pid.getOutput(1, 0), EPS4D);
// but the fading is slower than with 'weaker' derivative filter above
ASSERT_EQ(9.65337563f, pid.getOutput(1, 0));
ASSERT_NEAR(9.65337563f, pid.getOutput(1, 0), EPS4D);
pid.reset();
pid.derivativeFilterLoss = 0;
@ -153,8 +153,8 @@ TEST(util, pidIndustrial) {
// the first value is clipped, and that's when the anti-windup comes into effect
ASSERT_EQ(100.0f, pid.getOutput(1, 0));
// it stores a small negative offset in the I-term to avoid it's saturation!
ASSERT_EQ(-0.0455025025f, pid.getIntegration());
ASSERT_NEAR(-0.0455025025f, pid.getIntegration(), EPS4D);
// and that's why the second output is smaller then that of normal PID (=1.00999999)
ASSERT_EQ(0.959497511f, pid.getOutput(1, 0));
ASSERT_NEAR(0.959497511f, pid.getOutput(1, 0), EPS4D);
}