You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on May 9, 2019. It is now read-only.
Every vehicle system does not need to be responsible for juggling a list of optional sensors. A common sensor reader should be created that can read all available sensors, can perform sensor calibrations, and can reason about aggregate sensor health.
The text was updated successfully, but these errors were encountered:
@yoos any ideas on how to pull the calibration out of the vehicle system? Check out the branch to see the new aggregate sensor interface.
My current thought is to add a calibrateStep method to the Sensor interface that would be called every controller step. Then we can add a calibrated? check to the interface. The vehicle system would call calibrateStep until all active sensors have calibrated? = true. This would likely be implemented as a separate state in the system state machine.
switch(mode) {
case CALIBRATING:
sensors.calibrateStep();
if (sensors.calibrated()) {
mode = ...;
}
break;
case ...:
}
See comment in `sensor/sensor.hpp` - hopefully this axis config scheme is more
intuitive. Note this updates only the MPU-9250 driver. Also updates icarus axis
config.
Sensor calibration removes preset offsets while calibrating.
Calibration must remove preset offsets to measure the chip's inherent bias. This also means the Sensor's internal offset should not be modified until calibration is complete - otherwise we'd be calibrating against a moving target.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Every vehicle system does not need to be responsible for juggling a list of optional sensors. A common sensor reader should be created that can read all available sensors, can perform sensor calibrations, and can reason about aggregate sensor health.
The text was updated successfully, but these errors were encountered: