sensors: baro aggregator limit publication rate with new parameter SENS_BARO_RATE #15309
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@priseborough as discussed, this is a simple change to the barometer aggregator in the sensors hub (sensors/vehicle_air_data) that limits the publication rate of
vehicle_air_data
(primary baro static pressure, altitude, temperature) to something reasonable (SENS_BARO_RATE default is 20 Hz). If the underlying data exceeds the limit then it's accumulated and averaged until the next allowed publication.This is a small optimization to limit sending unnecessary data to the estimator. Saves a small amount of work per ekf2 instance, logging, and helps with replay logging. In cases where the sensor_baro publication rate is relatively high (nearly 100 Hz with the ms5611) it reduces the nominal default logging rate (estimator replay enabled) by 8%.
The will be done for magnetometer and airspeed as well.