James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 1 | #include "frc971/zeroing/imu_zeroer.h" |
| 2 | |
Philipp Schrader | 790cb54 | 2023-07-05 21:06:52 -0700 | [diff] [blame] | 3 | #include "gtest/gtest.h" |
| 4 | |
| 5 | #include "aos/flatbuffers.h" |
| 6 | |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 7 | namespace frc971::zeroing { |
| 8 | |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 9 | static constexpr int kMinSamplesToZero = |
| 10 | 2 * ImuZeroer::kSamplesToAverage * ImuZeroer::kRequiredZeroPoints; |
| 11 | |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 12 | aos::FlatbufferDetachedBuffer<IMUValues> MakeMeasurement( |
James Kuszmaul | ddc6970 | 2021-03-24 21:55:03 -0700 | [diff] [blame] | 13 | const Eigen::Vector3d &gyro, const Eigen::Vector3d &accel, |
| 14 | bool faulted = false) { |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 15 | flatbuffers::FlatBufferBuilder fbb; |
Austin Schuh | d7b15da | 2020-02-17 15:06:11 -0800 | [diff] [blame] | 16 | fbb.ForceDefaults(true); |
James Kuszmaul | ddc6970 | 2021-03-24 21:55:03 -0700 | [diff] [blame] | 17 | |
| 18 | ADIS16470DiagStatBuilder diag_stat_builder(fbb); |
| 19 | diag_stat_builder.add_clock_error(faulted); |
| 20 | diag_stat_builder.add_memory_failure(false); |
| 21 | diag_stat_builder.add_sensor_failure(false); |
| 22 | diag_stat_builder.add_standby_mode(false); |
| 23 | diag_stat_builder.add_spi_communication_error(false); |
| 24 | diag_stat_builder.add_flash_memory_update_error(false); |
| 25 | diag_stat_builder.add_data_path_overrun(false); |
| 26 | |
| 27 | const auto diag_stat_offset = diag_stat_builder.Finish(); |
| 28 | |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 29 | IMUValuesBuilder builder(fbb); |
| 30 | builder.add_gyro_x(gyro.x()); |
| 31 | builder.add_gyro_y(gyro.y()); |
| 32 | builder.add_gyro_z(gyro.z()); |
| 33 | builder.add_accelerometer_x(accel.x()); |
| 34 | builder.add_accelerometer_y(accel.y()); |
| 35 | builder.add_accelerometer_z(accel.z()); |
James Kuszmaul | ddc6970 | 2021-03-24 21:55:03 -0700 | [diff] [blame] | 36 | builder.add_previous_reading_diag_stat(diag_stat_offset); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 37 | fbb.Finish(builder.Finish()); |
| 38 | return fbb.Release(); |
| 39 | } |
| 40 | |
| 41 | // Tests that when we initialize everything is in a sane state. |
| 42 | TEST(ImuZeroerTest, InitializeUnzeroed) { |
| 43 | ImuZeroer zeroer; |
| 44 | ASSERT_FALSE(zeroer.Zeroed()); |
| 45 | ASSERT_FALSE(zeroer.Faulted()); |
| 46 | ASSERT_EQ(0.0, zeroer.GyroOffset().norm()); |
James Kuszmaul | 6d6e130 | 2022-03-12 15:22:48 -0800 | [diff] [blame] | 47 | ASSERT_EQ(0.0, zeroer.ZeroedGyro().value().norm()); |
| 48 | ASSERT_EQ(0.0, zeroer.ZeroedAccel().value().norm()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 49 | // A measurement before we are zeroed should just result in the measurement |
| 50 | // being passed through without modification. |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 51 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 52 | MakeMeasurement({0.01, 0.02, 0.03}, {4, 5, 6}).message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 53 | ASSERT_FALSE(zeroer.Zeroed()); |
| 54 | ASSERT_FALSE(zeroer.Faulted()); |
| 55 | ASSERT_EQ(0.0, zeroer.GyroOffset().norm()); |
James Kuszmaul | 6d6e130 | 2022-03-12 15:22:48 -0800 | [diff] [blame] | 56 | ASSERT_FLOAT_EQ(0.01, zeroer.ZeroedGyro().value().x()); |
| 57 | ASSERT_FLOAT_EQ(0.02, zeroer.ZeroedGyro().value().y()); |
| 58 | ASSERT_FLOAT_EQ(0.03, zeroer.ZeroedGyro().value().z()); |
| 59 | ASSERT_EQ(4.0, zeroer.ZeroedAccel().value().x()); |
| 60 | ASSERT_EQ(5.0, zeroer.ZeroedAccel().value().y()); |
| 61 | ASSERT_EQ(6.0, zeroer.ZeroedAccel().value().z()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 62 | } |
| 63 | |
| 64 | // Tests that we zero if we receive a bunch of identical measurements. |
| 65 | TEST(ImuZeroerTest, ZeroOnConstantData) { |
| 66 | ImuZeroer zeroer; |
| 67 | ASSERT_FALSE(zeroer.Zeroed()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 68 | for (size_t ii = 0; ii < kMinSamplesToZero; ++ii) { |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 69 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 70 | MakeMeasurement({0.01, 0.02, 0.03}, {4, 5, 6}).message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 71 | } |
| 72 | ASSERT_TRUE(zeroer.Zeroed()); |
| 73 | ASSERT_FALSE(zeroer.Faulted()); |
| 74 | // Gyro should be zeroed to {1, 2, 3}. |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 75 | ASSERT_FLOAT_EQ(0.01, zeroer.GyroOffset().x()); |
| 76 | ASSERT_FLOAT_EQ(0.02, zeroer.GyroOffset().y()); |
| 77 | ASSERT_FLOAT_EQ(0.03, zeroer.GyroOffset().z()); |
James Kuszmaul | 6d6e130 | 2022-03-12 15:22:48 -0800 | [diff] [blame] | 78 | ASSERT_EQ(0.0, zeroer.ZeroedGyro().value().x()); |
| 79 | ASSERT_EQ(0.0, zeroer.ZeroedGyro().value().y()); |
| 80 | ASSERT_EQ(0.0, zeroer.ZeroedGyro().value().z()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 81 | // Accelerometer readings should not be affected. |
James Kuszmaul | 6d6e130 | 2022-03-12 15:22:48 -0800 | [diff] [blame] | 82 | ASSERT_EQ(4.0, zeroer.ZeroedAccel().value().x()); |
| 83 | ASSERT_EQ(5.0, zeroer.ZeroedAccel().value().y()); |
| 84 | ASSERT_EQ(6.0, zeroer.ZeroedAccel().value().z()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 85 | // If we get another measurement offset by {1, 1, 1} we should read the result |
| 86 | // as {1, 1, 1}. |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 87 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 88 | MakeMeasurement({0.02, 0.03, 0.04}, {0, 0, 0}).message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 89 | ASSERT_FALSE(zeroer.Faulted()); |
James Kuszmaul | 6d6e130 | 2022-03-12 15:22:48 -0800 | [diff] [blame] | 90 | ASSERT_FLOAT_EQ(0.01, zeroer.ZeroedGyro().value().x()); |
| 91 | ASSERT_FLOAT_EQ(0.01, zeroer.ZeroedGyro().value().y()); |
| 92 | ASSERT_FLOAT_EQ(0.01, zeroer.ZeroedGyro().value().z()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 93 | } |
| 94 | |
| 95 | // Tests that we do not zero if the gyro is producing particularly high |
| 96 | // magnitude results. |
| 97 | TEST(ImuZeroerTest, NoZeroOnHighMagnitudeGyro) { |
| 98 | ImuZeroer zeroer; |
| 99 | ASSERT_FALSE(zeroer.Zeroed()); |
| 100 | for (size_t ii = 0; ii < kMinSamplesToZero; ++ii) { |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 101 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 102 | MakeMeasurement({0.1, 0.2, 0.3}, {4, 5, 6}).message()); |
| 103 | ASSERT_FALSE(zeroer.Zeroed()); |
| 104 | } |
| 105 | ASSERT_FALSE(zeroer.Faulted()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 106 | } |
| 107 | |
| 108 | // Tests that we tolerate small amounts of noise in the incoming data and can |
| 109 | // still zero. |
| 110 | TEST(ImuZeroerTest, ZeroOnLowNoiseData) { |
| 111 | ImuZeroer zeroer; |
| 112 | ASSERT_FALSE(zeroer.Zeroed()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 113 | for (size_t ii = 0; ii < kMinSamplesToZero; ++ii) { |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 114 | const double offset = |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 115 | (static_cast<double>(ii) / (kMinSamplesToZero - 1) - 0.5) * 0.001; |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 116 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 117 | MakeMeasurement({0.01 + offset, 0.02 + offset, 0.03 + offset}, |
Philipp Schrader | 790cb54 | 2023-07-05 21:06:52 -0700 | [diff] [blame] | 118 | {4 + offset, 5 + offset, 6 + offset}) |
| 119 | .message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 120 | } |
| 121 | ASSERT_TRUE(zeroer.Zeroed()); |
| 122 | ASSERT_FALSE(zeroer.Faulted()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 123 | ASSERT_NEAR(0.01, zeroer.GyroOffset().x(), 1e-3); |
| 124 | ASSERT_NEAR(0.02, zeroer.GyroOffset().y(), 1e-3); |
| 125 | ASSERT_NEAR(0.03, zeroer.GyroOffset().z(), 1e-3); |
| 126 | // If we get another measurement offset by {0.01, 0.01, 0.01} we should read |
| 127 | // the result as {0.01, 0.01, 0.01}. |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 128 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 129 | MakeMeasurement({0.02, 0.03, 0.04}, {0, 0, 0}).message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 130 | ASSERT_FALSE(zeroer.Faulted()); |
James Kuszmaul | 6d6e130 | 2022-03-12 15:22:48 -0800 | [diff] [blame] | 131 | ASSERT_NEAR(0.01, zeroer.ZeroedGyro().value().x(), 1e-3); |
| 132 | ASSERT_NEAR(0.01, zeroer.ZeroedGyro().value().y(), 1e-3); |
| 133 | ASSERT_NEAR(0.01, zeroer.ZeroedGyro().value().z(), 1e-3); |
| 134 | ASSERT_EQ(0.0, zeroer.ZeroedAccel().value().x()); |
| 135 | ASSERT_EQ(0.0, zeroer.ZeroedAccel().value().y()); |
| 136 | ASSERT_EQ(0.0, zeroer.ZeroedAccel().value().z()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 137 | } |
| 138 | |
| 139 | // Tests that we do not zero if there is too much noise in the input data. |
| 140 | TEST(ImuZeroerTest, NoZeroOnHighNoiseData) { |
| 141 | ImuZeroer zeroer; |
| 142 | ASSERT_FALSE(zeroer.Zeroed()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 143 | for (size_t ii = 0; ii < kMinSamplesToZero; ++ii) { |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 144 | ASSERT_FALSE(zeroer.Zeroed()); |
| 145 | const double offset = |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 146 | (static_cast<double>(ii) / (kMinSamplesToZero - 1) - 0.5) * 1.0; |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 147 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 148 | MakeMeasurement({0.01 + offset, 0.02 + offset, 0.03 + offset}, |
Philipp Schrader | 790cb54 | 2023-07-05 21:06:52 -0700 | [diff] [blame] | 149 | {4 + offset, 5 + offset, 6 + offset}) |
| 150 | .message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 151 | } |
| 152 | ASSERT_FALSE(zeroer.Zeroed()); |
| 153 | ASSERT_FALSE(zeroer.Faulted()); |
| 154 | } |
| 155 | |
| 156 | // Tests that we fault if we successfully rezero and get a significantly offset |
| 157 | // zero. |
| 158 | TEST(ImuZeroerTest, FaultOnNewZero) { |
| 159 | ImuZeroer zeroer; |
| 160 | ASSERT_FALSE(zeroer.Zeroed()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 161 | for (size_t ii = 0; ii < kMinSamplesToZero; ++ii) { |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 162 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 163 | MakeMeasurement({0.01, 0.02, 0.03}, {4, 5, 6}).message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 164 | } |
| 165 | ASSERT_TRUE(zeroer.Zeroed()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 166 | ASSERT_FALSE(zeroer.Faulted()) |
| 167 | << "We should not fault until we complete a second cycle of zeroing."; |
| 168 | for (size_t ii = 0; ii < kMinSamplesToZero; ++ii) { |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 169 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | ad2f8db | 2021-10-24 17:02:57 -0700 | [diff] [blame] | 170 | MakeMeasurement({0.01, -0.05, 0.03}, {4, 5, 6}).message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 171 | } |
| 172 | ASSERT_TRUE(zeroer.Faulted()); |
James Kuszmaul | ddc6970 | 2021-03-24 21:55:03 -0700 | [diff] [blame] | 173 | ASSERT_FALSE(zeroer.Zeroed()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 174 | } |
| 175 | |
| 176 | // Tests that we do not fault if the zero only changes by a small amount. |
| 177 | TEST(ImuZeroerTest, NoFaultOnSimilarZero) { |
| 178 | ImuZeroer zeroer; |
| 179 | ASSERT_FALSE(zeroer.Zeroed()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 180 | for (size_t ii = 0; ii < kMinSamplesToZero; ++ii) { |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 181 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 182 | MakeMeasurement({0.01, 0.02, 0.03}, {4, 5, 6}).message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 183 | } |
| 184 | ASSERT_TRUE(zeroer.Zeroed()); |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 185 | for (size_t ii = 0; ii < kMinSamplesToZero; ++ii) { |
James Kuszmaul | b7f45bb | 2020-02-26 20:27:48 -0800 | [diff] [blame] | 186 | zeroer.InsertAndProcessMeasurement( |
James Kuszmaul | b1e2937 | 2020-02-11 16:55:36 -0800 | [diff] [blame] | 187 | MakeMeasurement({0.01, 0.020001, 0.03}, {4, 5, 6}).message()); |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 188 | } |
| 189 | ASSERT_FALSE(zeroer.Faulted()); |
| 190 | } |
| 191 | |
James Kuszmaul | ddc6970 | 2021-03-24 21:55:03 -0700 | [diff] [blame] | 192 | // Tests that we fault on a bad diagnostic. |
| 193 | TEST(ImuZeroerTest, FaultOnBadDiagnostic) { |
| 194 | ImuZeroer zeroer; |
| 195 | ASSERT_FALSE(zeroer.Zeroed()); |
| 196 | ASSERT_FALSE(zeroer.Faulted()); |
| 197 | zeroer.InsertAndProcessMeasurement( |
| 198 | MakeMeasurement({0.01, 0.02, 0.03}, {4, 5, 6}, true).message()); |
| 199 | ASSERT_FALSE(zeroer.Zeroed()); |
| 200 | ASSERT_TRUE(zeroer.Faulted()); |
| 201 | } |
| 202 | |
James Kuszmaul | d3f9eb2 | 2020-01-12 15:02:07 -0800 | [diff] [blame] | 203 | } // namespace frc971::zeroing |