Skip to content

Commit

Permalink
fix(sensors): send the pressure sensor response with the baseline app…
Browse files Browse the repository at this point in the history
…lied (#783)

* send the pressure sensor response with the baseline applied

* aww dang missed a ;

* format

* keep signedness

* calculate the baseline correctly
  • Loading branch information
ryanthecoder authored Jun 4, 2024
1 parent 8738ed7 commit 83f8f09
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions include/sensors/core/tasks/pressure_driver.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,7 @@ class MMR920 {
}
}
if (bind_sync) {
if (std::fabs(pressure) - std::fabs(current_pressure_baseline_pa) >
if (std::fabs(pressure - current_pressure_baseline_pa) >
threshold_pascals) {
hardware.set_sync();
} else {
Expand All @@ -364,11 +364,10 @@ class MMR920 {
}

if (echo_this_time) {
auto response_pressure = pressure - current_pressure_baseline_pa;
#ifdef USE_PRESSURE_MOVE
// send a response with 9999 to make an overload of the buffer
// visible
if (pressure_buffer_index < PRESSURE_SENSOR_BUFFER_SIZE) {
(*p_buff).at(pressure_buffer_index) = pressure;
(*p_buff).at(pressure_buffer_index) = response_pressure;
pressure_buffer_index++;
}
#else
Expand All @@ -378,7 +377,8 @@ class MMR920 {
.message_index = m.message_index,
.sensor = can::ids::SensorType::pressure,
.sensor_id = sensor_id,
.sensor_data = mmr920::reading_to_fixed_point(pressure)});
.sensor_data =
mmr920::reading_to_fixed_point(response_pressure)});
#endif
}
}
Expand Down

0 comments on commit 83f8f09

Please sign in to comment.